[GitHub] keith-turner commented on issue #957: Fluo scripts copy jars into fluo software dir.

[GitBox <gi...@apache.org>]

keith-turner commented on issue #957: Fluo scripts copy jars into fluo software dir.
URL: https://github.com/apache/fluo/issues/957#issuecomment-339143985
 
 
   I think this task would consist of the following :
     * Add `FLUO_TMP` to [fluo-env.sh](https://github.com/apache/fluo/blob/9e87f477463602533d8611c9561484d41d135654/modules/distribution/src/main/config/fluo-env.sh)
     * Modify the [fluo script](https://github.com/apache/fluo/blob/9e87f477463602533d8611c9561484d41d135654/modules/distribution/src/main/scripts/fluo) to use `mktemp` and `FLUO_TMP` everywhere `FluoGetJars` is run.
   
   For example the code [fluo line 230](https://github.com/apache/fluo/blob/9e87f477463602533d8611c9561484d41d135654/modules/distribution/src/main/scripts/fluo#L230) could be changed to the following :
   
   ```bash
       # create a temp dir to fetch application jars to
       app_lib=mktemp -d $FLUO_TMP/fluo-"$app"-XXXXXXXXX
       $JAVA org.apache.fluo.command.FluoGetJars -d "$app_lib" -a "$app"
       export CLASSPATH="$conf:$app_lib/*:$CLASSPATH"
       $JAVA org.apache.fluo.command.FluoExec "${@:2}"
       # remove the temp app lib dir
      rm -rf $app_lib
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services