fix: avoid NPE when requirements.txt is not used

Author: ih0r-d

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

@@ -100,6 +100,9 @@ private void manageVenv() throws MojoExecutionException {
 		MavenDelegateLog log = new MavenDelegateLog(getLog());
 		Path lockFile = getLockFile();
 		Path reqFile = resolveReqFile();
+		if ((packages == null || packages.isEmpty()) && reqFile == null) {
+			return;
+		}
 		try {
 			VFSUtils.createVenv(venvDirectory, packages, lockFile, MISSING_LOCK_FILE_WARNING, createLauncher(),
 					getGraalPyVersion(project), log, reqFile);

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

@@ -552,24 +552,27 @@ public static void createVenv(Path venvDirectory, List<String> packages, Path lo
 
 	static boolean installPackagesFromReqFile(Path venvDirectory, Launcher launcher, String graalPyVersion,
 			BuildToolLog log, Path lockFilePath, Path reqFile) throws IOException {
-		if (reqFile != null && Files.exists(reqFile)) {
-			log.info("Using <requirements.txt> dependency mode.");
-			log.info("Installing Python dependencies from: " + reqFile);
+		if (reqFile == null) {
+			return false;
+		}
+		if (!Files.exists(reqFile)) {
+			return false;
+		}
+		log.info("Using <requirements.txt> dependency mode.");
+		log.info("Installing Python dependencies from: " + reqFile);
 
-			warnIfLockFileExists(lockFilePath, log);
+		warnIfLockFileExists(lockFilePath, log);
 
-			VenvContents vc = ensureVenv(venvDirectory, graalPyVersion, launcher, log);
+		VenvContents vc = ensureVenv(venvDirectory, graalPyVersion, launcher, log);
 
-			runPip(venvDirectory, "install", log, "--compile", "-r", reqFile.toString());
-			List<String> reqPackages = requirementsPackages(reqFile);
-			vc.write(reqPackages);
+		runPip(venvDirectory, "install", log, "--compile", "-r", reqFile.toString());
+		List<String> reqPackages = requirementsPackages(reqFile);
+		vc.write(reqPackages);
 
-			InstalledPackages installed = InstalledPackages.fromVenv(venvDirectory);
-			installed.freeze(log);
+		InstalledPackages installed = InstalledPackages.fromVenv(venvDirectory);
+		installed.freeze(log);
 
-			return true;
-		}
-		return false;
+		return true;
 	}
 
 	private static void warnIfLockFileExists(Path lockFilePath, BuildToolLog log) {
@@ -591,7 +594,7 @@ static void installPackages(Path venvDirectory, List<String> packages, Path lock
 
 		List<String> pluginPackages = trim(packages);
 
-		LockFile lockFile = null;
+		VFSUtils.LockFile lockFile = null;
 		if (lockFilePath != null && Files.exists(lockFilePath)) {
 			log.info("Lock-file detected: " + lockFilePath);
 			lockFile = LockFile.fromFile(lockFilePath, log);