Fixed a bug to pass build directory to javax.tools compiler

barretg · barretg · commit a56edfd79470 · 2024-02-28T12:12:58.000-06:00
diff --git a/drjava/src/edu/rice/cs/drjava/model/compiler/JavaxToolsCompiler.java b/drjava/src/edu/rice/cs/drjava/model/compiler/JavaxToolsCompiler.java
@@ -74,8 +74,16 @@ public List<? extends DJError> compile(List<? extends File> files, List<? extend
             optionList.add(sourceVersion);
         }
         if (destination != null) {
-            optionList.add("-d");
-            optionList.add(destination.getAbsolutePath());
+            try {
+                fileManager.setLocation(StandardLocation.CLASS_OUTPUT, Collections.singletonList(destination));
+            } catch (IOException e) {
+                List<DJError> errors = new ArrayList<>();
+                errors.add(new DJError("Error setting build directory: " + e.getMessage(), false));
+                return errors;
+            }
+            // This doesn't work for javax.tools compiler
+            // optionList.add("-d");
+            // optionList.add(destination.getAbsolutePath());
         }
 
         // Prepare a diagnostic collector to collect compile errors
