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

Fix generate package on build pack as tool #3255

Merged
merged 2 commits into from
May 22, 2019

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 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