Re: [ovs-dev] [PATCH 1/2] tests: Add ability to automatically rerun failed tests.

2016-02-24 Thread Ben Pfaff 
On Wed, Feb 24, 2016 at 02:34:11PM -0800, Andy Zhou wrote:
> On Tue, Feb 9, 2016 at 11:44 AM, Ben Pfaff  wrote:
> 
> > A lot of packaging was doing this already, so this simplifies their
> > implementation.
> >
> > Signed-off-by: Ben Pfaff 
> 
> 
> Is it worth while to mention this flag in "CONTRIBUTING.md" or make a
> reference to the "Self-Tests"
> section in the INSTALL.md?

Good idea.  I added a sentence in CONTRIBUTING.md:

  - A patch that modifies existing code should be tested with `make
check` before submission.  Please see INSTALL.md, under
"Self-Tests", for more information.

Thanks.
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 1/2] tests: Add ability to automatically rerun failed tests.

2016-02-24 Thread Andy Zhou 
On Tue, Feb 9, 2016 at 11:44 AM, Ben Pfaff  wrote:

> A lot of packaging was doing this already, so this simplifies their
> implementation.
>
> Signed-off-by: Ben Pfaff 


Is it worth while to mention this flag in "CONTRIBUTING.md" or make a
reference to the "Self-Tests"
section in the INSTALL.md?

At any rate, the change is a nice, Thanks!

Acked-by: Andy Zhou 
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 1/2] tests: Add ability to automatically rerun failed tests.

2016-02-22 Thread Ben Pfaff 
On Tue, Feb 09, 2016 at 11:44:39AM -0800, Ben Pfaff wrote:
> A lot of packaging was doing this already, so this simplifies their
> implementation.
> 
> Signed-off-by: Ben Pfaff 

This could use a review.
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev