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

The 'PushToBlobFeed' call cannot find the NuPkgs #1736

Merged
merged 2 commits into from
Nov 15, 2017

Conversation

johnbeisner
Copy link
Contributor

@johnbeisner johnbeisner commented Nov 15, 2017

The 'PushToBlobFeed' call cannot find the NuPkgs because "OutputPath" changes.

@johnbeisner johnbeisner self-assigned this Nov 15, 2017
@johnbeisner johnbeisner reopened this Nov 15, 2017
The 'PushToBlobFeed' call cannot find the NuPkg because "OutputPath" is changed.
@johnbeisner johnbeisner changed the title "TargetFramework" is not needed: BuildTools.csproj The 'PushToBlobFeed' call cannot find the NuPkgs Nov 15, 2017
@@ -1,8 +1,8 @@
<?xml version="1.0" encoding="utf-8"?>
<Project ToolsVersion="14.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
<ItemGroup>
<ItemsToPushGlobPattern Include="$(OutputPath)Packages\Microsoft.NET.Sdk.*.nupkg" />
<ItemsToPushGlobPattern Include="$(OutputPath)Packages\Microsoft.NET.Build.Extensions.*.nupkg" />
<ItemsToPushGlobPattern Include="$(RepositoryRootDirectory)bin\$(Configuration)\Packages\Microsoft.NET.Sdk.*.nupkg" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is it outputpath does not work here? This is duplicating this information, which is not ideal.

Copy link
Member

@tannergooding tannergooding left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. We should ideally log a bug to figure out what is causing OutputPath to be modified.

@johnbeisner johnbeisner merged commit 1218d9d into dotnet:release/2.0.0 Nov 15, 2017
@johnbeisner johnbeisner deleted the FixPushToBlobFeed2 branch December 12, 2017 18:18
JL03-Yue pushed a commit that referenced this pull request Mar 19, 2024
[main] Update dependencies from dotnet/roslyn
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.

3 participants