-
Notifications
You must be signed in to change notification settings - Fork 255
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
Support sourceUri passed on nuget restore/install and dotnet restore package or other equivalent commands #10948
Comments
@aortiz-msft @nkolev92 |
Are you planning to validate both packages.config and PackageReference with this one? |
@nkolev92 |
I see I'll let you and @kartheekp-ms decide, but consider splitting up the work in 2 issues. |
For complete here are the scenarios of interest PC & PR.
Note that the last 2 are not relevant in PackageReference. There's a warning message when you try that. |
@nkolev92 |
You can do the following and it overrides the package sources specified in the nuget config. <Project Sdk="Microsoft.NET.Sdk">
<PropertyGroup>
<OutputType>Exe</OutputType>
<TargetFramework>net6.0</TargetFramework>
<RestoreSources>https://api.nuget.org/v3/index.json</RestoreSources>
</PropertyGroup>
<ItemGroup>
<PackageReference Include="nuget.common" Version="4.9.1"/>
</ItemGroup>
</Project> |
@nkolev92 @kartheekp-ms |
Yes, but I'd expect they'd be automatically handled. |
I created follow up issue for |
Details about Problem
NuGet product used (NuGet.exe | Visual Studio | MSBuild.exe | dotnet.exe):
Product version: N/A
Worked before? If so, with which NuGet version:
Repro steps and/or sample project
Address Karteek's comment about there are chances that users might pass -source option to the restore command.
To support this we may need small changes
SearchTree
,PackageDownloader.cs
andPackageNamespacesConfiguration.cs
files.
Verbose Logs
The text was updated successfully, but these errors were encountered: