X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=src%2Forg%2Fargeo%2Fbuild%2FMake.java;h=cfedf9a06912e5a74fcc4fd1ccd30315c5f81940;hb=4d2955f554f19e8f77eae527b5831d533eb8d27f;hp=23477388f1861b46428805759e20d0b56291ebf9;hpb=ed86c203d9fcb9113431795edf3e7712b81e2c65;p=cc0%2Fargeo-build.git

diff --git a/src/org/argeo/build/Make.java b/src/org/argeo/build/Make.java
index 2347738..cfedf9a 100644
--- a/src/org/argeo/build/Make.java
+++ b/src/org/argeo/build/Make.java
@@ -20,6 +20,7 @@ import java.nio.file.Path;
 import java.nio.file.PathMatcher;
 import java.nio.file.Paths;
 import java.nio.file.SimpleFileVisitor;
+import java.nio.file.StandardCopyOption;
 import java.nio.file.attribute.BasicFileAttributes;
 import java.util.ArrayList;
 import java.util.Arrays;
@@ -302,7 +303,7 @@ public class Make {
 	}
 
 	/** Install the bundles. */
-	void install(Map<String, List<String>> options) throws IOException {
+	void install(Map<String, List<String>> options, boolean uninstall) throws IOException {
 		// check arguments
 		List<String> bundles = options.get("--bundles");
 		Objects.requireNonNull(bundles, "--bundles argument must be set");
@@ -320,6 +321,7 @@ public class Make {
 		if (targetDirs.size() != 1)
 			throw new IllegalArgumentException("One and only one --target must be specified");
 		Path targetA2 = Paths.get(targetDirs.get(0));
+		logger.log(INFO, (uninstall ? "Uninstalling from " : "Installing to ") + targetA2);
 
 		final String branch;
 		Path branchMk = sdkSrcBase.resolve(BRANCH_MK);
@@ -341,6 +343,7 @@ public class Make {
 		String minor = properties.getProperty("minor");
 		Objects.requireNonNull(minor, "'minor' must be set");
 
+		int count = 0;
 		for (String bundle : bundles) {
 			Path bundlePath = Paths.get(bundle);
 			Path bundleParent = bundlePath.getParent();
@@ -348,9 +351,36 @@ public class Make {
 					: a2Output.resolve(category);
 			Path jarP = a2JarDirectory.resolve(bundlePath.getFileName() + "." + major + "." + minor + ".jar");
 
-			Path targetJarP = targetA2.resolve(a2JarDirectory.relativize(jarP));
-			Files.createDirectories(targetJarP.getParent());
-			Files.copy(jarP, targetJarP);
+			Path targetJarP = targetA2.resolve(a2Output.relativize(jarP));
+			if (uninstall) { // uninstall
+				if (Files.exists(targetJarP)) {
+					Files.delete(targetJarP);
+					logger.log(DEBUG, "Removed " + targetJarP);
+					count++;
+				}
+				Path targetParent = targetJarP.getParent();
+				deleteEmptyParents(targetA2, targetParent);
+			} else { // install
+				Files.createDirectories(targetJarP.getParent());
+				boolean update = Files.exists(targetJarP);
+				Files.copy(jarP, targetJarP, StandardCopyOption.REPLACE_EXISTING);
+				logger.log(DEBUG, (update ? "Updated " : "Installed ") + targetJarP);
+				count++;
+			}
+		}
+		logger.log(INFO, uninstall ? count + " bundles removed" : count + " bundles installed or updated");
+	}
+
+	/** Delete empty parent directory up to the A2 target (included). */
+	void deleteEmptyParents(Path targetA2, Path targetParent) throws IOException {
+		if (!Files.isDirectory(targetParent))
+			throw new IllegalArgumentException(targetParent + " must be a directory");
+		boolean isA2target = Files.isSameFile(targetA2, targetParent);
+		if (!Files.list(targetParent).iterator().hasNext()) {
+			Files.delete(targetParent);
+			if (isA2target)
+				return;// stop after deleting A2 base
+			deleteEmptyParents(targetA2, targetParent.getParent());
 		}
 	}
 
@@ -478,6 +508,15 @@ public class Make {
 
 			// Add all resources from src/
 			Files.walkFileTree(srcP, new SimpleFileVisitor<Path>() {
+				@Override
+				public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) throws IOException {
+					// skip directories ending with .js
+					// TODO find something more robust?
+					if (dir.getFileName().toString().endsWith(".js"))
+						return FileVisitResult.SKIP_SUBTREE;
+					return super.preVisitDirectory(dir, attrs);
+				}
+
 				@Override
 				public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
 					if (file.getFileName().toString().endsWith(".java")
@@ -640,7 +679,8 @@ public class Make {
 			case "compile" -> argeoMake.compile(options);
 			case "bundle" -> argeoMake.bundle(options);
 			case "all" -> argeoMake.all(options);
-			case "install" -> argeoMake.install(options);
+			case "install" -> argeoMake.install(options, false);
+			case "uninstall" -> argeoMake.install(options, true);
 
 			default -> throw new IllegalArgumentException("Unkown action: " + action);
 			}