# HG changeset patch # User wenzelm # Date 1625665233 -7200 # Node ID 4a708e15090896b394fce8c59f266aeb1cbc50c1 # Parent 17c09d1b358830854ef835acda39cf203bb19b1f more robust treatment of empty string; diff -r 17c09d1b3588 -r 4a708e150908 src/Tools/Setup/isabelle/setup/Build_Scala.java --- a/src/Tools/Setup/isabelle/setup/Build_Scala.java Wed Jul 07 14:32:43 2021 +0200 +++ b/src/Tools/Setup/isabelle/setup/Build_Scala.java Wed Jul 07 15:40:33 2021 +0200 @@ -60,9 +60,17 @@ public String main() { return props.getProperty("main", ""); } - public String[] sources() { return props.getProperty("sources", "").split("\\s+"); } - public String[] resources() { return props.getProperty("resources", "").split("\\s+"); } - public String[] services() { return props.getProperty("services", "").split("\\s+"); } + private List get_list(String name) + { + List list = new LinkedList(); + for (String s : props.getProperty(name, "").split("\\s+")) { + if (!s.isEmpty()) { list.add(s); } + } + return List.copyOf(list); + } + public List sources() { return get_list("sources"); } + public List resources() { return get_list("resources"); } + public List services() { return get_list("services"); } public Path path(String file) { return component_dir.resolve(file); } public boolean exists(String file) { return Files.exists(path(file)); } @@ -103,7 +111,9 @@ args.add(target_dir.toString()); args.add("-bootclasspath"); args.add(Environment.join_paths(deps)); - for (String s : options.split("\\s+")) { args.add(s); } + for (String s : options.split("\\s+")) { + if (!s.isEmpty()) { args.add(s); } + } args.add("--"); for (Path p : sources) { args.add(p.toString()); } @@ -156,10 +166,10 @@ Path jar_path = context.path(jar_name); String shasum_name = context.shasum_name(); - String[] sources = context.sources(); - String[] resources = context.resources(); + List sources = context.sources(); + List resources = context.resources(); - if (sources.length == 0) { + if (sources.isEmpty()) { Files.deleteIfExists(jar_path); Files.deleteIfExists(context.path(shasum_name)); } @@ -188,7 +198,9 @@ List compiler_deps = new LinkedList(); for (String s : isabelle_class_path.split(":", -1)) { - compiler_deps.add(Path.of(Environment.platform_path(s))); + if (!s.isEmpty()) { + compiler_deps.add(Path.of(Environment.platform_path(s))); + } } List compiler_sources = new LinkedList();