Deprecate python-community artifact

Author: steve-s

graalpy-maven-plugin/src/main/java/org/graalvm/python/maven/plugin/AbstractGraalPyMojo.java

@@ -293,10 +293,15 @@ protected static String getGraalPyVersion(MavenProject project) throws IOExcepti
 
 	private static Artifact getGraalPyArtifact(MavenProject project) throws IOException {
 		var projectArtifacts = resolveProjectDependencies(project);
-		Artifact graalPyArtifact = projectArtifacts.stream().filter(a -> isPythonArtifact(a)).findFirst().orElse(null);
-		return Optional.ofNullable(graalPyArtifact).orElseThrow(() -> new IOException(
-				"Missing GraalPy dependency. Please add to your pom either %s:%s or %s:%s".formatted(POLYGLOT_GROUP_ID,
-						PYTHON_COMMUNITY_ARTIFACT_ID, POLYGLOT_GROUP_ID, PYTHON_ARTIFACT_ID)));
+		Artifact graalPyArtifact = projectArtifacts.stream()
+				.filter(a -> (POLYGLOT_GROUP_ID.equals(a.getGroupId()) || GRAALPY_GROUP_ID.equals(a.getGroupId())))
+				.filter(a -> PYTHON_ARTIFACT_ID.equals(a.getArtifactId())).findFirst()
+				.orElse(projectArtifacts.stream().filter(
+						a -> (POLYGLOT_GROUP_ID.equals(a.getGroupId()) || GRAALPY_GROUP_ID.equals(a.getGroupId())))
+						.filter(a -> PYTHON_COMMUNITY_ARTIFACT_ID.equals(a.getArtifactId())).findFirst().orElse(null));
+		return Optional.ofNullable(graalPyArtifact)
+				.orElseThrow(() -> new IOException("Missing GraalPy dependency. Please add to your pom %s:%s"
+						.formatted(POLYGLOT_GROUP_ID, PYTHON_ARTIFACT_ID)));
 	}
 
 	private static boolean isPythonArtifact(Artifact a) {
@@ -333,6 +338,11 @@ private Set<String> calculateLauncherClasspath(MavenProject project) throws IOEx
 			// 2.) graalpy dependencies
 			Artifact graalPyArtifact = getGraalPyArtifact(project);
 			assert graalPyArtifact != null;
+			if (PYTHON_COMMUNITY_ARTIFACT_ID.equals(graalPyArtifact.getArtifactId())) {
+				getLog().warn("Deprecated artifact detected on classpath: " + graalPyArtifact.getGroupId() + ":"
+						+ graalPyArtifact.getArtifactId() + ". Please use '" + POLYGLOT_GROUP_ID + ":"
+						+ PYTHON_ARTIFACT_ID + "' instead.");
+			}
 			launcherClassPath.addAll(resolveDependencies(graalPyArtifact));
 		}
 		return launcherClassPath;

integration-tests/test_gradle_plugin.py

@@ -49,6 +49,7 @@
 MISSING_FILE_WARNING = "The list of installed Python packages does not match the packages specified in the graalpy-maven-plugin configuration"
 PACKAGES_CHANGED_ERROR = "packages and their version constraints in graalpy-gradle-plugin configuration are different then previously used to generate the lock file"
 VENV_UPTODATE = "Virtual environment is up to date with lock file, skipping install"
+DEPRECATION_MSG = "python-community' is deprecated"
 
 def append(file, txt):
     with open(file, "a") as f:
@@ -156,6 +157,7 @@ def check_gradle_generated_app(self):
             util.check_ouput("BUILD SUCCESS", out, logger=log)
             util.check_ouput("Virtual filesystem is deployed to default resources directory", out, logger=log)
             util.check_ouput("This can cause conflicts if used with other Java libraries that also deploy GraalPy virtual filesystem", out, logger=log)
+            util.check_ouput(DEPRECATION_MSG, out, contains=False, logger=log)
             self.check_filelist(target_dir, log)
 
             gradlew_cmd = util.get_gradle_wrapper(target_dir, self.env)
@@ -224,6 +226,7 @@ def check_lock_packages(self):
             out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir, logger=log)
             util.check_ouput("pip install", out, logger=log)
             util.check_ouput("BUILD SUCCESS", out, logger=log)
