[ovs-dev] [PATCH] vlog: Be more liberal in syntax for -v and vlog/set.

Ethan Jackson ethan at nicira.com
Fri Feb 24 21:01:38 UTC 2012


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?

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.

Looks good,
Ethan



More information about the dev mailing list