-
-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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
DiffMerge Resources references /Applications #13934
Comments
That should be fixed upstream. Not like the script can’t know its own location, and hardcoding the path assuming every user installs to the same location is lazy and leads to these types of issues. We shouldn’t be altering the script, since that is a recipe for disaster when it changes. Labelled as upstream, and added to #12858 for fixing later. |
Until the upstream fix, the broken package should be removed. Workaround: curl -O http://download-us.sourcegear.com/DiffMerge/4.2.0/DiffMerge.4.2.0.697.intel.stable.pkg
sudo installer -pkg ~/Downloads/DiffMerge.4.2.0.697.intel.stable.pkg -target / |
Also add a file to the zap :delete stanza, and further DRY up the url stanza.
I just installed DiffMerge (brew cask install diffmerge), which places it in
/opt/homebrew-cask/Caskroom/diffmerge/4.2.0.697/DiffMerge.app
.This normally would not be a problem, but internal shell script,
DiffMerge.app/Contents/Resources/diffmerge.sh
, as well as the one in the Extras folder refers to its executable as:EXPECTED BEHAVIOR: If this application lives somewhere else, then the support scripts should be updated to refer to the correct location of the application. Thus
DIFFMERGE_PATH=/opt/homebrew-cask/Caskroom/diffmerge/4.2.0.697/DiffMerge.app
.The text was updated successfully, but these errors were encountered: