-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Enable clean + build of crosstargeting projects #1665
Enable clean + build of crosstargeting projects #1665
Conversation
This is a better long-term fix than NuGet/NuGet.Client#1167, but it affects all multi-targeting SDK-using projects, rather than just consumers of |
@nguerrera Thoughts on risk for this fix (which also does |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't seem very risky and it's the right fix. I'd rather go with this.
Retargeted to xplat based on the decision today to not take this for RTW. @dotnet-bot test this please |
Are we taking the nuget workaround for rtm ? |
My understanding was that we would do neither for RTW.
|
for rtm, we've decided we will live without either. happy to discuss tomorrow. |
Fixes dotnet#1065 by duplicating `DispatchToInnerBuilds` to the crosstargeting version of `Clean`. This enables Clean to run before Build (or the arbitrary user target set via the `InnerTargets` property), reducing the surprise of command line builds that use `/t:Clean;Build` rather than `/t:Rebuild`. This change also changes the cross-targeting definition of `Rebuild` to match the standard `Clean;Build` rather than invoking `Rebuild` on each inner build. That makes it easier to author outer-build extension logic, fixing NuGet/Home#4475.
a718103
to
36b9ced
Compare
Fixes #1065 by duplicating
DispatchToInnerBuilds
to the crosstargetingversion of
Clean
. This enables Clean to run before Build (or thearbitrary user target set via the
InnerTargets
property), reducing thesurprise of command line builds that use
/t:Clean;Build
rather than/t:Rebuild
.This change also changes the cross-targeting definition of
Rebuild
tomatch the standard
Clean;Build
rather than invokingRebuild
on eachinner build. That makes it easier to author outer-build extension logic,
fixing NuGet/Home#4475.