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

Feature request: improve installer message #35269

Closed
muescha opened this issue Jun 6, 2017 · 1 comment
Closed

Feature request: improve installer message #35269

muescha opened this issue Jun 6, 2017 · 1 comment

Comments

@muescha
Copy link
Contributor

muescha commented Jun 6, 2017

Description of feature/enhancement

i would like to see the installer message should include the command line for the command

Justification

my tries to run the command ....

/usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app
Black Screen: command "/usr/local/Caskroom/little-snitch/3.7.4/Little" not found.

'/usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app'
bash: /usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app: is a directory

open /usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app
The files /usr/local/Caskroom/little-snitch/3.7.4/Little, /Users/muescha/Work/github/WWDC/Snitch, and /Users/muescha/Work/github/WWDC/Installer.app do not exist.

open '/usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app'

Example use case

To complete the installation of Cask little-snitch, you must also
run the installer at

  '/usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app'

🍺  little-snitch was successfully installed!

to

To complete the installation of Cask little-snitch, you must also
run the installer at

  '/usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app'

with command:

  open '/usr/local/Caskroom/little-snitch/3.7.4/Little Snitch Installer.app'

🍺  little-snitch was successfully installed!
@miccal miccal added the awaiting maintainer feedback Issue needs response from a maintainer. label Jun 7, 2017
@vitorgalvao
Copy link
Member

I don’t think this is a problem, as you’re the only one I recall mentioning this. Also, open is not mandatory: the point is that you open the app manually, at your leisure.

#24377 is a better solution than shoehorning a message. If more people had complained about this, I’d be more inclined to add it.

@miccal miccal removed the awaiting maintainer feedback Issue needs response from a maintainer. label Jun 8, 2017
@lock lock bot locked as resolved and limited conversation to collaborators May 7, 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