Skip to content

fix: filter invalid chars for gradle wrapper#1813

Merged
skylot merged 1 commit intoskylot:masterfrom
nitram84:gradle_wrapper_invalid_chars
Apr 8, 2023
Merged

fix: filter invalid chars for gradle wrapper#1813
skylot merged 1 commit intoskylot:masterfrom
nitram84:gradle_wrapper_invalid_chars

Conversation

@nitram84
Copy link
Copy Markdown
Contributor

@nitram84 nitram84 commented Apr 7, 2023

Some special chars in the app name can cause conflicts when you try to add a gradle wrapper. In this pull request I filter all invalid chars before generating settings.gradle

Here is an example app containing invalid chars: https://xn--who-becomes-rich--84ns4b8gwbu6jza3u.apk.gold/

Steps to reproduce:

  • save as gradle project
  • run 'gradle wrapper'
FAILURE: Build failed with an exception.

* Where:
Settings file 'settings.gradle' line: 2

* What went wrong:
A problem occurred evaluating settings.
> The project name 'Who Becomes Rich?' must not contain any of the following characters: [/, \, :, <, >, ", ?, *, |]. Set the 'rootProject.name' or adjust the 'include' statement (see https://docs.gradle.org/7.6/dsl/org.gradle.api.initialization.Settings.html#org.gradle.api.initialization.Settings:include(java.lang.String[]) for more details).

This issue can also easily be reproduced with new Android Studio project. Android Studio does not allow any special chars in the application name when the app is generated, but AndroidManifest.xml can be edited later like in my unit test.

@skylot skylot merged commit 0421ad8 into skylot:master Apr 8, 2023
@skylot
Copy link
Copy Markdown
Owner

skylot commented Apr 8, 2023

@nitram84 thanks! Nice fix.

@nitram84 nitram84 deleted the gradle_wrapper_invalid_chars branch April 18, 2023 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants