Version 12.42

Started by fxstein, June 06, 2022, 11:12:25 PM

Previous topic - Next topic

fxstein

Hi,

Quick question: When will 12.42 get published on MetaCPAN?

Trying to update my homebrew formula with the latest production version.

Thank you!

Oliver
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX

Phil Harvey

Hi Oliver,

Sorry, I forgot to upload the release to CPAN.  Done now.  It should appear shortly on MetaCPAN.

- Phil
...where DIR is the name of a directory/folder containing the images.  On Mac/Linux/PowerShell, use single quotes (') instead of double quotes (") around arguments containing a dollar sign ($).

Phil Harvey

Well, the fetch failed because apparently PAUSE can't handle secure https (I had forgotten that).  And it wouldn't let me manually upload the same file because it doesn't allow duplicate files of the same name (even though the fetch didn't work).  So I had to upload the new release with a different name:  Image-ExifTool-12.42.0.tar.gz

- Phil
...where DIR is the name of a directory/folder containing the images.  On Mac/Linux/PowerShell, use single quotes (') instead of double quotes (") around arguments containing a dollar sign ($).

fxstein

Thank you very much! I will later today try to create a pull request for homebrew to get the new release included for us Mac users.
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX

Phil Harvey

The failed fetch finally timed out and I was able to manually upload the package with the correct name (Image-ExifTool-12.42.tar.gz), so I have scheduled Image-ExifTool-12.42.0.tar.gz for deletion, and it should be removed tomorrow, but by then the package with the correct name should be available.

- Phil
...where DIR is the name of a directory/folder containing the images.  On Mac/Linux/PowerShell, use single quotes (') instead of double quotes (") around arguments containing a dollar sign ($).

fxstein

Meanwhile, I have started a PR for the new release. This might have overlapped your last post.

For reference here is the PR: https://github.com/Homebrew/homebrew-core/pull/103157

Also here is the syntax needed on MacOS with homebrew installed:

brew bump-formula-pr --version 12.42 exiftool

Really neat how the homebrew team has automated this and all the tests down to a single line.
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX

fxstein

Looks like the timing was good. It picked up the corrected filename and SHA256:

https://github.com/Homebrew/homebrew-core/pull/103157/files

Hopefully, this will get merged soon.

Again thank you very much for your help!
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX

fxstein

Ok the version was merged but I am getting the following error message when upgrading to 12.42 from 12.30:

/usr/local/bin/exiftool
Can't locate Image/ExifTool.pm in @INC (you may need to install the Image::ExifTool module) (@INC contains: /usr/local/bin/../Cellar/exiftool/12.42/bin/lib /Library/Perl/5.30/darwin-thread-multi-2level /Library/Perl/5.30 /Network/Library/Perl/5.30/darwin-thread-multi-2level /Network/Library/Perl/5.30 /Library/Perl/Updates/5.30.3 /System/Library/Perl/5.30/darwin-thread-multi-2level /System/Library/Perl/5.30 /System/Library/Perl/Extras/5.30/darwin-thread-multi-2level /System/Library/Perl/Extras/5.30) at /usr/local/bin/exiftool line 40.
BEGIN failed--compilation aborted at /usr/local/bin/exiftool line 40.


Any thoughts on what that could be?
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX

fxstein

Did some investigating...

For some reason 12.42 does not seem to work when run through a link.

When installed in
/usr/local/Cellar/exiftool/12.42/bin/exiftool
you can execute the tool directly from there without issues

However a link at
/usr/local/bin/exiftool -> ../Cellar/exiftool/12.42/bin/exiftool
will break the tool

The test:
/usr/local/Cellar/exiftool/12.42/bin/exiftool -ver
12.42

/usr/local/bin/exiftool -ver
Can't locate Image/ExifTool.pm in @INC (you may need to install the Image::ExifTool module) (@INC contains: /usr/local/bin/../Cellar/exiftool/12.42/bin/lib /Library/Perl/5.30/darwin-thread-multi-2level /Library/Perl/5.30 /Network/Library/Perl/5.30/darwin-thread-multi-2level /Network/Library/Perl/5.30 /Library/Perl/Updates/5.30.3 /System/Library/Perl/5.30/darwin-thread-multi-2level /System/Library/Perl/5.30 /System/Library/Perl/Extras/5.30/darwin-thread-multi-2level /System/Library/Perl/Extras/5.30) at /usr/local/bin/exiftool line 40.
BEGIN failed--compilation aborted at /usr/local/bin/exiftool line 40.


For 12.30 that was no issue and has been working. I wonder if something changed since 12.30 that treats paths/relative paths differently than before?
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX

Phil Harvey

#9
Odd.  This was supposed to be broken in 12.30 and fixed in 12.35:

Nov. 11, 2021 - Version 12.35
  - Fixed bug introduced in 11.91 where exiftool couldn't find its libraries
    when run via a soft link.  Also changed to look for config file in the link
    target directory instead of the directory of the link itself


And I get this, which is opposite to what you are seeing:

% ls -l
total 0
drwxr-xr-x  16 phil  staff  512 12 Aug  2021 Image-ExifTool-12.30
drwxr-xr-x  16 phil  staff  512  1 Jun 14:45 Image-ExifTool-12.42
lrwxr-xr-x   1 phil  staff   29  7 Jun 21:29 e1 -> Image-ExifTool-12.30/exiftool
lrwxr-xr-x   1 phil  staff   29  7 Jun 21:29 e2 -> Image-ExifTool-12.42/exiftool
% ./e1 -ver
Can't locate Image/ExifTool.pm in @INC (you may need to install the Image::ExifTool module) (@INC contains: ./lib /Library/Perl/5.18/darwin-thread-multi-2level /Library/Perl/5.18 /Network/Library/Perl/5.18/darwin-thread-multi-2level /Network/Library/Perl/5.18 /Library/Perl/Updates/5.18.4 /System/Library/Perl/5.18/darwin-thread-multi-2level /System/Library/Perl/5.18 /System/Library/Perl/Extras/5.18/darwin-thread-multi-2level /System/Library/Perl/Extras/5.18 .) at ./e1 line 37.
BEGIN failed--compilation aborted at ./e1 line 37.
% ./e2 -ver
12.42


This is on MacOS 10.14.6 running in a tcsh.

- Phil

Edit:  Ah.  I think I understand.  The exiftool "lib" directory must be in the same directory as the "exiftool" script if it isn't installed in one of the perl include directories.  So in your case /usr/local/Cellar/exiftool/12.42/bin should contain the "lib" directory with the ExifTool modules, but it seems as if this isn't the case.
...where DIR is the name of a directory/folder containing the images.  On Mac/Linux/PowerShell, use single quotes (') instead of double quotes (") around arguments containing a dollar sign ($).

fxstein

Turns out someone fixed the homebrew install for 12.30 and possibly earlier versions to replace the lib link as part of the install. Your fix simply broke their fix. A PR has already been started to fix that in homebrew but I am not sure the currently proposed fix is correct as it breaks on some of the test builds.

https://github.com/Homebrew/homebrew-core/pull/103182

I will check back later tomorrow if this has been resolved.
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX

fxstein

This PR has been merged and fixed on the homebrew core repository. Thanks for your help pointing me to the change in exiftool!
If you want to help fix GoPro and related EXIF metadata please check out: https://github.com/fxstein/GoProX