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

Can't run Chocolatey GUI if "old" User Setting is in place #234

Closed
gep13 opened this issue Mar 6, 2015 · 0 comments
Closed

Can't run Chocolatey GUI if "old" User Setting is in place #234

gep13 opened this issue Mar 6, 2015 · 0 comments
Assignees
@gep13
Labels
Bug Issues where something has happened which was not expected or intended
Milestone
0.13.0

Comments

@gep13
Copy link
Member

gep13 commented Mar 6, 2015

As a result of upgrading from Chocolatey (was had a default installation path of c:\chocolatey) to new Chocolatey, the underlying Environment Variable for ChocolateyInstall has been changed. However, if you have a previous installation of ChocolateyGUI, it's internal User Setting will still be c:\chocolatey, which means that you will be met with this:

image

Which I don't think is a great user experience.

@RichiCoder1 what was the thought behind storing this value locally? Why can't we just use the environment variable, or the PATH variable that you are checking for here:

image

Why store it locally? This seems to limit options when these external settings are changed.

Thoughts?
@gep13 gep13 added this to the 0.12.5 milestone Mar 17, 2015
@gep13 gep13 added the Bug Issues where something has happened which was not expected or intended label Mar 17, 2015
@gep13 gep13 self-assigned this Mar 17, 2015
gep13 added a commit that referenced this issue Mar 18, 2015
@gep13
(GH-234) Removed User Setting
cdbf178 
- For chocolateyInstall, this has been replaced with a
ChocolateyConfigurationProvider class, instantiated through IoC, that
can provide this information as/when required
gep13 added a commit that referenced this issue Mar 18, 2015
@gep13
(GH-234) Added ChocolateyConfigurationProvider
9ac20d6 
- Which at the minute, only provide the ChocolateyInstall property,
however, it could be extended to include other ChocolateyConfiguration
properties, as/when required.
@gep13 gep13 modified the milestones: 0.12.5, 0.13 Mar 21, 2015
@gep13 gep13 closed this as completed Mar 24, 2015
RichiCoder1 pushed a commit to RichiCoder1/ChocolateyGUI that referenced this issue Jun 29, 2015
@gep13 @RichiCoder1
(chocolateyGH-234) Removed User Setting
7661a7a 
- For chocolateyInstall, this has been replaced with a
ChocolateyConfigurationProvider class, instantiated through IoC, that
can provide this information as/when required
RichiCoder1 pushed a commit to RichiCoder1/ChocolateyGUI that referenced this issue Jun 29, 2015
@gep13 @RichiCoder1
(chocolateyGH-234) Added ChocolateyConfigurationProvider
d2e4c96 
- Which at the minute, only provide the ChocolateyInstall property,
however, it could be extended to include other ChocolateyConfiguration
properties, as/when required.
@gep13 gep13 changed the title Can't run ChocolateyGUI if "old" User Setting is in place Can't run Chocolatey GUI if "old" User Setting is in place Sep 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gep13 gep13

Labels
Bug Issues where something has happened which was not expected or intended
Projects
None yet
Milestone
0.13.0
Development

No branches or pull requests
1 participant
@gep13