-
Notifications
You must be signed in to change notification settings - Fork 22
Support for cache directory #9
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
base: master
Are you sure you want to change the base?
Conversation
|
Thanks for your interest in the project and for your pull request. I will review the commits next week and consider merging it. |
|
Hi fekir, I reviewed your code and everything appears to work for your use case. I found that using 2 paths was a little confusing because it's hard to describe what the paths mean. I thought about the problem a little more and came up with an alternate solution which I checked into the I added an option to save relative paths instead of absolute paths. This makes the entire "store" portable and allows you to move it to a new file path or a new computer. The only downside is that you must run Thanks |
|
Hi, thank you for your time. I'll be able to test it during this weekend, it may do the work, but having to run My problem is that I would like to use Chocolatey on a PC without an Internet connection. Your solution is less than ideal because I should Maybe the best solution, to avoid confusions between the two paths, would be to have a download command, that works like your original project, and a "change store command" that updates the path of all scripts to a given path. What do you think? |
|
There's no need to test my solution if you feel the compromise of needing to run I understand your scenario but I feel it may be an uncommon one. The reason I'm hesitant to merge your solution is because it can leave the resulting store in an unusable state until you move it to a machine that has the proper folder structure. I think this could lead to confusion about the purpose and proper use of this feature. I like your idea about having a second command for changing the store path. It decouples changing paths from downloading. The store path can be changed multiple times without re-downloading (like my solution) and it allows |
Update README.md
|
A possibly common application for this feature could be if ChocolateStore runs on a linux machine with Mono. The usual \\Share\directory paths won't work there when the files are being downloaded, but this format is still required for the Windows clients. The best way to approach all these issues, though, might be to put the downloaded files into the nuget packages themselves. That way there aren't involved any absolute paths at all. |
Hello,
I've used with success your Project, for downloading Chocolatey Packages.
My problem was that I have some computers on a different network wihtout an internet connection, so I had to copy the download folder from one server to another. Chocolatey was unable to upgrade the packages since the PATH in the .ps1 scripts was not right anymore.
This is why I added a third (optional) parameter to specify a "cache directory", where ChocolateStore is going to download all the files, but the path used in the .ps1 files is still going to be the directory path.