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

Add MEGAsync.app latest #8803

Closed
wants to merge 1 commit into from
Closed

Conversation

jeroenseegers
Copy link

This is a follow-up to #5338. I had no issues installing and running
the MEGAsync client using this caskfile.

This is a follow-up to Homebrew#5338. I had no issues installing and running
the MEGAsync client using this caskfile.
@jeroenseegers
Copy link
Author

Also, it seems that #5338 turned into a VMWare Fusion caskfile somewhere along the way.

@vitorgalvao
Copy link
Member

You had no issues because you’re likely linking apps to /Applications (with --appdir), which is part of the issue (since it’s not the default).

It doesn’t look like they’ll actually do anything about it anytime soon, though, and we since had similar cases that were solved, so please add to the cask (below app, with one empty line above it and none below):

caveats do
  <<-EOS.undent
    #{token} only works if called from /Applications, so you may need to install it with
      brew cask --appdir=/Applications install #{token}

    Alternatively, you can create an extra link in /Applications, after installing
      ln -s '#{staged_path}/MEGAsync.app' '/Applications/MEGAsync.app'
  EOS
end

@vitorgalvao
Copy link
Member

Thank you for the contribution. It will be merged in a different pull request due to some simple recommended alterations. Your contribution is still included (and still credited to you), with the appropriate modifications, when applicable.

Changes:

  • Added the mentioned caveat.

@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

Successfully merging this pull request may close these issues.

2 participants