On Tue, Oct 11, 2016 at 04:58:26PM -0700, Andy Zhou wrote: > On Wed, Oct 5, 2016 at 6:28 PM, Ben Pfaff <b...@ovn.org> wrote: > > > When I ran "make check-valgrind -j10" and the testsuite needed to be > > rebuilt, two copies of it were rebuilt in parallel and sometimes they > > raced with each other. I don't have the full story on exactly why this > > happened, but this commit, which eliminates redundant dependencies from > > check-* targets, fixes the problem for me. The dependencies are redundant > > because these targets depend on "all", which also depends on them. > > > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > > > I was not clear to me either why the race happened, but the change looks > good to me. > > Acked-by: Andy Zhou <az...@ovn.org> > > There are two additional targets that have the similar pattern, wonder if > we should change them as well...
You're right. I made those changes too. My confidence is somewhat shaky on this, but it's easy enough to revert if we missed something, so I applied this to master. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
