Re: pkg/48083: devel/lua-gi Support to be able to get the file from github

[Arnaud Degroote <arnaud.degroote%laas.fr@localhost>]

The following reply was made to PR pkg/48083; it has been noted by GNATS.

From: Arnaud Degroote <arnaud.degroote%laas.fr@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: degroote%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, 
pkgsrc-bugs%netbsd.org@localhost,
        szptvlfn%a-net.email.ne.jp@localhost
Subject: Re: pkg/48083: devel/lua-gi Support to be able to get the file from
 github
Date: Thu, 25 Jul 2013 14:00:46 +0000

 --i9LlY+UWpKt15+FH
 Content-Type: multipart/mixed; boundary="sdtB3X0nJg68CQEu"
 Content-Disposition: inline
 
 
 --sdtB3X0nJg68CQEu
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline
 Content-Transfer-Encoding: quoted-printable
 
 On 25/Jul - 11:50, Arnaud Degroote wrote:
 > The following reply was made to PR pkg/48083; it has been noted by GNATS.
 >=20
 > From: Arnaud Degroote <degroote%netbsd.org@localhost>
 > To: gnats-bugs%netbsd.org@localhost
 > Cc: degroote%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, 
 > pkgsrc-bugs%netbsd.org@localhost,
 >         szptvlfn%a-net.email.ne.jp@localhost
 > Subject: Re: pkg/48083: devel/lua-gi Support to be able to get the file f=
 rom
 >  github
 > Date: Thu, 25 Jul 2013 13:47:38 +0200
 >=20
 >  --ZPt4rx8FFjLCG7dd
 >  Content-Type: text/plain; charset=3Dus-ascii
 >  Content-Disposition: inline
 >  Content-Transfer-Encoding: quoted-printable
 > =20
 >  On 25/Jul - 11:10, OBATA Akio wrote:
 >  > The following reply was made to PR pkg/48083; it has been noted by GNA=
 TS.
 >  >=3D20
 >  > From: "OBATA Akio" <obache%netbsd.org@localhost>
 >  > To: gnats-bugs%netbsd.org@localhost
 >  > Cc:=3D20
 >  > Subject: Re: pkg/48083: devel/lua-gi Support to be able to get the fil=
 e f=3D
 >  rom
 >  >  github
 >  > Date: Thu, 25 Jul 2013 20:07:57 +0900
 >  >=3D20
 >  >  >  I don't understand -- why is this an improvement?
 >  > =3D20
 >  >  Currently, DISTNAME=3D3D0.7.1, so 0.7.1.tar.gz will be put in ${DISTD=
 IR}.
 >  >  It is unwanted filename (at least, DIST_SUBDIR should be set).
 >  >  Provided filename from https download server is lgi.0.7.1.tar.gz,
 >  >  so it should be used instead.
 > =20
 >  Yes, you are right. We definitively need curl or wget to download the
 >  https:// archive. And I don't think about it in the first place but yes,
 >  0.7.1.tar.gz is not a good filename for ${DISTDIR}. I will merge your
 >  patch if no objection from Thomas.
 
 The attached patch should fix the two issues. Is it ok for everyone ?=20
 
 --sdtB3X0nJg68CQEu
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename=patch_lua-gi
 Content-Transfer-Encoding: quoted-printable
 
 ? work
 Index: Makefile
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
 RCS file: /cvsroot/pkgsrc/devel/lua-gi/Makefile,v
 retrieving revision 1.1
 diff -u -r1.1 Makefile
 --- Makefile   22 Jul 2013 19:38:07 -0000      1.1
 +++ Makefile   25 Jul 2013 11:59:14 -0000
 @@ -1,11 +1,11 @@
  # $NetBSD: Makefile,v 1.1 2013/07/22 19:38:07 degroote Exp $
  #
 =20
 +DISTNAME=3D   lgi-${VERSION}
  PKGNAME=3D    lua-gi-${VERSION}
  VERSION=3D    0.7.1
 -DISTNAME=3D   ${VERSION}
 -CATEGORIES=3D         devel
 -MASTER_SITES=3D       http://github.com/pavouk/lgi/archive/
 +CATEGORIES=3D devel
 +MASTER_SITES=3D       
-https://codeload.github.com/pavouk/lgi/tar.gz/${VERSION}
 =20
  MAINTAINER=3D pkgsrc-users%NetBSD.org@localhost
  HOMEPAGE=3D   https://github.com/pavouk/lgi
 @@ -14,8 +14,7 @@
 =20
  USE_LANGUAGES+=3D     c
  USE_TOOLS+=3D pkg-config gmake
 -
 -WRKSRC=3D             ${WRKDIR}/lgi-${VERSION}
 +FETCH_USING=3D        curl
 =20
  .include "../../devel/glib2/buildlink3.mk"
  .include "../../devel/libffi/buildlink3.mk"
 Index: distinfo
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
 RCS file: /cvsroot/pkgsrc/devel/lua-gi/distinfo,v
 retrieving revision 1.1
 diff -u -r1.1 distinfo
 --- distinfo   22 Jul 2013 19:38:07 -0000      1.1
 +++ distinfo   25 Jul 2013 11:59:14 -0000
 @@ -1,6 +1,6 @@
  $NetBSD: distinfo,v 1.1 2013/07/22 19:38:07 degroote Exp $
 =20
 -SHA1 (0.7.1.tar.gz) =3D ae1268852b38507c9324e89aa3d950602b5ae8e1
 -RMD160 (0.7.1.tar.gz) =3D 87852dfbc6f3f9944df56324aadbaab6e3b2b802
 -Size (0.7.1.tar.gz) =3D 268327 bytes
 +SHA1 (lgi-0.7.1.tar.gz) =3D ae1268852b38507c9324e89aa3d950602b5ae8e1
 +RMD160 (lgi-0.7.1.tar.gz) =3D 87852dfbc6f3f9944df56324aadbaab6e3b2b802
 +Size (lgi-0.7.1.tar.gz) =3D 268327 bytes
  SHA1 (patch-Makefile) =3D ecba51fb9bc78c08e89bebfc981206a2587e63f4
 
 --sdtB3X0nJg68CQEu--
 
 --i9LlY+UWpKt15+FH
 Content-Type: application/pgp-signature
 
 -----BEGIN PGP SIGNATURE-----
 Version: GnuPG v1.4.13 (NetBSD)
 
 iQEcBAEBAgAGBQJR8S+OAAoJEAeVGKMo0YKehWMH/in4ylNuuKULbrKCCWmWlUUM
 akuEb+AhXehXEoe8BDn6bVfpuF34OqUYTrzEV0gMqWuxj1Dx6q0R0i/omjCpU3/X
 rBRcWrnAXofJq57xbxM2QeBohaGuRFbNSbh6t6FAEHxMKQwt1cY+VwTgbPMD1RTi
 jORFQ4Jg5VDpZRpyJ4d0iDPd0c8mS4GXTjp4ejh0BzklAsUvH3ee9saWyovvU6WG
 j/LGxF/uY6vfiw+D3aPF0+Rkmy3lbX7n2zo58eonZdihfo8yN4YMW3ptsy2xTw6D
 PmR1+P3xGx3jNRE7Svd5EQoKOc68h4LEdgKaH49TxbW+nccc9JlgrRS3CKUsQJU=
 =6glq
 -----END PGP SIGNATURE-----
 
 --i9LlY+UWpKt15+FH--