Re: [PATCH] Add a test case that checks there are no bogus entries in .syms

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/24/2012 09:48 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
>
> During refactoring of code, it has proved common to forget to
> remove old symbols from the .syms file. While the Win32 linker
> will complain about this, the Linux ELF linker does not. The
> new test case validates that every symbol listed in the .syms
> file actually exists in the built ELF libraries.

And apparently it works - it found the following three unimplemented
functions in a fresh checkout from master:

    GEN    check-symfile
  Expected symbol virNetClientSetEOFNotify is not in ELF library
  Expected symbol virNWFilterGetIpAddrForIfname is not in ELF library
  Expected symbol virNWFilterDelIpAddrForIfname is not in ELF library
  make[2]: *** [check-symfile] Error 1


It would be even nicer if it 1) verified that each symbol was defined in
the proper symfile for its library, and 2) was able to check even those
libraries that weren't built for the given platform, but (1) isn't
really practical, since we currently mix a lot of stuff together in
libvirt_private.syms, and I can't think of a reasonable way to do (2).

ACK (assuming that anything that causes it to fail is fixed in git
first, of course).


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]