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

Possible solution for allowing user to override Nuget packages installation folder #7

Merged
merged 5 commits into from May 21, 2011

Conversation

staxmanade
Copy link
Contributor

initial idea of how to override the default installation directory. TODO: fix up all the packages to leverage env var (if you like this option).

@staxmanade
Copy link
Contributor Author

FYI: I only tested the new code to get an installation folder/path... Once I integrated it - I haven't done a complete test run.

Thoughts?

@staxmanade staxmanade closed this Apr 28, 2011
@staxmanade staxmanade reopened this Apr 28, 2011
@staxmanade
Copy link
Contributor Author

Damnit - that Comment & Close button is in the wrong place.

@ferventcoder
Copy link
Owner

Two notes:

  1. Always overwrite. Otherwise upgrades wouldn't work at all.
  2. Let's not stop for input. Add it to the message. When someone runs initialize-chocolately they can add an optional location parameter. Otherwise it could not be a silent install.

Concerning # 2: Knowing that you would want it to be silent, how do you handle chocolatey upgrades?

@staxmanade
Copy link
Contributor Author

I removed the "stop for input".
I also added some powershell module help to describe what's going on.

  1. Always overwrite. Otherwise upgrades wouldn't work at all.
    Could you elaborate on what you mean by that?

@staxmanade staxmanade closed this May 2, 2011
@staxmanade staxmanade reopened this May 2, 2011
@staxmanade
Copy link
Contributor Author

Damnit - that Comment & Close button is in the wrong place. (I did it again)

@ferventcoder
Copy link
Owner

Sure. When you upgrade chocolatey, it is just running initialize-chocolatey on top of itself.

@ferventcoder ferventcoder merged commit f1fb057 into ferventcoder:master May 21, 2011
@ferventcoder
Copy link
Owner

You can see your name up with v0.9.8.2 - thanks for bearing with me while I got back to this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants