Hi! I did ran the test on Linux but failed saying vlog/close not recognized, maybe it was my mistake and didn't updated the sources. I will update and retest before submitting the next version.
Regards, Paull > -----Original Message----- > From: Ben Pfaff [mailto:b...@ovn.org] > Sent: Friday, June 3, 2016 2:05 AM > To: Joe Stringer > Cc: Paul Boca; dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH V2 1/4] tests: Fixed access denied on ovs- > vswitchd.log > > On Thu, Jun 02, 2016 at 03:55:41PM -0700, Joe Stringer wrote: > > On 1 June 2016 at 04:45, Paul Boca <pb...@cloudbasesolutions.com> 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 <pb...@cloudbasesolutions.com> > > > > 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. > > Or for something as simple as this, just note in the change log that you > only tested on Windows, and then whoever reviews it should run the test > locally (I often do this in any case). _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
