search

m4gr3d / Godot-Android-Plugin-Template

This repository serves as a quickstart template for building a Godot Android plugin for Godot 4.2+.
MIT License
43 stars 11 forks source link

Build Failure in gltf_viewer: Resource name must contain only lowercase files #1

Closed luc4smoreira closed 12 months ago

luc4smoreira commented 12 months ago

Environment IDE: Android Studio Giraffe | 2022.3.1 Computer: Mac Mini M1

Description

I encountered a build failure when trying to run the gltf_viewer app in the Godot-Android-Samples project. The issue seems to be related to a resource naming convention.

Steps do Reproduce:

  1. Clone the repository
  2. Run the command: ./gradlew :apps:gltf_viewer:installDebug
  3. Observe the build failure:

Godot-Android-Samples/apps/gltf_viewer/src/main/res/drawable-nodpi/iceCreamCup.png: Resource and asset merger: 'C' is not a valid file-based resource name character: File-based resource names must contain only lowercase a-z, 0-9, or underscore

FAILURE: Build failed with an exception.

luc4smoreira commented 12 months ago

To fix this issue, I have renamed the files to lowercase in the folder:

Godot-Android-Samples/apps/gltf_viewer/src/main/res/drawable-nodpi

This aligns with the Android naming conventions that require resource names to be all in lowercase. I have made a commit with these changes in my fork, which can be found here:

Renamed images to lowercase for Android resource naming compliance.

Unfortunately, I am unable to create a pull request for this change as GitHub is not recognizing the difference in file names due to case sensitivity issues.

Please check if this change resolves the issue. I am available for any further adjustments or discussions if needed.

m4gr3d commented 12 months ago

@luc4smoreira This issue was opened in the wrong repo, it should be opened in the Godot-Android-Samples repo instead.