-
Notifications
You must be signed in to change notification settings - Fork 551
Remove <CreateMsymManifest/>
MSBuild task and $(MonoSymbolArchive)
property
#10221
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
Conversation
Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
<CreateMsymManifest/>
MSBuild task and $(MonoSymbolArchive)
property<CreateMsymManifest/>
MSBuild task and $(MonoSymbolArchive)
property
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Find other mentions of mono-symbolicate
in the source code and remove them.
I think we could delete tools\scripts\mono-symbolicate
,
… lines Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
Removed all additional mono-symbolicate references including the tools/scripts/mono-symbolicate script, mono-symbolicate from _MonoScript items, and MonoSymbolArchive property from DefaultProperties.targets. Fixed in 9810216. |
This PR removes the legacy mono-symbolicate support components that were never implemented for .NET 6+ as requested in the issue.
Changes Made
Removed Task Classes:
CreateMsymManifest
MSBuild task (src/Xamarin.Android.Build.Tasks/Tasks/CreateMsymManifest.cs
)MonoSymbolicate
MSBuild task (src/Xamarin.Android.Build.Tasks/Tasks/MonoSymbolicate.cs
)Updated MSBuild Targets (
Xamarin.Android.Common.targets
):$(MonoSymbolArchive)
property definitions and conditions$(_AndroidSequencePointsMode)
logic (removed MonoSymbolArchive dependencies)$(_MSYMDirectory)
property definitionUpdated Documentation:
Preserved:
Microsoft.Android.Sdk.DefaultProperties.targets
that setsMonoSymbolArchive=false
for .NET compatibilityTasks/Aot.cs
(separate functionality unrelated to mono-symbolicate tool)Verification
The changes are surgical and minimal, removing only the specific mono-symbolicate functionality while preserving all other build system functionality.
Fixes #10220.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.