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

DiffMerge Resources references /Applications #13934

Closed
darkn3rd opened this issue Sep 22, 2015 · 2 comments
Closed

DiffMerge Resources references /Applications #13934

darkn3rd opened this issue Sep 22, 2015 · 2 comments

Comments

@darkn3rd
Copy link

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:

DIFFMERGE_PATH=/Applications/DiffMerge.app
DIFFMERGE_EXE=${DIFFMERGE_PATH}/Contents/MacOS/DiffMerge

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.

@darkn3rd darkn3rd changed the title DiffMerge Resources references /Appplications DiffMerge Resources references /Applications Sep 22, 2015
@vitorgalvao vitorgalvao added the upstream Issue which needs to be resolved by the upstream project. label Sep 22, 2015
@vitorgalvao
Copy link
Member

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.

@darkn3rd
Copy link
Author

darkn3rd commented Oct 9, 2015

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 /

Mohitpandey referenced this issue Apr 6, 2016
Also add a file to the zap :delete stanza, and further DRY up the url
stanza.
@miccal miccal removed the upstream Issue which needs to be resolved by the upstream project. label Dec 23, 2016
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants