Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Perl not installed error #182

Closed
Circuit8 opened this issue May 23, 2024 · 3 comments
Closed

Perl not installed error #182

Circuit8 opened this issue May 23, 2024 · 3 comments

Comments

@Circuit8
Copy link
Contributor

Hi there. I'm running into an issue with exiftool-vendored versions 22 and above on Mac and Linux in Electron. When attempting to use exiftool v26 I get the following error:

Perl must be installed. Please add Perl to your $PATH and try again.
exiftool-vendored/src/ExifTool.ts:169:11

Looks like it's to do with the introduction of which("perl").

Downgrading to version 21 solves the issue.

Exiftool-vendored: v22-26
OS: Ubuntu 22 & Mac 12.6
Node: v20.x
Electron: v29.x

Let me know if you need any more info. Thanks again for the great work with exiftool-vendored.

@Circuit8 Circuit8 changed the title Perl not install Error Perl not install error May 23, 2024
@prantikseal
Copy link

facing the same error, worked in version 21 thanks

@Circuit8 Circuit8 changed the title Perl not install error Perl not installed error May 28, 2024
@mceachen
Copy link
Member

mceachen commented May 29, 2024

mrm@m1 test % node
Welcome to Node.js v20.13.1.
Type ".help" for more information.
> require("exiftool-vendored").exiftool.version().then(console.dir)
Promise {
  <pending>,
  [Symbol(async_id_symbol)]: 105,
  [Symbol(trigger_async_id_symbol)]: 90
}
> '12.84'

If you open a bug, at least provide some sort of reproduction. I've got to be able to reproduce the bug in order to know that I've addressed it. Feel free to reopen when/if you provide that.

Edit: I'll just assume this is on some random customer's computer, and that they fsck'ed up their $PATH. The next release will check specifically for the shebang path (/usr/bin/perl) before throwing, which I think should be a reasonable backstop in case PATH is empty for whatever reason.

mceachen added a commit that referenced this issue May 29, 2024
@Circuit8
Copy link
Contributor Author

Apologies for not providing a reproduction. 26.1.0 has seemed to fix the issue though, thanks for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants