DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=42616>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=42616 ------- Additional Comments From [EMAIL PROTECTED] 2007-10-31 13:07 ------- (In reply to comment #13) > So how about > > JARPATH=$(SEP="";find ${FOP}/lib -name '*.jar'|\ > while read jarfile;do echo -n $SEP"$jarfile"; SEP=${pathSepChar}; done) > > if [ -z "$LOCALCLASSPATH" ] ; then > LOCALCLASSPATH="$JARPATH" > else > LOCALCLASSPATH="$JARPATH"${pathSepChar}$LOCALCLASSPATH > fi > Excellent. But keep it simple and use =`...` instead of =$(...) (is this a bashism?). Then the following should replace the libs section in the fop script: # add in the dependency .jar files, which reside in $FOP_HOME/lib if [ -z "$LOCALCLASSPATH" ] ; then SEP="" else SEP="${pathSepChar}" fi LOCALCLASSPATH=${LOCALCLASSPATH}`find "${FOP_HOME}/lib" -name '*.jar' \ | while read jarfile do echo -n "$SEP$jarfile" SEP="${pathSepChar}" done` Note, however, that the complaint with which this bug was reopened, has never been validated; see comments #9 and #10. So one may argue that this solution is not sufficiently better to warrant being committed. I think it is, because it removes the fiddling with IFS and the parsing of a list of libs. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.