+            util.check_ouput(DEPRECATION_MSG, out, contains=False, logger=log)
             util.check_ouput(MISSING_FILE_WARNING, out, contains=True, logger=log)
             assert not os.path.exists(os.path.join(target_dir, "test-graalpy.lock")), log
 
@@ -641,6 +644,23 @@ def test_gradle_python_resources_dir_and_external_dir_error(self):
     def test_proxy_settings(self):
         self.check_proxy_settings()
 
+    def test_gradle_community_deprecation(self):
+        with TemporaryTestDirectory() as tmpdir:
+            target_dir = os.path.join(str(tmpdir), "community_deprecation_gradle" + self.target_dir_name_sufix())
+            self.generate_app(target_dir)
+            build_file = os.path.join(target_dir, self.build_file_name)
+            append(build_file, textwrap.dedent("""
+                graalPy {
+                    community = true
+                    packages = ["termcolor"]
+                }
+            """))
+            gradlew_cmd = util.get_gradle_wrapper(target_dir, self.env)
+            # Use a task that triggers afterEvaluate and dependency resolution
+            cmd = gradlew_cmd + ["graalPyInstallPackages"]
+            out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir)
+            util.check_ouput(DEPRECATION_MSG, out, contains=True)
+
     def target_dir_name_sufix(self):
         return "_groovy"
 

integration-tests/test_maven_plugin.py

@@ -51,6 +51,7 @@
 MISSING_FILE_WARNING = "The list of installed Python packages does not match the packages specified in the graalpy-maven-plugin configuration."
 PACKAGES_CHANGED_ERROR = "but packages and their version constraints in graalpy-maven-plugin configuration are different then previously used to generate the lock file"
 VENV_UPTODATE = "Virtual environment is up to date with lock file, skipping install"
+DEPRECATION_MSG = "Deprecated artifact detected on classpath"
 
 class MavenPluginTest(util.BuildToolTestBase):
     @classmethod
@@ -204,6 +205,7 @@ def check_generated_app(self, use_default_vfs_path):
             util.check_ouput("BUILD SUCCESS", out, logger=log)
             util.check_ouput("Virtual filesystem is deployed to default resources directory", out, contains=use_default_vfs_path, logger=log)
             util.check_ouput("This can cause conflicts if used with other Java libraries that also deploy GraalPy virtual filesystem.", out, contains=use_default_vfs_path, logger=log)
+            util.check_ouput(DEPRECATION_MSG, out, contains=False, logger=log)
 
             # check fileslist.txt
             fl_path = os.path.join(target_dir, "target", "classes", vfs_prefix, "fileslist.txt")
@@ -310,6 +312,7 @@ def test_lock_file(self):
             out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir)
             util.check_ouput("pip install", out)
             util.check_ouput("BUILD SUCCESS", out)
+            util.check_ouput(DEPRECATION_MSG, out, contains=False)
             util.check_ouput(MISSING_FILE_WARNING, out, contains=True)
             assert not os.path.exists(os.path.join(target_dir, "test-graalpy.lock"))
 
@@ -411,6 +414,7 @@ def test_generated_app_external_resources(self):
             cmd = mvnw_cmd + ["package"] + native_image_arg + ["-DmainClass=it.pkg.GraalPy"]
             out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir)
             util.check_ouput("BUILD SUCCESS", out)
+            util.check_ouput(DEPRECATION_MSG, out, contains=False)
 
             # execute and check JVM mode
             cmd = mvnw_cmd + ["exec:java", "-Dexec.mainClass=it.pkg.GraalPy"]
@@ -448,7 +452,7 @@ def test_fail_without_graalpy_dep(self):
 
             cmd = mvnw_cmd + ["process-resources"]
             out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir)
-            util.check_ouput("Missing GraalPy dependency. Please add to your pom either org.graalvm.polyglot:python-community or org.graalvm.polyglot:python", out)
+            util.check_ouput("Missing GraalPy dependency. Please add to your pom org.graalvm.polyglot:python", out)
 
 
     def test_check_home_warning(self):
@@ -835,6 +839,26 @@ def test_multiple_namespaced_vfs(self):
             assert return_code == 0, log
 
 
