Great, I see the difference now. Looks good.

Acked-by: Joe Stringer <[email protected]>


On 22 January 2014 08:55, Ethan Jackson <[email protected]> wrote:

> Yep, you're missing something.  This patch suppresses the output of
> the actual configure command.  The second patch suppresses the output
> of the makefiles configure generates.
>
> Ethan
>
> On Tue, Jan 21, 2014 at 5:29 PM, Joe Stringer <[email protected]>
> wrote:
> > Am I missing something? This appears to be overridden by Patch #2.
> >
> > On 21 January 2014 17:05, Ethan Jackson <[email protected]> wrote:
> >>
> >> Signed-off-by: Ethan Jackson <[email protected]>
> >> ---
> >>  utilities/ovs-dev.py | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/utilities/ovs-dev.py b/utilities/ovs-dev.py
> >> index 7b69094..80f0b80 100755
> >> --- a/utilities/ovs-dev.py
> >> +++ b/utilities/ovs-dev.py
> >> @@ -63,7 +63,7 @@ def conf():
> >>      configure = ["../configure", "--prefix=" + ROOT,
> "--localstatedir=" +
> >> ROOT,
> >>                   "--with-logdir=%s/log" % ROOT, "--with-rundir=%s/run"
> %
> >> ROOT,
> >>                   "--with-linux=/lib/modules/%s/build" % uname(),
> >> -                 "--with-dbdir=" + ROOT]
> >> +                 "--with-dbdir=" + ROOT, "--silent"]
> >>
> >>      if options.werror:
> >>          configure.append("--enable-Werror")
> >> --
> >> 1.8.1.2
> >>
> >> _______________________________________________
> >> dev mailing list
> >> [email protected]
> >> http://openvswitch.org/mailman/listinfo/dev
> >
> >
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to