Remove Android environment variable file handling for Aspire #32121
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Removed Android-specific environment variable handling code.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR removes Android-specific environment variable file handling that was previously added for Aspire integration. Since the Aspire+MAUI integration approach has changed and no longer requires reading environment variables from a file on Android devices, this code is being removed as unnecessary. The feature toggle for environment variable configuration remains intact for potential future use.
Key Changes:
- Removed Android-specific code that read environment variables from
/data/local/tmp/ide-launchenv.txt - Eliminated file-based environment variable merging logic for Android platform
Member
Author
|
/backport to release/10.0.1xx |
Contributor
|
Started backporting to release/10.0.1xx: https://github.com/dotnet/maui/actions/runs/18696960751 |
This was referenced Oct 21, 2025
rmarinho
approved these changes
Oct 22, 2025
jfversluis
added
the
p/0
This was referenced Nov 11, 2025
Closed
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of Change
Removes the Android specific environment variable I added for Aspire in #31494. Since this implementation of the Aspire+MAUI integration is broken atm anyway, we can just remove this code since its also not needed for the native Aspire integration (see dotnet/aspire#11942).
This code wasn't great to begin with, let's remove it. I did leave the feature toggle intact in case we might want to use that in other scenarios.
Issues Fixed
Closes #32119
Closes #32120