Skip to content

chore: remove usage of jcenter() #680

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

Closed
wants to merge 1 commit into from

Conversation

r3nya
Copy link

@r3nya r3nya commented Sep 28, 2021

Summary

I got a warning message about shuting down jcenter.

WARNING:: Please remove usages of jcenter() Maven repository from your build scripts and migrate your build to other Maven repositories.
This repository is deprecated and it will be shut down in the future.
See http://developer.android.com/r/tools/jcenter-end-of-service for more information.

Using this information we can drop this repo from gradle file.

@tido64
Copy link
Member

tido64 commented Sep 28, 2021

We cannot do this yet. Please see the discussion in #610 (comment):

Unfortunately, we cannot remove JCenter without breaking people on older versions of React Native:

Could not determine the dependencies of task ':app:lintVitalRelease'.
> Could not resolve all artifacts for configuration ':app:debugCompileClasspath'.
   > Could not find com.facebook.flipper:flipper:0.54.0.
     Required by:
         project :app
   > Could not find com.facebook.flipper:flipper-fresco-plugin:0.54.0.
     Required by:
         project :app
   > Could not find com.facebook.flipper:flipper-network-plugin:0.54.0.
     Required by:
         project :app
   > Could not find com.facebook.yoga:proguard-annotations:1.14.1.
     Required by:
         project :app > com.facebook.react:react-native:0.63.4
   > Could not find com.facebook.fbjni:fbjni-java-only:0.0.3.
     Required by:
         project :app > com.facebook.react:react-native:0.63.4

Like I mentioned earlier, there is no urgency here since JCenter will be up in read-only mode indefinitely.

Read the update here: https://developer.android.com/studio/build/jcenter-migration

This is just a warning and will not affect your final output.

@tido64 tido64 closed this Sep 28, 2021
@r3nya
Copy link
Author

r3nya commented Sep 28, 2021

Hi @tido64!

Got it, thanks

Bibazavr added a commit to Bibazavr/async-storage that referenced this pull request Oct 22, 2021
@tido64 said:
> Unfortunately, we cannot remove JCenter without breaking people on older versions of React Native:

I disagree with it and I will proof soon

Refs: react-native-async-storage#610, react-native-async-storage#680, react-native-async-storage#681, react-native-async-storage#666, react-native-async-storage#611
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