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

remote installs do not use original nuget sources #6

Closed
mwrock opened this issue Sep 16, 2014 · 5 comments
Closed

remote installs do not use original nuget sources #6

mwrock opened this issue Sep 16, 2014 · 5 comments
Labels
5 - Released The issue has been resolved, and released to the public for consumption Bug Issues where something has happened which was not expected or intended

Comments

@mwrock
Copy link
Member

mwrock commented Sep 16, 2014

see https://boxstarter.codeplex.com/discussions/547713

UPDATE: Original link gone but available here

@mwallner
Copy link
Member

That discussion is gone, however the issue seems to still exist.

I've just ran into the problem that Install-BoxstarterPackage does not use NugetSources from Boxstarter Config. (also checked that AppData\Roambing\Boxstarter - the version of Boxstarter that is being pushed to the remote host actually contains the correct boxstarter.config)

as far as I've seen there is no mechanism for handling this inside https://github.com/chocolatey/boxstarter/blob/master/Boxstarter.Chocolatey/Install-BoxstarterPackage.ps1 - maybe another/default config file is being used?
(will continue investigation...)

@mwallner
Copy link
Member

actually, Install-BoxstarterPackage.ps1 states the observed behavior..

 The .nupkg file for the provided package name is searched in the following locations and order:
 - .\BuildPackages relative to the parent directory of the module file
 - The Chocolatey community feed
 This can be configured by editing $($Boxstarter.BaseDir)\Boxstarter.Config

@mwallner
Copy link
Member

The "problem" seems to be the fact that the original Boxstarter.Chocolatey\Boxstarter.zip is being sent to the remote host - with the default boxstarter.config ..
I take this is probably intentional?=

@mwallner
Copy link
Member

mwallner commented Jul 18, 2018

alright - there seems to be a reason for this: when using Boxstarter to install on remote hosts (i.e. azure) - you probably won't be able to access an organization internal server .. so it makes sense to fallback to community repository + some packages on a "local feed" ..

@mwrock @ferventcoder @gep13 @pauby
If that's ok with you I'll put in a PR that adds another parameter -DelegateChocolatySources (we can talk about the name of that switch) in order to be able to pass the sources.

edit: actually it's DelegateChocoSources as of #320

@pauby pauby added Bug Issues where something has happened which was not expected or intended 4 - Done Code has been added to the repository, and has been reviewed by a team member labels Mar 12, 2019
@pauby
Copy link
Member

pauby commented Mar 12, 2019

This has been merged. Closing.

@pauby pauby closed this as completed Mar 12, 2019
@gep13 gep13 added 5 - Released The issue has been resolved, and released to the public for consumption and removed 4 - Done Code has been added to the repository, and has been reviewed by a team member labels Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 - Released The issue has been resolved, and released to the public for consumption Bug Issues where something has happened which was not expected or intended
Projects
None yet
Development

No branches or pull requests

4 participants