On 03/13/2014 10:52 PM, [email protected] wrote: > Hi! >> This patch is a long cleanup, thanks for reviewing this. >>> * The C option is removed >>> (this one makes sense because it's not supported) >> Yes. >>> * The B background option is removed >> Here I think the "-B" option does not make much sense. >> And if "-B" option is specified, the parent process will always >> return 0, which won't reflect the correct test results. >>> * The d dir option no longer copies the dir name, why? >> "-d" option is used to specify a directory, where named pipe is created, >> other >> than the current directory. Tst_tmpdir() will create a temporary directory, >> so I think >> "-d" option may be removed. > This is exactly what should be in the commit message. Because if you > don't write the motivation for the changes there, I need to figure it > out myself, which takes much more time.
OK, I see. Thanks. Wang > ------------------------------------------------------------------------------ Learn Graph Databases - Download FREE O'Reilly Book "Graph Databases" is the definitive new guide to graph databases and their applications. Written by three acclaimed leaders in the field, this first edition is now available. Download your free book today! http://p.sf.net/sfu/13534_NeoTech _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
