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

Pass missing compiler parameters #45257

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Pass missing compiler parameters #45257

merged 1 commit into from
Jan 13, 2025

Conversation

jaredpar
Copy link
Member

@jaredpar jaredpar commented Dec 2, 2024

Pass a few parameters to <Csc> that were missing and impact build reproducibility.

closes #45256

Pass a few parameters to `<Csc>` that were missing and impact build reproducibility.

closes #45256
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-ILLink untriaged Request triage from a team member labels Dec 2, 2024
@baronfel
Copy link
Member

baronfel commented Dec 2, 2024

Happy to take this immediately. For the future, is there a way between CoreCompile, Razor compile, and CoreGenerateSatelliteAssemblies that we can keep critical parameters in sync?

@Forgind
Copy link
Member

Forgind commented Dec 3, 2024

Should this be targeting a release* branch?

@baronfel
Copy link
Member

baronfel commented Dec 3, 2024

@Forgind we can always backport to 200 or something - it sounds like from #45256 (comment) there's not a huge correctness problem that might motivate going further back.

@Forgind
Copy link
Member

Forgind commented Dec 3, 2024

I wasn't thinking we needed to go back further than 200; just seemed like since that's active development, may as well 🙂

@baronfel baronfel merged commit 0eb8dac into main Jan 13, 2025
38 checks passed
@baronfel baronfel deleted the dev/jaredpar/satellite branch January 13, 2025 22:29
@baronfel
Copy link
Member

/backport to release/9.02xx

Copy link
Contributor

Started backporting to release/9.02xx: https://github.com/dotnet/sdk/actions/runs/12757300904

@baronfel
Copy link
Member

/backport to release/9.0.2xx

Copy link
Contributor

Started backporting to release/9.0.2xx: https://github.com/dotnet/sdk/actions/runs/12757304371

Copy link
Contributor

@baronfel an error occurred while backporting to "release/9.02xx", please check the run log for details!

Error: The specified backport target branch "release/9.02xx" wasn't found in the repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-ILLink untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Satellite assemblies do not pass PathMap to Csc task
3 participants