This is an automated email from the ASF dual-hosted git repository.

jaikiran pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ant.git


The following commit(s) were added to refs/heads/master by this push:
     new 616db837d Removed outdated Microsoft vendor hardcoding which can cause 
errors
616db837d is described below

commit 616db837d1e05c41f158dc0813cf1b10e481b255
Author: jmjaffe37 <[email protected]>
AuthorDate: Mon Apr 13 09:12:40 2026 -0700

    Removed outdated Microsoft vendor hardcoding which can cause errors
---
 src/main/org/apache/tools/ant/types/Path.java | 66 ++++++++++++---------------
 1 file changed, 28 insertions(+), 38 deletions(-)

diff --git a/src/main/org/apache/tools/ant/types/Path.java 
b/src/main/org/apache/tools/ant/types/Path.java
index 46d37bd2c..110325279 100644
--- a/src/main/org/apache/tools/ant/types/Path.java
+++ b/src/main/org/apache/tools/ant/types/Path.java
@@ -608,49 +608,39 @@ public class Path extends DataType implements Cloneable, 
ResourceCollection {
             addExisting(systemBootClasspath);
         }
 
-        if 
(System.getProperty("java.vendor").toLowerCase(Locale.ENGLISH).contains("microsoft"))
 {
-            // TODO is this code still necessary? is there any 1.2+ port?
-            // Pull in *.zip from packages directory
-            FileSet msZipFiles = new FileSet();
-            msZipFiles.setDir(new File(JavaEnvUtils.getJavaHome()
-                    + File.separator + "Packages"));
-            msZipFiles.setIncludes("*.ZIP");
-            addFileset(msZipFiles);
-        } else {
-            // JDK 1.2+ seems to set java.home to the JRE directory.
-            addExisting(new Path(null, JavaEnvUtils.getJavaHome()
-                    + File.separator + "lib" + File.separator + "rt.jar"));
-            // Just keep the old version as well and let addExisting
-            // sort it out.
+        // JDK 1.2+ seems to set java.home to the JRE directory.
+        addExisting(new Path(null, JavaEnvUtils.getJavaHome()
+                + File.separator + "lib" + File.separator + "rt.jar"));
+        // Just keep the old version as well and let addExisting
+        // sort it out.
+        addExisting(new Path(null, JavaEnvUtils.getJavaHome()
+                + File.separator + "jre" + File.separator + "lib"
+                + File.separator + "rt.jar"));
+
+        // Sun's and Apple's 1.4 have JCE and JSSE in separate jars.
+        for (String secJar : Arrays.asList("jce", "jsse")) {
             addExisting(new Path(null, JavaEnvUtils.getJavaHome()
-                    + File.separator + "jre" + File.separator + "lib"
-                    + File.separator + "rt.jar"));
-
-            // Sun's and Apple's 1.4 have JCE and JSSE in separate jars.
-            for (String secJar : Arrays.asList("jce", "jsse")) {
-                addExisting(new Path(null, JavaEnvUtils.getJavaHome()
-                        + File.separator + "lib"
-                        + File.separator + secJar + ".jar"));
-                addExisting(new Path(null, JavaEnvUtils.getJavaHome()
-                        + File.separator + ".." + File.separator + "Classes"
-                        + File.separator + secJar + ".jar"));
-            }
-
-            // IBM's 1.4 has rt.jar split into 4 smaller jars and a combined
-            // JCE/JSSE in security.jar.
-            for (String ibmJar : Arrays.asList("core", "graphics", "security", 
"server", "xml")) {
-                addExisting(new Path(null, JavaEnvUtils.getJavaHome()
-                        + File.separator + "lib" + File.separator + ibmJar + 
".jar"));
-            }
-
-            // Added for MacOS X
+                    + File.separator + "lib"
+                    + File.separator + secJar + ".jar"));
             addExisting(new Path(null, JavaEnvUtils.getJavaHome()
                     + File.separator + ".." + File.separator + "Classes"
-                    + File.separator + "classes.jar"));
+                    + File.separator + secJar + ".jar"));
+        }
+
+        // IBM's 1.4 has rt.jar split into 4 smaller jars and a combined
+        // JCE/JSSE in security.jar.
+        for (String ibmJar : Arrays.asList("core", "graphics", "security", 
"server", "xml")) {
             addExisting(new Path(null, JavaEnvUtils.getJavaHome()
-                    + File.separator + ".." + File.separator + "Classes"
-                    + File.separator + "ui.jar"));
+                    + File.separator + "lib" + File.separator + ibmJar + 
".jar"));
         }
+
+        // Added for MacOS X
+        addExisting(new Path(null, JavaEnvUtils.getJavaHome()
+                + File.separator + ".." + File.separator + "Classes"
+                + File.separator + "classes.jar"));
+        addExisting(new Path(null, JavaEnvUtils.getJavaHome()
+                + File.separator + ".." + File.separator + "Classes"
+                + File.separator + "ui.jar"));
     }
 
     /**

Reply via email to