On Thu, Sep 04, 2014 at 12:30:19PM +0200, chru...@suse.cz wrote:
> Hi!
> > parent use signal handler to deal with SIGTERM, make sure children processes
> > will be over before parent.
> > command "killall fsstress; wait;" can be used with this patch.
> 
> This patch seems to be verbatim copy of the fix included in the xfstests
> repository. You should at least mention the original authors in the
> commit message here.

Seems Zorro mentioned "backporting the fix from xfstests" in his v1
patch, not sure why he left that part out in v2..

http://sourceforge.net/p/ltp/mailman/ltp-list/thread/53E4EE6C.6090307%40redhat.com/#msg32696440

> 
> And the commit message for the original patch is better as well...
> 
> I'm all for getting back the fixes from xfstests but taking the patch
> and not giving credit to the original authors is plain wrong.

Agreed

Thanks,
Eryu Guan
> 
> -- 
> Cyril Hrubis
> chru...@suse.cz
> 
> ------------------------------------------------------------------------------
> Slashdot TV.  
> Video for Nerds.  Stuff that matters.
> http://tv.slashdot.org/
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list

------------------------------------------------------------------------------
Slashdot TV.  
Video for Nerds.  Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to