[libvirt] fix two "make syntax-check" failures

Daniel Veillard veillard at redhat.com
Thu Jul 9 21:04:14 UTC 2009


On Thu, Jul 09, 2009 at 08:09:05PM +0200, Jim Meyering wrote:
> Mark McLoughlin pointed out this failure:
> 
>     ChangeLog-old: if (foo) free (foo)
>     ChangeLog-old: if (foo != NULL) free (foo)
>     maint.mk: found useless "if" before "free" above
>     make: *** [sc_avoid_if_before_free] Error 1

  Heh, :-)

> Here's the other:
> 
> --- po/POTFILES.in
> +++ po/POTFILES.in
> @@ -1,3 +1,4 @@
> +cfg.mk
>  qemud/qemud.c
>  qemud/remote.c
>  src/bridge.c
> maint.mk: you have changed the set of files with translatable diagnostics;
>  apply the above patch
> 
> I had fixed the latter in gnulib (now that main.mk comes
> from there), but had forgotten to tell libvirt to use
> the latest version of gnulib, in order to get that fix.
> To do that, I ran "git syncsub" (where syncsub is an alias:
> syncsub = submodule foreach git pull origin master)
> then git commit -a.
> 
> Here are the two patches I'm about to push:

  Fine, ACK,

  thanks !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list