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
Don't run _HandlePackageFileConflicts target if there are no references #3912
Don't run _HandlePackageFileConflicts target if there are no references #3912
Conversation
This should go for 3.1.2xx at this point right? |
@livarcocc Yes, but we don't have that branch yet :-) |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@@ -25,7 +25,8 @@ Copyright (c) .NET Foundation. All rights reserved. | |||
<Target Name="_HandlePackageFileConflicts" | |||
BeforeTargets="$(_HandlePackageFileConflictsBefore)" | |||
AfterTargets="$(_HandlePackageFileConflictsAfter)" | |||
DependsOnTargets="GetFrameworkPaths;GetReferenceAssemblyPaths"> | |||
DependsOnTargets="GetFrameworkPaths;GetReferenceAssemblyPaths" | |||
Condition="'@(Reference)' != '' Or '@(ReferencePath)' != ''"> |
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.
It feels wrong that this conditions on ReferencePath but doesn't use ReferencePath.
3f44ec8
to
81037da
Compare
@nguerrera @livarcocc I pushed an update based on Nick's feedback. One of you may want to merge this assuming the tests pass. |
Fixes #3585