Hi! > > + 84?? .??cmdlib.sh > > Patch series looks good to me except this.
The patch series looks really good. > sourcing a lib inside a function seems not so good. Yet I don't see any > code style rules telling which is the correct place to put a shell lib > source (commonly I think it would be the top in the file) It's always better to include the bash library first so that your variables are not rewritten by these used in the library. I've seen cases where this was a issue (one test included a file from init scripts which is different in diferent version of distribution and in one case the name clashed) and the test was broken because the file wasn't included at the top. So actually this is not hard requirement but rather a good practice. -- Cyril Hrubis [email protected] ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
