From 384e1a0c7bc50d2aab5b59bcedcea5a3e98f1659 Mon Sep 17 00:00:00 2001 From: Kudo Chien Date: Fri, 28 Jan 2022 06:14:34 -0800 Subject: [PATCH] Add missing sources jar into published android artifacts (#32982) Summary: when migrated to `maven-publish` in https://github.com/facebook/react-native/issues/31611, the sources jar is not included by default from `maven-publish`. so react-native 0.66 / 0.67 doesn't include sources jar in npm published artifacts. it's not ideal for debug or tracing code. this pr added the sources jar into the published artifact. ## Changelog [Android] [Fixed] - Add missing sources jar into published android artifacts Pull Request resolved: https://github.com/facebook/react-native/pull/32982 Test Plan: make sure sources jar is included in artifact. ``` $ ./gradlew :ReactAndroid:installArchives $ find android -name '*sources.jar*' ``` Reviewed By: ShikaSD Differential Revision: D33842979 Pulled By: cortinico fbshipit-source-id: f99ad46ce0cca0cfc2ab1d5c5a4fcb40a02683e7 --- ReactAndroid/build.gradle | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/ReactAndroid/build.gradle b/ReactAndroid/build.gradle index 142cbb70c8f0b8..84821820cd2ff6 100644 --- a/ReactAndroid/build.gradle +++ b/ReactAndroid/build.gradle @@ -272,6 +272,12 @@ task installArchives { dependsOn("publishReleasePublicationToNpmRepository") } +// Creating sources with comments +task androidSourcesJar(type: Jar) { + classifier = 'sources' + from android.sourceSets.main.java.srcDirs +} + android { compileSdkVersion 31 @@ -414,6 +420,9 @@ afterEvaluate { // Applies the component for the release build variant. from components.release + // Add additional sourcesJar to artifacts + artifact(androidSourcesJar) + // You can then customize attributes of the publication as shown below. artifactId = POM_ARTIFACT_ID groupId = GROUP