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

Enable format-job for windows #45015

Closed
wants to merge 1 commit into from

Conversation

kunalspathak
Copy link
Member

Revert changes in #40035 and #40517 and re-enable format job for Windows.

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@kunalspathak
Copy link
Member Author

The build job that produces configure_commands.json file fails for windows. I also see weird failures around:

The application to execute does not exist: 'D:/a/1/s/artifacts/bin/coreclr/windows.x64.Debug/DacTableGen/DacTableGen.dll'

@sandreenko or @BruceForstall - any idea? Also the build-runtime.cmd command takes around 20 minutes to complete where as on linux it completes in few minutes.

Linux

src/coreclr/build-runtime.sh x64 Checked configureonly -cmakeargs -DCMAKE_EXPORT_COMPILE_COMMANDS=1

Windows:

src\coreclr\build-runtime.cnd x64 Checked usenmakemakefiles

Can we invoke this command consistently?

@BruceForstall
Copy link
Member

These build scripts have changed dramatically recently. It may be that the formatting job uses script functionality that was only used for the formatting job, which is not a good way to keep it working over time. Maybe there's a way to invoke the build scripts in a more standard way (or at least have only minimal build script requirements for the formatting job).

Otherwise, I have no real insight into how that json file gets created or how the formatting job uses it.

@kunalspathak
Copy link
Member Author

These build scripts have changed dramatically recently. It may be that the formatting job uses script functionality that was only used for the formatting job, which is not a good way to keep it working over time. Maybe there's a way to invoke the build scripts in a more standard way (or at least have only minimal build script requirements for the formatting job).

Otherwise, I have no real insight into how that json file gets created or how the formatting job uses it.

In that case, I don't think I have time to figure out the right command for this. Do you know anyone who can help or else I can just close this PR and re-open it when I get time to figure that out.

@sandreenko
Copy link
Contributor

In that case, I don't think I have time to figure out the right command for this. Do you know anyone who can help or else I can just close this PR and re-open it when I get time to figure that out.

I think this work can be postponed, we have been living with a Linux formatting job only and can continue it for a while.

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.

4 participants