Re: [PATCH] build: don't export avahi syms when unused

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

 



On Tue, Jul 31, 2012 at 10:54:52AM +0200, Jiri Denemark wrote:
> On Mon, Jul 30, 2012 at 17:31:20 -0600, Eric Blake wrote:
> > Detected when building --without-avahi.
> > 
> > * src/libvirt_private.syms (virnetservermdns.h): Move...
> > * src/libvirt_avahi.syms: ...to new file.
> > * src/Makefile.am (USED_SYM_FILES, EXTRA_DIST): Use it.
> > ---
> 
> ACK
> 
> > I'm debating whether to push this under the build-breaker rule.
> > Unfortunately, while this solved my build on an Ubuntu machine,
> > I have other machines which are failing for other symbols (some
> > SASL related, and one for xenLinuxDomainBlockStats), so it's
> > probably better to fix everything instead of just half the problem.
> 
> I agree it would be nice to fix all issues but we can fix them with several
> patches, esp. since each of the issue may need to be solved in different way.
> 
> My build also fails for xenLinuxDomainBlockStats, which should be easy to fix.
> The symbol is defined and used within Xen driver and nowhere else, thus the
> following patch should solve it:
> 
>     diff --git a/src/libvirt_linux.syms b/src/libvirt_linux.syms
>     index 2258739..67c1d7f 100644
>     --- a/src/libvirt_linux.syms
>     +++ b/src/libvirt_linux.syms
>     @@ -5,7 +5,6 @@
>      
>      # stats_linux.h
>      linuxDomainInterfaceStats;
>     -xenLinuxDomainBlockStats;
>      
>      # nodeinfo.h
>      linuxNodeInfoCPUPopulate;

ACK to that - I see no reason for it to be exported.

> However, my build also fails for 5 network symbols. I wonder if it's the same
> case as with xenLinuxDomainBlockStats. It's probably worth mentioning that I
> build with --without-driver-modules.

Can you show me what the actual symbols you see are


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
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]