On Fri, Feb 24, 2012 at 01:01:38PM -0800, Ethan Jackson wrote:
> There's some trailing whitespace in one of the man pages.
> 
> >  %if %{?kernel_name:0}%{!?kernel_name:1}
> > -%define kernel %(rpm -qa 'kernel*xen-devel')
> > +%define kernel %(rpm -qa 'kernel*xen-devel' | head -1)
> >  %define kernel_name %(rpm -q --queryformat "%%{Name}" %{kernel} | sed 
> > 's/-devel//' | sed 's/kernel-//')
> >  %define kernel_version %(rpm -q --queryformat "%%{Version}-%%{Release}" 
> > %{kernel})
> >  %define kernel_flavor xen
> 
> Is this change intended for this patch?

Hmm, that's odd.  It wasn't in the version of this patch that I had in
my development repo.  Perhaps I stomped on it after I posted the
patch (but I don't remember doing that).

> Thanks for doing this, I've always found the vlog syntax a bit
> confusing, I think this is a significant improvement.  I'll put
> updating the python version on my TODO list.

Thanks.

I'll push these patches soon.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to