8000 Remove `<CreateMsymManifest/>` MSBuild task and `$(MonoSymbolArchive)` property by Copilot · Pull Request #10221 · dotnet/android · GitHub
[go: up one dir, main page]

Skip to content

Conversation

Copilot
Copy link
Contributor
@Copilot Copilot AI commented Jun 17, 2025

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/ 8000 Tasks/CreateMsymManifest.cs)
  • MonoSymbolicate MSBuild task (src/Xamarin.Android.Build.Tasks/Tasks/MonoSymbolicate.cs)

Updated MSBuild Targets (Xamarin.Android.Common.targets):

  • Removed UsingTask declarations for both tasks
  • Removed $(MonoSymbolArchive) property definitions and conditions
  • Simplified $(_AndroidSequencePointsMode) logic (removed MonoSymbolArchive dependencies)
  • Removed $(_MSYMDirectory) property definition
  • Removed MonoSymbolArchive from property cache
  • Removed complete mono-symbolicate task usage section in build targets

Updated Documentation:

  • Removed MonoSymbolArchive section from build properties documentation
  • Updated OneDotNet migration guide to clarify mono-symbolicate is not supported for .NET 6+
  • Removed CreateMsymManifest task prefix reference from message documentation

Preserved:

  • Default property in Microsoft.Android.Sdk.DefaultProperties.targets that sets MonoSymbolArchive=false for .NET compatibility
  • AOT-related .msym file creation in Tasks/Aot.cs (separate functionality unrelated to mono-symbolicate tool)

Verification

  • XML syntax validated for all modified targets files
  • Comprehensive search confirms all mono-symbolicate related code removed
  • No existing tests reference the removed components
  • Total impact: 152 lines removed, 6 lines added (formatting adjustments)

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.

Copilot AI and others added 3 commits June 17, 2025 22:11
Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
Co-authored-by: jonathanpeppers <840039+jonathanpeppers@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Remove <CreateMsymManifest/> MSBuild task and $(MonoSymbolArchive) property Remove <CreateMsymManifest/> MSBuild task and $(MonoSymbolArchive) property Jun 17, 2025
@Copilot Copilot AI requested a review from jonathanpeppers June 17, 2025 22:15
Copilot finished work on behalf of jonathanpeppers June 17, 2025 22:15
Copy link
Member
@jonathanpeppers jonathanpeppers left a 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>
Copy link
Contributor Author
Copilot AI commented Jun 18, 2025

Find other mentions of mono-symbolicate in the source code and remove them.

I think we could delete tools\scripts\mono-symbolicate,

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.

@Copilot Copilot AI requested a review from jonathanpeppers June 18, 2025 14:49
Copilot finished work on behalf of jonathanpeppers June 18, 2025 14:49
@jonathanpeppers jonathanpeppers marked this pull request as ready for review June 18, 2025 14:50
@jonathanpeppers jonathanpeppers enabled auto-merge (squash) June 18, 2025 14:53
@jonathanpeppers jonathanpeppers merged commit b8d0669 into main Jun 18, 2025
59 checks passed
@jonathanpeppers jonathanpeppers deleted the copilot/fix-10220 branch June 18, 2025 16:06
@github-actions github-actions bot locked and limited conversation to collaborators Jul 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove <CreateMsymManifest/> MSBuild task and $(MonoSymbolArchive) property

2 participants

0