ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From co...@apache.org
Subject cvs commit: jakarta-ant/src/main/org/apache/tools/ant/taskdefs ExecTask.java Execute.java
Date Fri, 06 Jul 2001 11:15:02 GMT
conor       01/07/06 04:15:01

  Modified:    src/main/org/apache/tools/ant/taskdefs ExecTask.java
                        Execute.java
  Log:
  Add new vmlauncher attribute to exec. This defaults to true to execute
  commands using the VM's capabilities, where available. If it is set to false
  the underlying OS's shell will be used, either directly, or through the
  antRun scripts. Allows use of some shell features (such as associating scripts
  with their interpreters under Windows
  
  PR:	413
  
  Revision  Changes    Path
  1.14      +12 -0     jakarta-ant/src/main/org/apache/tools/ant/taskdefs/ExecTask.java
  
  Index: ExecTask.java
  ===================================================================
  RCS file: /home/cvs/jakarta-ant/src/main/org/apache/tools/ant/taskdefs/ExecTask.java,v
  retrieving revision 1.13
  retrieving revision 1.14
  diff -u -r1.13 -r1.14
  --- ExecTask.java	2001/03/13 03:04:06	1.13
  +++ ExecTask.java	2001/07/06 11:14:53	1.14
  @@ -80,6 +80,9 @@
       protected Commandline cmdl = new Commandline();
       private FileOutputStream fos = null;
   
  +    /** Controls whether the VM (1.3 and above) is used to execute the command */
  +    private boolean vmLauncher = true;
  +     
       /**
        * Timeout in milliseconds after which the process will be killed.
        */
  @@ -194,6 +197,14 @@
       }
   
       /**
  +     * Control whether the VM is used to launch the new process or
  +     * whether the OS's shell is used.
  +     */
  +    public void setVMLauncher(boolean vmLauncher) {
  +        this.vmLauncher = vmLauncher;
  +    }
  +    
  +    /**
        * Create an Execute instance with the correct working directory set.
        */
       protected Execute prepareExec() throws BuildException {
  @@ -205,6 +216,7 @@
           Execute exe = new Execute(createHandler(), createWatchdog());
           exe.setAntRun(project);
           exe.setWorkingDirectory(dir);
  +        exe.setVMLauncher(vmLauncher);
           String[] environment = env.getVariables();
           if (environment != null) {
               for (int i=0; i<environment.length; i++) {
  
  
  
  1.18      +44 -17    jakarta-ant/src/main/org/apache/tools/ant/taskdefs/Execute.java
  
  Index: Execute.java
  ===================================================================
  RCS file: /home/cvs/jakarta-ant/src/main/org/apache/tools/ant/taskdefs/Execute.java,v
  retrieving revision 1.17
  retrieving revision 1.18
  diff -u -r1.17 -r1.18
  --- Execute.java	2001/04/21 02:20:36	1.17
  +++ Execute.java	2001/07/06 11:14:55	1.18
  @@ -89,18 +89,21 @@
       private Project project = null;
       private boolean newEnvironment = false;
   
  +    /** Controls whether the VM is used to launch commands, where possible */
  +    private boolean useVMLauncher = true;    
  +    
       private static String antWorkingDirectory = System.getProperty("user.dir");
  -    private static CommandLauncher launcher = createCommandLauncher();
  +    private static CommandLauncher vmLauncher = null;
  +    private static CommandLauncher shellLauncher = null;
       private static Vector procEnvironment = null;
   
       /** 
        * Builds a command launcher for the OS and JVM we are running under
        */
  -    private static CommandLauncher createCommandLauncher()
  -    {
  +    static {
           // Try using a JDK 1.3 launcher
           try {
  -            return new Java13CommandLauncher();
  +            vmLauncher = new Java13CommandLauncher();
           }
           catch ( NoSuchMethodException exc ) {
               // Ignore and keep try
  @@ -109,11 +112,11 @@
           String osname = System.getProperty("os.name").toLowerCase();
           if ( osname.indexOf("mac os") >= 0 ) {
               // Mac
  -            return new MacCommandLauncher(new CommandLauncher());
  +            shellLauncher = new MacCommandLauncher(new CommandLauncher());
           }
           else if ( osname.indexOf("os/2") >= 0 ) {
               // OS/2 - use same mechanism as Windows 2000
  -            return new WinNTCommandLauncher(new CommandLauncher());
  +            shellLauncher = new WinNTCommandLauncher(new CommandLauncher());
           }
           else if ( osname.indexOf("windows") >= 0 ) {
               // Windows.  Need to determine which JDK we're running in
  @@ -130,16 +133,16 @@
               // Determine if we're running under 2000/NT or 98/95
               if ( osname.indexOf("nt") >= 0 || osname.indexOf("2000") >= 0 ) {
                   // Windows 2000/NT
  -                return new WinNTCommandLauncher(baseLauncher);
  +                shellLauncher = new WinNTCommandLauncher(baseLauncher);
               }
               else {
                   // Windows 98/95 - need to use an auxiliary script
  -                return new ScriptCommandLauncher("bin/antRun.bat", baseLauncher);
  +                shellLauncher = new ScriptCommandLauncher("bin/antRun.bat", baseLauncher);
               }
           }
           else {
               // Generic
  -            return new ScriptCommandLauncher("bin/antRun", new CommandLauncher());
  +            shellLauncher = new ScriptCommandLauncher("bin/antRun", new CommandLauncher());
           }
       }
   
  @@ -353,6 +356,19 @@
       }
   
       /**
  +     * Launch this execution through the VM, where possible, rather than through
  +     * the OS's shell. In some cases and operating systems using the shell will 
  +     * allow the shell to perform additional processing such as associating an 
  +     * executable with a script, etc
  +     *
  +     * @param vmLauncher true if exec should launch through thge VM, 
  +     *                   false if the shell should be used to launch the command.
  +     */
  +    public void setVMLauncher(boolean useVMLauncher) {
  +        this.useVMLauncher = useVMLauncher;
  +    }
  +    
  +    /**
        * Runs a process defined by the command line and returns its exit status.
        *
        * @return the exit status of the subprocess or <code>INVALID</code>
  @@ -360,6 +376,11 @@
        *            of the subprocess failed
        */
       public int execute() throws IOException {
  +        CommandLauncher launcher = vmLauncher != null ? vmLauncher : shellLauncher;
  +        if (!useVMLauncher) {
  +            launcher = shellLauncher;
  +        }
  +        
           final Process process = launcher.exec(project, getCommandline(), getEnvironment(),
workingDirectory);
           try {
               streamHandler.setProcessInputStream(process.getOutputStream());
  @@ -605,8 +626,9 @@
            */
           public Process exec(Project project, String[] cmd, String[] env, File workingDir)
throws IOException
           {
  +            File commandDir = workingDir;
               if ( workingDir == null ) {
  -                return exec(project, cmd, env);
  +                commandDir = project.getBaseDir();
               }
   
               // Use cmd.exe to change to the specified directory before running
  @@ -617,7 +639,7 @@
               newcmd[1] = "/c";
               newcmd[2] = "cd";
               newcmd[3] = "/d";
  -            newcmd[4] = workingDir.getAbsolutePath();
  +            newcmd[4] = commandDir.getAbsolutePath();
               newcmd[5] = "&&";
               System.arraycopy(cmd, 0, newcmd, preCmdLength, cmd.length);
   
  @@ -674,14 +696,14 @@
            */
           public Process exec(Project project, String[] cmd, String[] env, File workingDir)
throws IOException
           {
  -            if ( workingDir == null ) {
  -                return exec(project, cmd, env);
  -            }
  -
  -            // Locate the auxiliary script
               if ( project == null ) {
  +                if ( workingDir == null ) {
  +                    return exec(project, cmd, env);
  +                }
                   throw new IOException("Cannot locate antRun script: No project provided");
               }
  +            
  +            // Locate the auxiliary script
               String antHome = project.getProperty("ant.home");
               if ( antHome == null ) {
                   throw new IOException("Cannot locate antRun script: Property 'ant.home'
not found");
  @@ -689,9 +711,14 @@
               String antRun = project.resolveFile(antHome + File.separator + _script).toString();
   
               // Build the command
  +            File commandDir = workingDir;
  +            if ( workingDir == null ) {
  +                commandDir = project.getBaseDir();
  +            }
  +
               String[] newcmd = new String[cmd.length + 2];
               newcmd[0] = antRun;
  -            newcmd[1] = workingDir.getAbsolutePath();
  +            newcmd[1] = commandDir.getAbsolutePath();
               System.arraycopy(cmd, 0, newcmd, 2, cmd.length);
               
               return exec(project, newcmd, env);
  
  
  

Mime
View raw message