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.

Sign up for GitHub

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

Jump to bottom

[Windows build perf] CreatePkgInfo is always executed #19608

Open
Tracked by #19611
emaf opened this issue Dec 8, 2023 · 0 comments
Open
Tracked by #19611

[Windows build perf] CreatePkgInfo is always executed #19608

emaf opened this issue Dec 8, 2023 · 0 comments
Labels
delighter-sc msbuild Issues affecting our msbuild tasks/targets performance If an issue or pull request is related to performance windows-only The issue only occur on Windows
Milestone
.NET 10

Comments

@emaf
Copy link
Contributor

emaf commented Dec 8, 2023

Steps to Reproduce

The CreatePkgInfo is always executed from Windows since there is no check to avoid running it if the file already exist. We should probably create an output file on Windows to add an input/output check.

image

Expected Behavior

Actual Behavior

Environment

Version information

Build Logs

Example Project (If Possible)
@emaf emaf added performance If an issue or pull request is related to performance msbuild Issues affecting our msbuild tasks/targets windows-only The issue only occur on Windows labels Dec 8, 2023
@emaf emaf mentioned this issue Dec 8, 2023
Open
15 tasks
@rolfbjarne rolfbjarne added this to the .NET 9 milestone Dec 11, 2023
@rolfbjarne rolfbjarne modified the milestones: .NET 9, .NET 10 Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
delighter-sc msbuild Issues affecting our msbuild tasks/targets performance If an issue or pull request is related to performance windows-only The issue only occur on Windows
Projects
None yet
Milestone
.NET 10
Development

No branches or pull requests
3 participants
@rolfbjarne @emaf @samhouts