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

fix: bottomsheet-1.3.1.aar incorrectly includes aapt/AndroidManifest.xml #23

Merged
merged 1 commit into from
Jun 19, 2019
Merged

fix: bottomsheet-1.3.1.aar incorrectly includes aapt/AndroidManifest.xml #23

merged 1 commit into from
Jun 19, 2019

Conversation

mikehardy
Copy link
Contributor

I removed it, re-packed it, put it in android/libs and referenced it from libs
The dependency is effectively dead / unsupported anyway, so should not be
a maintenance burden

Fixes #22

Demonstrated working at https://github.com/mikehardy/rn-androidx-demo / https://github.com/mikehardy/rn-androidx-demo/blob/master/make-demo.sh#L34

I removed it, re-packed it, put it in android/libs and referenced it from libs
The dependency is effectively dead / unsupported anyway, so should not be
a maintenance burden
@mikehardy
Copy link
Contributor Author

I should mention, if you don't trust me enough to take my binary aar file (and who am I anyway?? don't trust me!) the alteration to the aar was:

mkdir temp && cd temp
jar -xvf ../bottomsheet-1.3.1.aar && rm ../bottomsheet-1.3.1.aar
\rm -fr aapt # This is the fix, aapt/AndroidManifest.xml is the problem
jar -cvf ../bottomsheet-1.3.1.aar *

Then just throw that in android/libs and reference it like so in the gradle.

@mikehardy
Copy link
Contributor Author

@Doko-Demo-Doa I think this will fix the dependency for your library so you're library users are ready for AndroidX if they want

@Doko-Demo-Doa
Copy link
Contributor

Sure, let me check and do the merging.

@Doko-Demo-Doa Doko-Demo-Doa merged commit fb1519e into Clip-sub:master Jun 19, 2019
@Doko-Demo-Doa
Copy link
Contributor

Thank you. Your implementation works well. v2.0.1 pushed with crashing issue fixed too.

@JerakRus
Copy link

JerakRus commented Dec 2, 2020

It seems in version 2.2.0 this error occurs again, or something very similar.
Снимок экрана 2020-12-02 в 10 42 35

react-native:0.63.4
react-native-bottomsheet: 2.2.0

I just ran ./gradlew assembleRelease in my project.

UPD: I downgraded back to 2.0.1 but the problem persists. It seems to be a matter of updating react-native.
Then I decided to look into the library files, and find out what is outdated there. Here's what I saw: we use android.permission.SYSTEM_ALERT_WINDOW and here's some information on it. Complete obsolescence just happens on android api 29, which uses react-native 0.63. It is not entirely clear why the problem did not appear earlier, but it seems in any case the library has no future, because the alternative that Google offers us does not have the necessary functionality.

P.s. Perhaps someone who is better versed in native code than me will look at this and say that I'm wrong. I'll be glad if this is so, but so far I realized that things are bad here.

P.s.s. Apparently the problem is still different from the original one, and I should have opened new issues, but I beg your pardon, I don't have much time for this now.

@Doko-Demo-Doa
Copy link
Contributor

I see. I think it's better to fork another lib instead of continuing to use current one. Any suggestions?

@Doko-Demo-Doa
Copy link
Contributor

2.5.2 released and it should be what you are looking for.

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.

Migrating app to Androidx fails due to unability to transform bottomsheet-1.3.1 dependency
3 participants