fix: correct requirements.txt handling and lock-file behaviour

Author: ih0r-d

integration-tests/prepare_venv_requirements_pom.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!--
-Copyright (c) 2023, 2023, Oracle and/or its affiliates. All rights reserved.
+Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
 DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
 
 The Universal Permissive License (UPL), Version 1.0
@@ -44,11 +44,10 @@ SOFTWARE.
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
   <modelVersion>4.0.0</modelVersion>
 
-  <groupId>org.apache.maven.plugin.my.unit</groupId>
-  <artifactId>project-prepare-venv</artifactId>
+  <groupId>archetype.it</groupId>
+  <artifactId>requirements_packages</artifactId>
   <version>1.0-SNAPSHOT</version>
   <packaging>jar</packaging>
-  <name>Test MyMojo</name>
 
   <dependencies>
     <dependency>
@@ -68,6 +67,11 @@ SOFTWARE.
       <artifactId>python-launcher</artifactId>
       <version>${env.GRAALPY_VERSION}</version>
     </dependency>
+    <dependency>
+      <groupId>org.graalvm.python</groupId>
+      <artifactId>python-embedding</artifactId>
+      <version>${env.GRAALPY_VERSION}</version>
+    </dependency>
   </dependencies>
 
   <build>
@@ -84,6 +88,9 @@ SOFTWARE.
           </execution>
         </executions>
           <configuration>
+              <resourceDirectory>
+                  GRAALPY-VFS/${project.groupId}/${project.artifactId}
+              </resourceDirectory>
               <requirementsFile>requirements.txt</requirementsFile>
           </configuration>
       </plugin>

integration-tests/test_maven_plugin.py

@@ -848,7 +848,7 @@ def test_requirements_txt_packages(self):
         requirements_txt = os.path.join(target_dir, "requirements.txt")
         if not os.path.exists(requirements_txt):
           with open(requirements_txt, "w", encoding="utf-8") as f:
-            f.write("pyfiglet==1.0.2\n")
+            f.write("termcolor==2.4.0\n")
 
         mvnw_cmd = util.get_mvn_wrapper(target_dir, self.env)
 
@@ -858,7 +858,17 @@ def test_requirements_txt_packages(self):
         assert return_code == 0
 
         lock_file = os.path.join(target_dir, "graalpy.lock")
-        assert not os.path.exists(lock_file)
+        assert not os.path.exists(lock_file), "lock-file must NOT exist for requirements.txt mode"
+        cmd = mvnw_cmd + ["package", "-DmainClass=it.pkg.GraalPy"]
+        out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir)
+        util.check_ouput("BUILD SUCCESS", out)
+        assert return_code == 0
+
+        cmd = mvnw_cmd + ["exec:java", "-Dexec.mainClass=it.pkg.GraalPy"]
+        out, return_code = util.run_cmd(cmd, self.env, cwd=target_dir)
+        util.check_ouput("hello java", out)
+        util.check_ouput("BUILD SUCCESS", out)
+        assert return_code == 0
 
 
 if __name__ == "__main__":

org.graalvm.python.embedding.tools/src/main/java/org/graalvm/python/embedding/tools/vfs/VFSUtils.java

@@ -557,8 +557,8 @@ static boolean installPackagesFromReqFile(Path venvDirectory, Launcher launcher,
         if (reqFile != null) {
             log.info("Using <requirements.txt> dependency mode.");
             log.info("Installing Python dependencies from: " + reqFile);
-            log.warning("Lock file is ignored in <requirements.txt> mode.");
-            log.warning("The 'lock-packages' goal should not be used together with <requirementsFile>.");
+
+            warnIfLockFileExists(venvDirectory,log);
 
             VenvContents vc = ensureVenv(venvDirectory, graalPyVersion, launcher, log);
 
@@ -574,6 +574,14 @@ static boolean installPackagesFromReqFile(Path venvDirectory, Launcher launcher,
         return false;
     }
 
+    private static void warnIfLockFileExists(Path venvDirectory, BuildToolLog log) {
+        Path lockFile = venvDirectory.resolve("graalpy.lock");
+        if (Files.exists(lockFile)) {
+            log.warning("Lock file is ignored in <requirements.txt> mode.");
+        }
+    }
+
+
     static void installPackages(Path venvDirectory, List<String> packages, Path lockFilePath,
             String missingLockFileWarning, Launcher launcher, String graalPyVersion,
             BuildToolLog log) throws IOException, PackagesChangedException {