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.

Sign up for GitHub

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

Jump to bottom

Enhance "Access to path is denied" message on no admin rights #177

Closed
thigg opened this issue Mar 19, 2015 · 5 comments
Closed

Enhance "Access to path is denied" message on no admin rights #177

thigg opened this issue Mar 19, 2015 · 5 comments
Assignees
@ferventcoder
Labels
5 - Released Improvement Priority_LOW
Milestone
0.9.9.9

Comments

@thigg
Copy link

thigg commented Mar 19, 2015

It is actually good, that chocolatey doesn't spam you with warnings if you don't have admin rights. Very useful if you intend to use it without.

But if there are no admin rights the message that a path wasn't accesable should be enhanced with something like:

You started chocolatey without admin rights, please try again from an elevated shell.

I am not to happy with this text but something like that should be pretty useful. Needed yesterday 10 minutes until I saw that my shell wasn't elevated.
@ferventcoder
Copy link
Member

Are you using choco 0.9.9? It does actually say this if you are an administrative user in a non-elevated shell. However the detection is not 100%. I'd be interested to know where you may fall with this.

@thigg
Copy link
Author

thigg commented Mar 19, 2015

Ehm... Wait, I see, its printed there in green. I am used to just read red error messages :D

@ferventcoder
Copy link
Member

Any color you would prefer there? Perhaps magenta for the warnings?

@thigg
Copy link
Author

thigg commented Mar 27, 2015

I am used to red for errors but there are a lot of colors... ;)
I think the most useful change would be to have it as first line... but I am not sure...

@ferventcoder
Copy link
Member

Let's go warning ?

@ferventcoder ferventcoder added this to the 0.9.9.9 milestone Jun 30, 2015
@ferventcoder ferventcoder self-assigned this Sep 18, 2015
ferventcoder added a commit that referenced this issue Sep 18, 2015
@ferventcoder
(GH-177) Warn when admin not running elevated.
1adf65a 
In some cases the message related to non-elevated shells will come
through, except when using blended options. Also, just an informative
message is not enough to let folks know they may need to do something.
Beef up the message a bit and give it a nice dark warning.
ferventcoder added a commit that referenced this issue Sep 18, 2015
@ferventcoder
Merge branch 'stable'
38d3565 
* stable:
  (GH-177) Warn when admin not running elevated.
  (GH-347) Unpack choco files prior to nuget pack
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ferventcoder ferventcoder

Labels
5 - Released Improvement Priority_LOW
Projects
None yet
Milestone
0.9.9.9
Development

No branches or pull requests
3 participants
@ferventcoder @gep13 @thigg