-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Long image filenames result in assembleRelease crash #16263
Comments
Same issue:
|
The names of assets files converting at the getAndroidResourceIdentifier() method.
I suggest to modify getAndroidResourceIdentifier() method by drastically change filenames with using md5
Return value should not begins from digit. For this at the beginning of the line the symbol 'a' (shortened from the 'assets') |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. If you think this issue should definitely remain open, please let us know why. Thank you for your contributions. |
This should really not be closed as this is still a problem and it will affect more people in the future. |
Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may also label this issue as "For Discussion" or "Good first issue" and I will leave it open. Thank you for your contributions. |
Hi this is the future and i confirm this is still an issue... |
Related Android bug: https://issuetracker.google.com/issues/36953084 It looks like the resulting drawable file name should be no longer than 100 characters. |
👋 hey all, thanks for the report and additional Android issue tracker details. I'm going to close this issue as it's out of our control, but let's track this over on the Known Issues list instead. Feel free to continue the conversation though if needed and update on the status of the Android issue tracker should anything change. Thank you |
Is this a bug report?
Yes
Have you read the Contributing Guidelines?
Yes
Environment
react-native -v
: 0.45.1node -v
: v8.2.1npm -v
: 5.3.0Target Platform: Android
Development Operating System: macOS
Build tools:
compileSdkVersion 23
buildToolsVersion '25.0.0'
Steps to Reproduce
/shared/components/Component/components/AnotherComponent/components/AnotherComponent/components/AnotherComponent/img/image.png
)assembleRelease
Expected Behavior
Successful assembly
Actual Behavior
Assembly fails when processing the file:
It's probably the long filename that cannot be handled by Java. When I undeepen the structure, the problem disappears. If a hash of the path would be used as a filename, this problem would not exist.
Reproducible Demo
https://github.com/joachimroeleveld/rn-issue-15509
Steps to reproduce:
npm install
./android/assembleRelease
The text was updated successfully, but these errors were encountered: