Re: NEW graphics/exiftool

2018-03-13 Thread Stuart Henderson 
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

2018-03-13 Thread patrick keshishian 
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

2018-03-13 Thread Stuart Henderson 
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

2018-03-13 Thread George Rosamond 
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

2018-03-13 Thread 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 ?
  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

Re: NEW graphics/exiftool

2018-03-12 Thread patrick keshishian 
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

2018-03-02 Thread George Rosamond 
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

2018-03-02 Thread 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

> 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
>

NEW graphics/exiftool

2018-03-02 Thread George Rosamond 
Attached is exiftool-10.82:

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

-- 


5822 F82D 665B 5C6A 915B FAD4 B014 1CEE 545A A6C6


exiftool-10.82.tgz
Description: Binary data