Skip to content

Fix source-build oob package orchestration#84720

Merged
lewing merged 1 commit into
mainfrom
ViktorHofer-patch-1
Apr 13, 2023
Merged

Fix source-build oob package orchestration#84720
lewing merged 1 commit into
mainfrom
ViktorHofer-patch-1

Conversation

@ViktorHofer
Copy link
Copy Markdown
Member

Regressed with e3c3700. By replacing "net48" with $(NetFrameworkCurrent), the condition became true (as both BuildTargetFramework and NetFrameworkCurrent are empty in source build) and projects were filtered out.

Unblocks dotnet/sdk#31752

Regressed with e3c3700. By replacing "net48" with `$(NetFrameworkCurrent)`, the condition became true (as both BuildTargetFramework and NetFrameworkCurrent are empty in source build) and projects were filtered out.
@ghost
Copy link
Copy Markdown

ghost commented Apr 12, 2023

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Regressed with e3c3700. By replacing "net48" with $(NetFrameworkCurrent), the condition became true (as both BuildTargetFramework and NetFrameworkCurrent are empty in source build) and projects were filtered out.

Unblocks dotnet/sdk#31752

Author: ViktorHofer
Assignees: ViktorHofer
Labels:

area-Infrastructure-libraries

Milestone: -

Copy link
Copy Markdown
Member

@tmds tmds left a comment

Choose a reason for hiding this comment

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

Thanks!

@lewing lewing closed this Apr 13, 2023
@lewing lewing reopened this Apr 13, 2023
@lewing lewing merged commit fbcb1e7 into main Apr 13, 2023
@lewing lewing deleted the ViktorHofer-patch-1 branch April 13, 2023 00:35
@ghost ghost locked as resolved and limited conversation to collaborators May 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants