stride
stride copied to clipboard
Fix AssemblyProcessor builds.
PR Details
Fixes problems with the AssemblyProcessor solution and how it is used throughout Stride.
Description
- Clean up the projects in the AssemblyProcessor solution and have them build for
net472andnet6.0. - Remove assembly binding redirection configuration because all related issues are solved by disregarding the
netstandard2.0TFM. - Update
Stride.Core.targetsin two places (two files) to select the right version of AssemblyProcessor to use depending onMSBuildRuntimeType. Stride.GameStudioselects the version of the AssemblyProcessor DLL to use with[MSBuild]::IsTargetFrameworkCompatible.- Update
Stride.GameStudioversion ofMono.Cecilto match the version used by the AssemblyProcessor DLL.
Related Issue
#1479
Motivation and Context
Fixes errors and warnings in the AssemblyProcessor solution. Simplifies making changes to the assembly processor which I might want to make to obtain diagnostics and verify that it does or doesn't do certain things (anymore).
Types of changes
- [x] ~Docs change / refactoring~ / dependency upgrade
- [x] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Breaking change (fix or feature that would cause existing functionality to change)
- [x] Taking the PR as-is will break
mastersince it deletes the DLL's indeps\AssemblyProcessor. A followon PR by someone able to update those files and potentially signing them etc is needed to make this change non-breaking. In the mean time, the instruction is to open and buildStride.AssemblyProcessor.slnbefore attempting to buildStride.sln.
- [x] Taking the PR as-is will break
Checklist
- [ ] My change requires a change to the documentation.
- [ ] I have added tests to cover my changes.
- [x] All new and existing tests passed. Yes, the tests in the assembly processor solution now 1) compile and 2) succeed.