Regression in postinstall scripts

[MaximKalinin]

Regression in postinstall scripts

[mmaietta]

Excellent debugging, thank you! I'll take a look when I have a chance, potentially this week if I can find time after work.

[mmaietta]

Hmmm the post install script never changed, the only change was adding BundlePostInstallScriptPath and BundlePreInstallScriptPath keys and script names to the Info.plist. I'm not familiar with pkg and was only provided guidance on the expected behavior, it shouldn't have changed the $2 arg though AFAICT. What is the impact of the $2 argument not working as it previously was?

If a postinstall script was open -a $2 to autolauch the app, wouldn't /Applications/electron-builder-test.app be the correct arg?

[MaximKalinin]

What is the impact of the $2 argument not working as it previously was?

I am trying to access files that are placed inside the installed app folder. For that I access "${2}/electron-builder-test.app/resources/". It used to evaluate to "/Applications/electron-builder-test.app/resources/", but now it evaluates to "/Applications/electron-builder-test.app/electron-builder-test.app/resources/" which does not exist.

If a postinstall script was open -a $2 to autolauch the app, wouldn't /Applications/electron-builder-test.app be the correct arg?

The thing is, it used to be open -a "${2}/electron-builder-test.app".

[mmaietta]

I still think it's expected behavior from my quick research
https://stackoverflow.com/a/11636246/3498974

$0 - Script path
$1 - Package path
$2 - Target location
$3 - Target volume`

My guess is that adding BundlePostInstallScriptPath added the proper configuration for the postinstall script to properly run

[MaximKalinin]

I still think it's expected behavior from my quick research

Does it mean that all the previous versions were working incorrectly, passing /Applications as a second argument?

[mmaietta]

That's my current hypothesis. If you look through the PR code diff, there's nothing setting the path or changing the executable script itself. Rather, it's simply scanning the install scripts directory, identifying any that are preinstall/postinstall and adding that file entry into the Info.plist. It doesn't modify the pkg scripts' arguments itself.

[MaximKalinin]

What is the manual step for generating .pkg file? If I try pkgbuild --root --scripts --component-plist I will get the expected /Applications as a second argument.

If you look through the PR code diff, there's nothing setting the path or changing the executable script itself.

The bug is not necessarily introduced by this PR, it could just be revealed now.

[MaximKalinin]

Ok, I think I found the real issue. It actually runs the postinstall script twice. Once for the application and once for the component specified in plist file. This seems like a bug to me.