Date: Tue, 21 Feb 2023 02:33:10 +0800 From: Po-Chuan Hsieh <sunpoet@freebsd.org> To: Daniel Engberg <diizzy@freebsd.org> Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 23e0e7b1c30f - main - multimedia/mediainfo: Use all-in-one tarball Message-ID: <CAMHz58Qi9mdddkE0f30N_2J-7QE628DkRt499iYRQMsvhyoT4g@mail.gmail.com> In-Reply-To: <a499acc17d7f0a4151dcc459f09bffef@FreeBSD.org> References: <202301301306.30UD63Np053142@gitrepo.freebsd.org> <a499acc17d7f0a4151dcc459f09bffef@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000527dfb05f525e77b Content-Type: text/plain; charset="UTF-8" On Sat, Feb 11, 2023 at 3:43 PM Daniel Engberg <diizzy@freebsd.org> wrote: > Hi, > > Porters Handbook (indirectly) recommends you to sanitize sources. > > https://docs.freebsd.org/en/books/porters-handbook/book/#slow-patch-automatic-replacements Do you mean the removal of USES=dos2unix? This port does not install any text file but only 1 binary executable. Therefore USES=dos2unix is unnecessary. I also checked that the generated bin/mediainfo are identical. > Best regards, > Daniel > > On 2023-01-30 14:06, Po-Chuan Hsieh wrote: > > The branch main has been updated by sunpoet: > > > > URL: > > > https://cgit.FreeBSD.org/ports/commit/?id=23e0e7b1c30fc6f86957a7428e51a5ea9c594291 > > > > commit 23e0e7b1c30fc6f86957a7428e51a5ea9c594291 > > Author: Po-Chuan Hsieh <sunpoet@FreeBSD.org> > > AuthorDate: 2023-01-30 12:27:47 +0000 > > Commit: Po-Chuan Hsieh <sunpoet@FreeBSD.org> > > CommitDate: 2023-01-30 12:59:06 +0000 > > > > multimedia/mediainfo: Use all-in-one tarball > > > > - Update WWW > > - Remove unnecessary USES=dos2unix > > --- > > multimedia/mediainfo/Makefile | 20 +++++++++----------- > > multimedia/mediainfo/distinfo | 6 +++--- > > 2 files changed, 12 insertions(+), 14 deletions(-) > > > > diff --git a/multimedia/mediainfo/Makefile > > b/multimedia/mediainfo/Makefile > > index 5766752579b5..9b7c3737d89f 100644 > > --- a/multimedia/mediainfo/Makefile > > +++ b/multimedia/mediainfo/Makefile > > @@ -1,13 +1,14 @@ > > PORTNAME= mediainfo > > -DISTVERSION= 22.12 > > +PORTVERSION= 22.12 > > CATEGORIES= multimedia > > -MASTER_SITES= > https://mediaarea.net/download/source/${PORTNAME}/${DISTVERSION}/ > > \ > > +MASTER_SITES= > https://mediaarea.net/download/binary/mediainfo/${PORTVERSION}/ > > \ > > LOCAL/sunpoet > > -DISTNAME= ${PORTNAME}_${DISTVERSION} > > +DISTNAME= MediaInfo_CLI_${PORTVERSION}_GNU_FromSource > > > > MAINTAINER= sunpoet@FreeBSD.org > > COMMENT= Provides technical and tag information for video and audio > > files > > -WWW= https://mediaarea.net/en/MediaInfo > > +WWW= https://mediaarea.net/en/MediaInfo \ > > + https://github.com/MediaArea/MediaInfo > > > > LICENSE= BSD2CLAUSE > > LICENSE_FILE= ${WRKSRC}/../../../License.html > > @@ -15,17 +16,14 @@ LICENSE_FILE= ${WRKSRC}/../../../License.html > > LIB_DEPENDS= libmediainfo.so:multimedia/libmediainfo \ > > libzen.so:multimedia/libzen > > > > -USES= autoreconf compiler:c++11-lang cpe dos2unix > libtool pathfix \ > > - pkgconfig tar:xz > > +USES= compiler:c++11-lang cpe libtool pathfix pkgconfig > tar:xz > > > > GNU_CONFIGURE= yes > > -USE_CXXSTD= c++11 > > USE_LDCONFIG= yes > > -CPE_VENDOR= mediaarea > > -DOS2UNIX_GLOB= *.c *.cpp *.h > > -DOS2UNIX_WRKSRC= ${WRKDIR} > > > > PLIST_FILES= bin/mediainfo > > -WRKSRC= ${WRKDIR}/MediaInfo/Project/GNU/CLI > > +WRKSRC= > ${WRKDIR}/MediaInfo_CLI_GNU_FromSource/MediaInfo/Project/GNU/CLI > > + > > +CPE_VENDOR= mediaarea > > > > .include <bsd.port.mk> > > diff --git a/multimedia/mediainfo/distinfo > > b/multimedia/mediainfo/distinfo > > index 0d9caebdc92b..995778948454 100644 > > --- a/multimedia/mediainfo/distinfo > > +++ b/multimedia/mediainfo/distinfo > > @@ -1,3 +1,3 @@ > > -TIMESTAMP = 1672606752 > > -SHA256 (mediainfo_22.12.tar.xz) = > > 932b82739f738e7db603cf5bb170720731a9e7c61d145c2a54aabb3cd0b753bc > > -SIZE (mediainfo_22.12.tar.xz) = 2026408 > > +TIMESTAMP = 1674589524 > > +SHA256 (MediaInfo_CLI_22.12_GNU_FromSource.tar.xz) = > > e67ce7419ec6af9fb54ffbe3164a42ae1cd874a7e559bfff017868cef1106947 > > +SIZE (MediaInfo_CLI_22.12_GNU_FromSource.tar.xz) = 3352944 > --000000000000527dfb05f525e77b Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr">On Sat, Feb 11, 2023 at 3:43 PM Daniel En= gberg <<a href=3D"mailto:diizzy@freebsd.org" target=3D"_blank">diizzy@fr= eebsd.org</a>> wrote:<br></div><div class=3D"gmail_quote"><blockquote cl= ass=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid= rgb(204,204,204);padding-left:1ex">Hi,<br> <br> Porters Handbook (indirectly) recommends you to sanitize sources.<br> <a href=3D"https://docs.freebsd.org/en/books/porters-handbook/book/#slow-pa= tch-automatic-replacements" rel=3D"noreferrer" target=3D"_blank">https://do= cs.freebsd.org/en/books/porters-handbook/book/#slow-patch-automatic-replace= ments</a></blockquote><div><br></div><div>Do you mean the removal of USES= =3Ddos2unix?</div><div>This port does not install any text file but only 1 = binary executable.</div><div>Therefore USES=3Ddos2unix is unnecessary.</div= ><div><br></div><div>I also checked that the generated=C2=A0bin/mediainfo a= re identical.</div><div><br></div><blockquote class=3D"gmail_quote" style= =3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding= -left:1ex"> <br> Best regards,<br> Daniel<br> <br> On 2023-01-30 14:06, Po-Chuan Hsieh wrote:<br> > The branch main has been updated by sunpoet:<br> > <br> > URL:<br> > <a href=3D"https://cgit.FreeBSD.org/ports/commit/?id=3D23e0e7b1c30fc6f= 86957a7428e51a5ea9c594291" rel=3D"noreferrer" target=3D"_blank">https://cgi= t.FreeBSD.org/ports/commit/?id=3D23e0e7b1c30fc6f86957a7428e51a5ea9c594291</= a><br> > <br> > commit 23e0e7b1c30fc6f86957a7428e51a5ea9c594291<br> > Author:=C2=A0 =C2=A0 =C2=A0Po-Chuan Hsieh <sunpoet@FreeBSD.org><= br> > AuthorDate: 2023-01-30 12:27:47 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Po-Chuan Hsieh <sunpoet@FreeBSD.org><= br> > CommitDate: 2023-01-30 12:59:06 +0000<br> > <br> >=C2=A0 =C2=A0 =C2=A0multimedia/mediainfo: Use all-in-one tarball<br> > <br> >=C2=A0 =C2=A0 =C2=A0- Update WWW<br> >=C2=A0 =C2=A0 =C2=A0- Remove unnecessary USES=3Ddos2unix<br> > ---<br> >=C2=A0 multimedia/mediainfo/Makefile | 20 +++++++++-----------<br> >=C2=A0 multimedia/mediainfo/distinfo |=C2=A0 6 +++---<br> >=C2=A0 2 files changed, 12 insertions(+), 14 deletions(-)<br> > <br> > diff --git a/multimedia/mediainfo/Makefile <br> > b/multimedia/mediainfo/Makefile<br> > index 5766752579b5..9b7c3737d89f 100644<br> > --- a/multimedia/mediainfo/Makefile<br> > +++ b/multimedia/mediainfo/Makefile<br> > @@ -1,13 +1,14 @@<br> >=C2=A0 PORTNAME=3D=C2=A0 =C2=A0 mediainfo<br> > -DISTVERSION=3D 22.12<br> > +PORTVERSION=3D 22.12<br> >=C2=A0 CATEGORIES=3D=C2=A0 multimedia<br> > -MASTER_SITES=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 <a href=3D"https://mediaar= ea.net/download/source/$%7BPORTNAME%7D/$%7BDISTVERSION%7D/" rel=3D"noreferr= er" target=3D"_blank">https://mediaarea.net/download/source/${PORTNAME}/${D= ISTVERSION}/</a><br> > \<br> > +MASTER_SITES=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 <a href=3D"https://mediaar= ea.net/download/binary/mediainfo/$%7BPORTVERSION%7D/" rel=3D"noreferrer" ta= rget=3D"_blank">https://mediaarea.net/download/binary/mediainfo/${PORTVERSI= ON}/</a> <br> > \<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0LOCAL/sunpoet<br= > > -DISTNAME=3D=C2=A0 =C2=A0 ${PORTNAME}_${DISTVERSION}<br> > +DISTNAME=3D=C2=A0 =C2=A0 MediaInfo_CLI_${PORTVERSION}_GNU_FromSource<= br> > <br> >=C2=A0 MAINTAINER=3D=C2=A0 sunpoet@FreeBSD.org<br> >=C2=A0 COMMENT=3D=C2=A0 =C2=A0 =C2=A0Provides technical and tag informa= tion for video and audio <br> > files<br> > -WWW=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0<a href=3D"https://mediaarea.= net/en/MediaInfo" rel=3D"noreferrer" target=3D"_blank">https://mediaarea.ne= t/en/MediaInfo</a><br> > +WWW=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0<a href=3D"https://mediaarea.= net/en/MediaInfo" rel=3D"noreferrer" target=3D"_blank">https://mediaarea.ne= t/en/MediaInfo</a> \<br> > +=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0<a href=3D"https://gi= thub.com/MediaArea/MediaInfo" rel=3D"noreferrer" target=3D"_blank">https://= github.com/MediaArea/MediaInfo</a><br> > <br> >=C2=A0 LICENSE=3D=C2=A0 =C2=A0 =C2=A0BSD2CLAUSE<br> >=C2=A0 LICENSE_FILE=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 ${WRKSRC}/../../../Li= cense.html<br> > @@ -15,17 +16,14 @@ LICENSE_FILE=3D=C2=A0 =C2=A0 =C2=A0${WRKSRC}/../..= /../License.html<br> >=C2=A0 LIB_DEPENDS=3D libmediainfo.so:multimedia/libmediainfo \<br> >=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0libzen.so:multim= edia/libzen<br> > <br> > -USES=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 autore= conf compiler:c++11-lang cpe dos2unix libtool pathfix \<br> > -=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0pkgconfig tar:xz<br> > +USES=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 compil= er:c++11-lang cpe libtool pathfix pkgconfig tar:xz<br> > <br> >=C2=A0 GNU_CONFIGURE=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0yes<br> > -USE_CXXSTD=3D=C2=A0 c++11<br> >=C2=A0 USE_LDCONFIG=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 yes<br> > -CPE_VENDOR=3D=C2=A0 mediaarea<br> > -DOS2UNIX_GLOB=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0*.c *.cpp *.h<br> > -DOS2UNIX_WRKSRC=3D=C2=A0 =C2=A0 =C2=A0${WRKDIR}<br> > <br> >=C2=A0 PLIST_FILES=3D bin/mediainfo<br> > -WRKSRC=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ${WRKDIR}/M= ediaInfo/Project/GNU/CLI<br> > +WRKSRC=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 ${WRKDIR}/M= ediaInfo_CLI_GNU_FromSource/MediaInfo/Project/GNU/CLI<br> > +<br> > +CPE_VENDOR=3D=C2=A0 mediaarea<br> > <br> >=C2=A0 .include <<a href=3D"http://bsd.port.mk" rel=3D"noreferrer" t= arget=3D"_blank">bsd.port.mk</a>><br> > diff --git a/multimedia/mediainfo/distinfo <br> > b/multimedia/mediainfo/distinfo<br> > index 0d9caebdc92b..995778948454 100644<br> > --- a/multimedia/mediainfo/distinfo<br> > +++ b/multimedia/mediainfo/distinfo<br> > @@ -1,3 +1,3 @@<br> > -TIMESTAMP =3D 1672606752<br> > -SHA256 (mediainfo_22.12.tar.xz) =3D<br> > 932b82739f738e7db603cf5bb170720731a9e7c61d145c2a54aabb3cd0b753bc<br> > -SIZE (mediainfo_22.12.tar.xz) =3D 2026408<br> > +TIMESTAMP =3D 1674589524<br> > +SHA256 (MediaInfo_CLI_22.12_GNU_FromSource.tar.xz) =3D<br> > e67ce7419ec6af9fb54ffbe3164a42ae1cd874a7e559bfff017868cef1106947<br> > +SIZE (MediaInfo_CLI_22.12_GNU_FromSource.tar.xz) =3D 3352944<br> </blockquote></div></div> --000000000000527dfb05f525e77b--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAMHz58Qi9mdddkE0f30N_2J-7QE628DkRt499iYRQMsvhyoT4g>