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

dendroscope 3.3.2 (new formula) #12895

Closed
wants to merge 1 commit into from
Closed

Conversation

sjackman
Copy link
Member

No description provided.

@victorpopkov
Copy link
Member

Thank you for the contribution. It needed some fixes, so they were made in 36d61af. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

@sjackman sjackman deleted the dendroscope branch July 29, 2015 23:21
@sjackman
Copy link
Member Author

Thanks for the merge and fix, Victor.

@vitorgalvao
Copy link
Member

@victorpopkov two notes, here:

  1. It needs the # tuebingen.de is the official download host per the vendor homepage comment (we need to manually check download links when the host of url and homepage differ and have that comment as confirmation a maintainer verified it).
  2. Since it has installer :manual, it still needs an uninstall. installer :manual and pkg casks should not be merged until an uninstall is present, even if that means holding up the cask indefinitely.

@victorpopkov
Copy link
Member

Understood. Thank you @vitorgalvao for the explanations. The comment was added in #12915.

Before deciding, if it would be ok to merge without uninstall, I've tried to automate the both of installation and uninstallation processes. However, I didn't find the necessary solution that would fit this case. Furthermore, at that moment there were some casks already that didn't have the necessary uninstall when using the same installer :manual technique. For example:

  • adobe-creative-cloud.rb
  • backblaze.rb
  • default-folder-x.rb
  • fuze.rb
  • google-nik-collection.rb
  • little-snitch.rb
  • omnifocus-clip-o-tron.rb
  • ps3-media-server.rb
  • scout.rb
  • tuneup.rb
  • utorrent.rb
  • vlc-remote.rb

So at that point I have decided, that holding up the cask indefinitely wouldn't be ok and was referencing those casks. I will change the tactic for this cases from now on.

Thanks again for all the explanations!

@vitorgalvao
Copy link
Member

The comment is added in #12915.

Thank you. I’m currently without access to my computer on account of a broken charger, so I appreciate you making the change.

However, I didn't find the necessary solution that would fit this case.

Yeah, installer :manual is a pain.

So at that point I have decided, that holding up the cask indefinitely wouldn't be ok and was referencing those casks. I will change the tactic for this cases from now on.

You know what? On second thought, don’t hold them up. You made a judgement on this cask based on other casks, and I’d say you’re correct. I was under the impression we had uninstall in all installer :manual casks, but we don’t. We should strive to not make that list bigger, but it is true that finding uninstalls for these is a pain, so yeah, use your best judgement in these cases.

And thank you again for all the work you’ve been doing; it has been great.

@victorpopkov
Copy link
Member

The problem of manual installation and uninstallation processes that was discussed here earlier was solved in #13066. All the examples and jars for Dendroscope.app can be found now in /opt/dendroscope/<version>/.

@miccal miccal removed the cask 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

Successfully merging this pull request may close these issues.

4 participants