-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Fix MilCodeGen and Csp build #10417
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
Fix MilCodeGen and Csp build #10417
Conversation
6f9910c
to
ed4cd70
Compare
src/Microsoft.DotNet.Wpf/src/WpfGfx/codegen/mcg/tools/GeneratedResources.txt
Show resolved
Hide resolved
src/Microsoft.DotNet.Wpf/src/WpfGfx/codegen/mcg/tools/InvokeCSP.cmd
Outdated
Show resolved
Hide resolved
src/Microsoft.DotNet.Wpf/src/WpfGfx/codegen/mcg/generators/ProtocolFingerprint.cs
Outdated
Show resolved
Hide resolved
@ThomasGoulet73 I have reviewed the PR and left the comments. I am still to look into GenerateFiles.cmd, Resources.xml, Resources.xsd and related files, but I will be done by tomorrow. |
…gen-and-csp-build # Conflicts: # global.json
Thanks @dipeshmsft, I believe I've applied all your change requests and answered your questions. I also pulled main to fix the conflicts. Let me know if there's anything else. |
Thanks @ThomasGoulet73 for the quick turnaround on the comments and feedback. I have reviewed the PR again, and I have only one concern that I have left in the comments. Apart from that this PR is good to go. |
…gen-and-csp-build # Conflicts: # global.json
Great, I pushed a commit and answered your comment. |
Thanks @ThomasGoulet73 for support on this PR. I will wait for subsequent PRs on MilCodeGen. |
Thanks @dipeshmsft. P.S.: I opened #10429 to track the rest of the work. |
Description
This PR aims to fix MilCodeGen which seems to be very outdated and it doesn't seem like it ever worked in this repo (It probably worked in the internal repo for Microsoft).
The goal of this PR is to fix MilCodeGen and Csp to be able to build and run MilCodeGen. I'll open other PRs to fix the templates to match the generated code and then other PRs which includes the modification to the generated code.
I cherry-picked the changes from #6135, reworked it to make it easier to use and updated the documentation.
Customer Impact
None, this is non-shipping code.
Regression
No.
Testing
Local build + I generated the code locally using #6135 rebased over this PR.
Risk
Low to none, this is non-shipping code and MilCodeGen already doesn't run so it can't get any worse.
Microsoft Reviewers: Open in CodeFlow