Hi! On 06/16/2014 10:22 AM, Mike Frysinger wrote: > On Thu 27 Mar 2014 18:22:52 Stanislav Kholmanskikh wrote: >> +MNT_POINT="/tmp/isofs_$$" >> +COPY_DIR="/etc/" >> +TEMP_DIR="/tmp/for_isofs_test" >> +MAKE_FILE_SYS_DIR=$TEMP_DIR$COPY_DIR > > not really a new issue, but scripts really should never hardcode paths in > /tmp/ or to that dir at all. this is what `mktemp -d` is for. > >> +if [ "$NO_CLEANUP" == "no" ]; then > > it is "=" in POSIX, not "==" > -mike >
Thank you for review. But both issues are fixed in the next patch in the sequence - "[PATCH V3 2/4] testscripts/isofs.sh: use the new shell interface". ------------------------------------------------------------------------------ HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions Find What Matters Most in Your Big Data with HPCC Systems Open Source. Fast. Scalable. Simple. Ideal for Dirty Data. Leverages Graph Analysis for Fast Processing & Easy Data Exploration http://p.sf.net/sfu/hpccsystems _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
