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