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

[build] Remove make framework-assemblies target #9269

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Conversation

pjcollins
Copy link
Member

I'm often running make leeroy to test changes while building on macOS,
and I noticed that an extra handful of Mono.Android.csproj builds were
running each time. The framework-assemblies target that is declared
in the generated rules.mk file includes some additional project builds
that don't seem to be necessary in a .NET only world.

The build has been updated to stop generating rules.mk entirely, and
the make targets in BuildEverything.mk have been simplified.

I'm often running `make leeroy` to test changes while building on macOS,
and I noticed that an extra handful of Mono.Android.csproj builds were
running each time. The `framework-assemblies` target that is declared
in the generated `rules.mk` file includes some additional project builds
that don't seem to be necessary in a .NET only world.

The build has been updated to stop generating `rules.mk` entirely, and
the make targets in `BuildEverything.mk` have been simplified.
@pjcollins pjcollins merged commit e4a75a8 into main Sep 3, 2024
58 checks passed
@pjcollins pjcollins deleted the dev/pjc/no-extra-fx branch September 3, 2024 14:12
@github-actions github-actions bot locked and limited conversation to collaborators Oct 4, 2024
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.

2 participants