On Fri, May 06, 2022 at 01:51:53PM +0200, Claudio Fontana wrote: > On 5/6/22 1:02 PM, Daniel P. Berrangé wrote: > > On Thu, May 05, 2022 at 02:52:17PM +0200, Claudio Fontana wrote: > >> where it can be reused by other helpers. > >> No changes other than the move. > >> > >> Note that this makes iohelper now dependent on -lutil, because unused > >> (for iohelper) parts of virfile.c contain calls to openpty(3). > > > > Needs -lacl too on F35 at least. > > do we end up needing the whole of the virt_util_lib dependencies for this? > > dependencies: [ > acl_dep, > audit_dep, > capng_dep, > devmapper_dep, > gnutls_dep, > intl_dep, > libm_dep, > libnl_dep, > libutil_dep, > numactl_dep, > secdriver_dep, > src_dep, > thread_dep, > win32_dep, > yajl_dep, > ] > > iohelper did not need -lutil or anything else before, just a file. > going back to just a separate helper_runio.c/h would make sense to me, but > let me know.
The linker will discard all the .o files that don't have any functions referenced, so we'll not need most of what virt_util_lib depends on, just the virfile.o deps. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|