Hi! Will send a new version for this tomorrow with just the extra command.
Thanks, Paul > -----Original Message----- > From: Joe Stringer [mailto:[email protected]] > Sent: Friday, June 3, 2016 1:56 AM > To: Paul Boca > Cc: [email protected] > Subject: Re: [ovs-dev] [PATCH V2 1/4] tests: Fixed access denied on ovs- > vswitchd.log > > On 1 June 2016 at 04:45, Paul Boca <[email protected]> wrote: > > On Windows trying to overwrite the opened ovs-vswitchd.log > > fails with access denied. Closing it before trying to overwrite it > > solves the problem > > > > Signed-off-by: Paul-Daniel Boca <[email protected]> > > Ideally we don't introduce platform differences into the tests; the > extra command should be available on both platforms, so you should be > able to just add "ovs-appctl vlog/close" rather than introducing the > if/else. > > If you don't have a linux environment to test, then you can push the > patches to your github and get travis to try it for you. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