+    def test_community_dep_deprecation_message(self):
+        with util.TemporaryTestDirectory() as tmpdir:
+            target_name = "community_dep_deprecation_test"
+            target_dir = os.path.join(str(tmpdir), target_name)
+            self.generate_app(tmpdir, target_dir, target_name)
+
+            mvnw_cmd = util.get_mvn_wrapper(target_dir, self.env)
+            pom_path = os.path.join(target_dir, "pom.xml")
+
+            # Force community edition on classpath by replacing python with python-community in the generated POM
+            util.replace_in_file(pom_path, "<artifactId>python</artifactId>", "<artifactId>python-community</artifactId>")
+            util.replace_in_file(pom_path, "org.graalvm.python:python", "org.graalvm.python:python-community")
+            util.replace_in_file(pom_path, "org.graalvm.polyglot:python", "org.graalvm.polyglot:python-community")
+
+            # Run a goal that triggers the plugin resolution
+            cmd = mvnw_cmd + ["process-resources"]
+            out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir)
+            # Expect deprecation message for python-community
+            util.check_ouput(DEPRECATION_MSG, out, contains=True)
+
 if __name__ == "__main__":
     run_path = os.path.join(os.path.abspath(__file__), 'run.py')
     print(f"Run this file using the run.py driver ({run_path})")

org.graalvm.python.gradle.plugin/src/main/java/org/graalvm/python/GraalPyGradlePlugin.java

@@ -133,6 +133,11 @@ public void apply(Project project) {
 						"WARNING: Property 'polyglotVersion' is experimental and should be used only for testing pre-release versions.");
 			}
 
+			if (extension.getCommunity().convention(false).get()) {
+				proj.getLogger().warn(
+						"WARNING: 'python-community' is deprecated. The plugin defaults to 'python'. Support for 'python-community' may be removed in a future release.");
+			}
+
 			if (extension.getPythonResourcesDirectory().isPresent() && extension.getExternalDirectory().isPresent()) {
 				throw new GradleException(
 						"Cannot set both 'externalDirectory' and 'resourceDirectory' at the same time. "
@@ -324,18 +329,23 @@ private static void makeSureBothEditionsAreNotOnClasspathSimultaneously(Configur
 					boolean hasOracleEdition = false;
 					for (Dependency dep : deps) {
 						if (dep instanceof ExternalModuleDependency emd) {
-							if (GRAALPY_GROUP_ID.equals(emd.getModule().getGroup())) {
-								hasCommunityEdition |= PYTHON_COMMUNITY_ARTIFACT_ID.equals(emd.getModule().getName());
-								hasOracleEdition |= PYTHON_ARTIFACT_ID.equals(emd.getModule().getName());
+							String group = emd.getModule().getGroup();
+							String name = emd.getModule().getName();
+							if (GRAALPY_GROUP_ID.equals(group) || POLYGLOT_GROUP_ID.equals(group)) {
+								hasCommunityEdition |= PYTHON_COMMUNITY_ARTIFACT_ID.equals(name);
+								hasOracleEdition |= PYTHON_ARTIFACT_ID.equals(name);
 							}
 						}
 					}
 					if (hasCommunityEdition && hasOracleEdition) {
 						throw new GradleException(
-								"You have both 'org.graalvm.python:python' and 'org.graalvm.python:python-community' on the classpath. "
-										+ "This is likely due to an explicit dependency added, or duplicate dependencies. You may configure "
-										+ "the GraalPy plugin to inject the 'python-community' artifact by using the graalPy { community = true } "
+								"You have both 'org.graalvm.python:python' (or 'org.graalvm.polyglot:python') and 'org.graalvm.python:python-community' (or 'org.graalvm.polyglot:python-community') on the classpath. "
+										+ "The 'python-community' artifact is deprecated. Ensure only one edition is present. You may configure "
+										+ "the GraalPy plugin to inject the deprecated 'python-community' artifact by using the graalPy { community = true } "
 										+ "configuration block instead.");
+					} else if (hasCommunityEdition && !hasOracleEdition) {
+						org.gradle.api.logging.Logging.getLogger(GraalPyGradlePlugin.class).warn(
+								"WARNING: 'python-community' is deprecated. Please depend on 'org.graalvm.python:python' or 'org.graalvm.polyglot:python' instead.");
 					}
 				});
 	}