Skip to content

Conversation

@wli3
Copy link

@wli3 wli3 commented May 22, 2019

NuGet/Home#7801

(the easiest way to understand that is)since nuget no longer sets NoBuild on GeneratePackageOnBuild, we need to do similar logic to get the previous state.

fix #3253

@wli3 wli3 requested review from dsplaisted and peterhuene May 22, 2019 01:12
@wli3 wli3 added this to the 3.0.1xx milestone May 22, 2019
@wli3
Copy link
Author

wli3 commented May 22, 2019

(circular dependency come from circular bug, that's so meta)

@dsplaisted
Copy link
Member

@wli3 I merged #3252, so you should be good to rebase this on top of it and then re-enable the tests.

William Li added 2 commits May 21, 2019 22:27
NuGet/Home#7801

since nuget no longer sets NoBuild on GeneratePackageOnBuild we need to do similar logic to get the previous state.
@wli3 wli3 force-pushed the fix-GeneratePackageOnBuild-pack-as-tool branch from 988c8cb to d59a547 Compare May 22, 2019 05:27
@wli3
Copy link
Author

wli3 commented May 22, 2019

done

@wli3
Copy link
Author

wli3 commented May 22, 2019

ping review @dsplaisted @peterhuene

@wli3 wli3 merged commit adc9560 into dotnet:master May 22, 2019
@wli3 wli3 deleted the fix-GeneratePackageOnBuild-pack-as-tool branch May 22, 2019 20:40
wli3 pushed a commit that referenced this pull request Jun 5, 2019
* Fix GeneratePackageOnBuild with PackAsTool both on

NuGet/Home#7801

since nuget no longer sets NoBuild on GeneratePackageOnBuild we need to do similar logic to get the previous state.
wli3 pushed a commit that referenced this pull request Jun 5, 2019
* Fix GeneratePackageOnBuild with PackAsTool both on

NuGet/Home#7801

since nuget no longer sets NoBuild on GeneratePackageOnBuild we need to do similar logic to get the previous state.
dsplaisted pushed a commit to dsplaisted/sdk that referenced this pull request Feb 19, 2020
…0191021.7 (dotnet#3255)

- Microsoft.NETCore.App.Runtime.win-x64 - 3.1.0-preview2.19521.7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Project with PackAsTool and GeneratePackageOnBuild set to true fails to build

3 participants