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

[Android][build] Fix android package name containing - #93077

Merged
merged 2 commits into from
Oct 6, 2023

Conversation

mdh1418
Copy link
Member

@mdh1418 mdh1418 commented Oct 5, 2023

The android app builder formats the final package name based on the ProjectName. Some projects contain hyphens -, which is not allowed in the android package name.

This PR looks to substitute underscores in place of hyphens in the produced android package name.

@ghost
Copy link

ghost commented Oct 5, 2023

Tagging subscribers to this area: @directhex
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: mdh1418
Assignees: -
Labels:

area-Infrastructure-mono

Milestone: -

@mdh1418
Copy link
Member Author

mdh1418 commented Oct 5, 2023

/azp run runtime-android

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mdh1418
Copy link
Member Author

mdh1418 commented Oct 5, 2023

/azp run runtime-androidemulator

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mdh1418
Copy link
Member Author

mdh1418 commented Oct 5, 2023

/azp run runtime-android,runtime-androidemulator

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@mdh1418
Copy link
Member Author

mdh1418 commented Oct 6, 2023

Failures on runtime-androidemulator are known: #92723

@mdh1418 mdh1418 merged commit f37e653 into dotnet:main Oct 6, 2023
172 of 175 checks passed
@mdh1418 mdh1418 deleted the fix_android_package_name branch October 6, 2023 14:12
@ghost ghost locked as resolved and limited conversation to collaborators Nov 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants