Re: NEW graphics/exiftool
On 2018/03/13 12:37, patrick keshishian wrote: > Not suggesting my opinion counts on this, but when I first sent in > this port (years ago), I was told to use CPAN releases only. Then > somewhat recently (maybe a year or so ago), I was told go with latest > version even if not on CPAN (i.e. author's releases). > > Maybe there should be an official guideline stating OBSD's preferred > position on this. I don't find anything on ports@ re exiftool suggesting using the non-CPAN versions. As far as ports goes it doesn't necessarily have to be a CPAN version, but the official guideline is "don't use development releases, betas, release candidates and so on unless there's a specific reason to do so". In the case of exiftool, I'm looking at the author's history page at https://www.sno.phy.queensu.ca/~phil/exiftool/history.html which seems pretty clear in recommending 10.80 (and in general terms, versions which are uploaded to CPAN) as production releases.
Re: NEW graphics/exiftool
Not suggesting my opinion counts on this, but when I first sent in this port (years ago), I was told to use CPAN releases only. Then somewhat recently (maybe a year or so ago), I was told go with latest version even if not on CPAN (i.e. author's releases). Maybe there should be an official guideline stating OBSD's preferred position on this. -pk On 3/13/18, Stuart Henderson wrote: > On 2018/03/13 12:20, George Rosamond wrote: >> Giovanni Bechis: >> > On 03/02/18 17:02, George Rosamond wrote: >> >> Giovanni Bechis: >> >>> On 03/02/18 16:31, George Rosamond wrote: >> Attached is exiftool-10.82: >> >> >>> Hi, exiftool lives in graphics/p5-Image-ExifTool, >> >>> if you want to update it, you are welcome. >> >>> Thanks >> >>> Giovanni >> >> >> >> Thanks Giovanni, was confused since it's both a Perl module and a >> >> stand-alone application. >> >> >> >> CC'g MAINTAINER. >> >> >> >> Diff updating to 10.82 attached. >> >> >> >> Changes: >> >> >> >> distinfo >> >> version bump in Makefile >> >> wrapped pkg/DESCR at 80 characters >> >> a few PLIST additions >> >> >> >> g >> >> >> > There is no 10.82 version on Cpan, I would rather update to 10.80 >> > instead. >> > ok ? >> >> Yes, in cpan it's 10.80. >> >> FWIW, the port's Makefile is using the author's www and distfile, which >> just bumped to 10.84 yesterday. My diff was based on the author's www. >> >> If it's going to stick with the cpan version, should it also use cpan >> for the distfile? > > "Note: The most recent production release is Version 10.80. > (Other versions are considered development releases, and are > not uploaded to CPAN.)" > > 10.80 is the right version to use. I think MASTER_SITES is ok like it is > (using cpan with author's site as a second option). > >
Re: NEW graphics/exiftool
On 2018/03/13 12:20, George Rosamond wrote: > Giovanni Bechis: > > On 03/02/18 17:02, George Rosamond wrote: > >> Giovanni Bechis: > >>> On 03/02/18 16:31, George Rosamond wrote: > Attached is exiftool-10.82: > > >>> Hi, exiftool lives in graphics/p5-Image-ExifTool, > >>> if you want to update it, you are welcome. > >>> Thanks > >>> Giovanni > >> > >> Thanks Giovanni, was confused since it's both a Perl module and a > >> stand-alone application. > >> > >> CC'g MAINTAINER. > >> > >> Diff updating to 10.82 attached. > >> > >> Changes: > >> > >> distinfo > >> version bump in Makefile > >> wrapped pkg/DESCR at 80 characters > >> a few PLIST additions > >> > >> g > >> > > There is no 10.82 version on Cpan, I would rather update to 10.80 instead. > > ok ? > > Yes, in cpan it's 10.80. > > FWIW, the port's Makefile is using the author's www and distfile, which > just bumped to 10.84 yesterday. My diff was based on the author's www. > > If it's going to stick with the cpan version, should it also use cpan > for the distfile? "Note: The most recent production release is Version 10.80. (Other versions are considered development releases, and are not uploaded to CPAN.)" 10.80 is the right version to use. I think MASTER_SITES is ok like it is (using cpan with author's site as a second option).
Re: NEW graphics/exiftool
Giovanni Bechis: > On 03/02/18 17:02, George Rosamond wrote: >> Giovanni Bechis: >>> On 03/02/18 16:31, George Rosamond wrote: Attached is exiftool-10.82: >>> Hi, exiftool lives in graphics/p5-Image-ExifTool, >>> if you want to update it, you are welcome. >>> Thanks >>> Giovanni >> >> Thanks Giovanni, was confused since it's both a Perl module and a >> stand-alone application. >> >> CC'g MAINTAINER. >> >> Diff updating to 10.82 attached. >> >> Changes: >> >> distinfo >> version bump in Makefile >> wrapped pkg/DESCR at 80 characters >> a few PLIST additions >> >> g >> > There is no 10.82 version on Cpan, I would rather update to 10.80 instead. > ok ? Yes, in cpan it's 10.80. FWIW, the port's Makefile is using the author's www and distfile, which just bumped to 10.84 yesterday. My diff was based on the author's www. If it's going to stick with the cpan version, should it also use cpan for the distfile? g
Re: NEW graphics/exiftool
On 03/02/18 17:02, George Rosamond wrote: > Giovanni Bechis: >> On 03/02/18 16:31, George Rosamond wrote: >>> Attached is exiftool-10.82: >>> >> Hi, exiftool lives in graphics/p5-Image-ExifTool, >> if you want to update it, you are welcome. >> Thanks >> Giovanni > > Thanks Giovanni, was confused since it's both a Perl module and a > stand-alone application. > > CC'g MAINTAINER. > > Diff updating to 10.82 attached. > > Changes: > > distinfo > version bump in Makefile > wrapped pkg/DESCR at 80 characters > a few PLIST additions > > g > There is no 10.82 version on Cpan, I would rather update to 10.80 instead. ok ? Cheers Giovanni Index: Makefile === RCS file: /var/cvs/ports/graphics/p5-Image-ExifTool/Makefile,v retrieving revision 1.41 diff -u -p -r1.41 Makefile --- Makefile 20 May 2017 07:43:10 - 1.41 +++ Makefile 13 Mar 2018 07:31:23 - @@ -2,7 +2,7 @@ COMMENT= read and write meta information in image/audio/video files -DISTNAME= Image-ExifTool-10.50 +DISTNAME= Image-ExifTool-10.80 CATEGORIES= graphics HOMEPAGE= http://owl.phy.queensu.ca/~phil/exiftool/ Index: distinfo === RCS file: /var/cvs/ports/graphics/p5-Image-ExifTool/distinfo,v retrieving revision 1.34 diff -u -p -r1.34 distinfo --- distinfo 20 May 2017 07:43:10 - 1.34 +++ distinfo 13 Mar 2018 07:31:30 - @@ -1,2 +1,2 @@ -SHA256 (Image-ExifTool-10.50.tar.gz) = M53Y93H2c/1sRI9KSzbPUP6OO/iE2pWu9s7B0YLAiFs= -SIZE (Image-ExifTool-10.50.tar.gz) = 4247879 +SHA256 (Image-ExifTool-10.80.tar.gz) = vPiWYAd22O85qGcIG0hCaDftXujKGbU92dhqMXnJPJM= +SIZE (Image-ExifTool-10.80.tar.gz) = 4347722 Index: pkg/DESCR === RCS file: /var/cvs/ports/graphics/p5-Image-ExifTool/pkg/DESCR,v retrieving revision 1.1.1.1 diff -u -p -r1.1.1.1 DESCR --- pkg/DESCR 2 May 2006 20:52:34 - 1.1.1.1 +++ pkg/DESCR 13 Mar 2018 07:29:12 - @@ -1,5 +1,5 @@ -ExifTool provides an extensible set of Perl modules to read and write meta -information in image, audio and video files. +ExifTool provides an extensible set of Perl modules to read and +write meta information in image, audio and video files. ExifTool supports many different file formats. ExifTool also extracts information from the maker notes of many digital cameras by various Index: pkg/PLIST === RCS file: /var/cvs/ports/graphics/p5-Image-ExifTool/pkg/PLIST,v retrieving revision 1.29 diff -u -p -r1.29 PLIST --- pkg/PLIST 20 May 2017 07:43:10 - 1.29 +++ pkg/PLIST 13 Mar 2018 07:29:12 - @@ -31,6 +31,8 @@ ${P5SITE}/Image/ExifTool/Charset.pm ${P5SITE}/Image/ExifTool/Charset/Arabic.pm ${P5SITE}/Image/ExifTool/Charset/Baltic.pm ${P5SITE}/Image/ExifTool/Charset/Cyrillic.pm +${P5SITE}/Image/ExifTool/Charset/DOSLatin1.pm +${P5SITE}/Image/ExifTool/Charset/DOSLatinUS.pm ${P5SITE}/Image/ExifTool/Charset/Greek.pm ${P5SITE}/Image/ExifTool/Charset/Hebrew.pm ${P5SITE}/Image/ExifTool/Charset/JIS.pm @@ -82,6 +84,7 @@ ${P5SITE}/Image/ExifTool/GIMP.pm ${P5SITE}/Image/ExifTool/GPS.pm ${P5SITE}/Image/ExifTool/GeoTiff.pm ${P5SITE}/Image/ExifTool/Geotag.pm +${P5SITE}/Image/ExifTool/GoPro.pm ${P5SITE}/Image/ExifTool/H264.pm ${P5SITE}/Image/ExifTool/HP.pm ${P5SITE}/Image/ExifTool/HTML.pm @@ -168,6 +171,7 @@ ${P5SITE}/Image/ExifTool/PostScript.pm ${P5SITE}/Image/ExifTool/PrintIM.pm ${P5SITE}/Image/ExifTool/Qualcomm.pm ${P5SITE}/Image/ExifTool/QuickTime.pm +${P5SITE}/Image/ExifTool/QuickTimeStream.pl ${P5SITE}/Image/ExifTool/README ${P5SITE}/Image/ExifTool/RIFF.pm ${P5SITE}/Image/ExifTool/RSRC.pm @@ -176,6 +180,7 @@ ${P5SITE}/Image/ExifTool/Radiance.pm ${P5SITE}/Image/ExifTool/Rawzor.pm ${P5SITE}/Image/ExifTool/Real.pm ${P5SITE}/Image/ExifTool/Reconyx.pm +${P5SITE}/Image/ExifTool/Red.pm ${P5SITE}/Image/ExifTool/Ricoh.pm ${P5SITE}/Image/ExifTool/Samsung.pm ${P5SITE}/Image/ExifTool/Sanyo.pm @@ -258,6 +263,7 @@ ${P5SITE}/Image/ExifTool/iWork.pm @man man/man3p/Image::ExifTool::GPS.3p @man man/man3p/Image::ExifTool::GeoTiff.3p @man man/man3p/Image::ExifTool::Geotag.3p +@man man/man3p/Image::ExifTool::GoPro.3p @man man/man3p/Image::ExifTool::H264.3p @man man/man3p/Image::ExifTool::HP.3p @man man/man3p/Image::ExifTool::HTML.3p @@ -343,6 +349,7 @@ ${P5SITE}/Image/ExifTool/iWork.pm @man man/man3p/Image::ExifTool::PrintIM.3p @man man/man3p/Image::ExifTool::Qualcomm.3p @man man/man3p/Image::ExifTool::QuickTime.3p +@man man/man3p/Image::ExifTool::QuickTimeStream.3p @man man/man3p/Image::ExifTool::RIFF.3p @man man/man3p/Image::ExifTool::RSRC.3p @man man/man3p/Image::ExifTool::RTF.3p @@ -350,6 +357,7 @@ ${P5SITE}/Image/ExifTool/iWork.pm @man man/man3p/Image::ExifTool::Rawzor.3p @man man/man3p/Image::ExifTool::Real.3p @man man/man3p/Image::ExifTool::Reconyx.3p +@man man/man3p/Image::ExifTool::Red.3p @man man/man3p/Image::ExifTool::Ric
Re: NEW graphics/exiftool
Looks OK to me. Sorry for the delay in reply. --patrick On 3/2/18, George Rosamond wrote: > Giovanni Bechis: >> On 03/02/18 16:31, George Rosamond wrote: >>> Attached is exiftool-10.82: >>> >> Hi, exiftool lives in graphics/p5-Image-ExifTool, >> if you want to update it, you are welcome. >> Thanks >> Giovanni > > Thanks Giovanni, was confused since it's both a Perl module and a > stand-alone application. > > CC'g MAINTAINER. > > Diff updating to 10.82 attached. > > Changes: > > distinfo > version bump in Makefile > wrapped pkg/DESCR at 80 characters > a few PLIST additions > > g > >
Re: NEW graphics/exiftool
Giovanni Bechis: > On 03/02/18 16:31, George Rosamond wrote: >> Attached is exiftool-10.82: >> > Hi, exiftool lives in graphics/p5-Image-ExifTool, > if you want to update it, you are welcome. > Thanks > Giovanni Thanks Giovanni, was confused since it's both a Perl module and a stand-alone application. CC'g MAINTAINER. Diff updating to 10.82 attached. Changes: distinfo version bump in Makefile wrapped pkg/DESCR at 80 characters a few PLIST additions g Index: Makefile === RCS file: /cvs/ports/graphics/p5-Image-ExifTool/Makefile,v retrieving revision 1.41 diff -u -p -r1.41 Makefile --- Makefile 20 May 2017 07:43:10 - 1.41 +++ Makefile 2 Mar 2018 15:57:30 - @@ -2,7 +2,7 @@ COMMENT= read and write meta information in image/audio/video files -DISTNAME= Image-ExifTool-10.50 +DISTNAME= Image-ExifTool-10.82 CATEGORIES= graphics HOMEPAGE= http://owl.phy.queensu.ca/~phil/exiftool/ Index: distinfo === RCS file: /cvs/ports/graphics/p5-Image-ExifTool/distinfo,v retrieving revision 1.34 diff -u -p -r1.34 distinfo --- distinfo 20 May 2017 07:43:10 - 1.34 +++ distinfo 2 Mar 2018 15:57:30 - @@ -1,2 +1,2 @@ -SHA256 (Image-ExifTool-10.50.tar.gz) = M53Y93H2c/1sRI9KSzbPUP6OO/iE2pWu9s7B0YLAiFs= -SIZE (Image-ExifTool-10.50.tar.gz) = 4247879 +SHA256 (Image-ExifTool-10.82.tar.gz) = NWUm/mEltgL9R7ZChyNcePVZ/QMwFXUIFGvAhRV4uio= +SIZE (Image-ExifTool-10.82.tar.gz) = 4354328 Index: pkg/DESCR === RCS file: /cvs/ports/graphics/p5-Image-ExifTool/pkg/DESCR,v retrieving revision 1.1.1.1 diff -u -p -r1.1.1.1 DESCR --- pkg/DESCR 2 May 2006 20:52:34 - 1.1.1.1 +++ pkg/DESCR 2 Mar 2018 15:57:30 - @@ -1,5 +1,5 @@ -ExifTool provides an extensible set of Perl modules to read and write meta -information in image, audio and video files. +ExifTool provides an extensible set of Perl modules to read and +write meta information in image, audio and video files. ExifTool supports many different file formats. ExifTool also extracts information from the maker notes of many digital cameras by various Index: pkg/PLIST === RCS file: /cvs/ports/graphics/p5-Image-ExifTool/pkg/PLIST,v retrieving revision 1.29 diff -u -p -r1.29 PLIST --- pkg/PLIST 20 May 2017 07:43:10 - 1.29 +++ pkg/PLIST 2 Mar 2018 15:57:30 - @@ -31,6 +31,8 @@ ${P5SITE}/Image/ExifTool/Charset.pm ${P5SITE}/Image/ExifTool/Charset/Arabic.pm ${P5SITE}/Image/ExifTool/Charset/Baltic.pm ${P5SITE}/Image/ExifTool/Charset/Cyrillic.pm +${P5SITE}/Image/ExifTool/Charset/DOSLatin1.pm +${P5SITE}/Image/ExifTool/Charset/DOSLatinUS.pm ${P5SITE}/Image/ExifTool/Charset/Greek.pm ${P5SITE}/Image/ExifTool/Charset/Hebrew.pm ${P5SITE}/Image/ExifTool/Charset/JIS.pm @@ -82,6 +84,7 @@ ${P5SITE}/Image/ExifTool/GIMP.pm ${P5SITE}/Image/ExifTool/GPS.pm ${P5SITE}/Image/ExifTool/GeoTiff.pm ${P5SITE}/Image/ExifTool/Geotag.pm +${P5SITE}/Image/ExifTool/GoPro.pm ${P5SITE}/Image/ExifTool/H264.pm ${P5SITE}/Image/ExifTool/HP.pm ${P5SITE}/Image/ExifTool/HTML.pm @@ -168,6 +171,7 @@ ${P5SITE}/Image/ExifTool/PostScript.pm ${P5SITE}/Image/ExifTool/PrintIM.pm ${P5SITE}/Image/ExifTool/Qualcomm.pm ${P5SITE}/Image/ExifTool/QuickTime.pm +${P5SITE}/Image/ExifTool/QuickTimeStream.pl ${P5SITE}/Image/ExifTool/README ${P5SITE}/Image/ExifTool/RIFF.pm ${P5SITE}/Image/ExifTool/RSRC.pm @@ -176,6 +180,7 @@ ${P5SITE}/Image/ExifTool/Radiance.pm ${P5SITE}/Image/ExifTool/Rawzor.pm ${P5SITE}/Image/ExifTool/Real.pm ${P5SITE}/Image/ExifTool/Reconyx.pm +${P5SITE}/Image/ExifTool/Red.pm ${P5SITE}/Image/ExifTool/Ricoh.pm ${P5SITE}/Image/ExifTool/Samsung.pm ${P5SITE}/Image/ExifTool/Sanyo.pm @@ -258,6 +263,7 @@ ${P5SITE}/Image/ExifTool/iWork.pm @man man/man3p/Image::ExifTool::GPS.3p @man man/man3p/Image::ExifTool::GeoTiff.3p @man man/man3p/Image::ExifTool::Geotag.3p +@man man/man3p/Image::ExifTool::GoPro.3p @man man/man3p/Image::ExifTool::H264.3p @man man/man3p/Image::ExifTool::HP.3p @man man/man3p/Image::ExifTool::HTML.3p @@ -343,6 +349,7 @@ ${P5SITE}/Image/ExifTool/iWork.pm @man man/man3p/Image::ExifTool::PrintIM.3p @man man/man3p/Image::ExifTool::Qualcomm.3p @man man/man3p/Image::ExifTool::QuickTime.3p +@man man/man3p/Image::ExifTool::QuickTimeStream.3p @man man/man3p/Image::ExifTool::RIFF.3p @man man/man3p/Image::ExifTool::RSRC.3p @man man/man3p/Image::ExifTool::RTF.3p @@ -350,6 +357,7 @@ ${P5SITE}/Image/ExifTool/iWork.pm @man man/man3p/Image::ExifTool::Rawzor.3p @man man/man3p/Image::ExifTool::Real.3p @man man/man3p/Image::ExifTool::Reconyx.3p +@man man/man3p/Image::ExifTool::Red.3p @man man/man3p/Image::ExifTool::Ricoh.3p @man man/man3p/Image::ExifTool::Samsung.3p @man man/man3p/Image::ExifTool::Sanyo.3p
Re: NEW graphics/exiftool
On 03/02/18 16:31, George Rosamond wrote: > Attached is exiftool-10.82: > Hi, exiftool lives in graphics/p5-Image-ExifTool, if you want to update it, you are welcome. Thanks Giovanni > DESCR: > > ExifTool is a customizable set of Perl modules plus a full-featured > application for reading and writing meta information in a wide > variety of files, including the maker note information of many > digital cameras by various manufacturers such as Canon, Casio, FLIR, > FujiFilm, GE, HP, JVC/Victor, Kodak, Leaf, Minolta/Konica-Minolta, > Nikon, Nintendo, Olympus/Epson, Panasonic/Leica, Pentax/Asahi, Phase > One, Reconyx, Ricoh, Samsung, Sanyo, Sigma/Foveon and Sony. > > Notes: there's a bunch of possible run dependency perl modules for > specific use cases but not necessary for the core application (excluding > the three Win32 ones): > > Archive::Zip (ZIP, DOCX, PPTX, XLSX, ODP, ODS, ODT, EIP, iWork) > Compress::Zlib (DNG, PNG, PDF, DCM, MIE and SWF files) > Digest::MD5 (PDF files, IPTC information, and JPG Extended XMP) > Digest::SHA (PDF with AES-256 encryption) > IO::Compress::Bzip2 (RWZ files) > Time::HiRes (for generating ProcessingTime tag) > POSIX::strptime (for inverse date/time conversion) > Time::Piece (alternative to POSIX::strptime) > Unicode::LineBreak (for column alignment of alternate-language output) > IO::Compress::RawDeflate (for writing FLIF images) > IO::Uncompress::RawInflate (for reading FLIF images) > > Tests all run clean, and tested on a few jpgs and pngs. > > Feedback welcome. > > g >