Re: [libvirt PATCH 00/11] ci: Build RPMs on MinGW

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

 



On Thu, Nov 02, 2023 at 11:37:22PM +0100, Andrea Bolognani wrote:
> Test pipeline: https://gitlab.com/abologna/libvirt/-/pipelines/1059447352
> 
> The openSUSE Leap 15 job failed, but that's caused by an unrelated
> packaging issue:
> 
>   $ pkg-config --cflags libtirpc
>   -I/usr/include/tirpc
>   $ rpm -ql libtirpc-devel | grep usr/include
>   /usr/include/netconfig.h
>   /usr/include/rpc
>   /usr/include/rpc/auth.h
>   /usr/include/rpc/auth_des.h
>   ...
>   /usr/include/rpc/types.h
>   /usr/include/rpc/xdr.h
>   /usr/include/rpcsvc
>   /usr/include/rpcsvc/crypt.h
>   /usr/include/rpcsvc/crypt.x

That appears to be due to their update to the new version
that was pushed to repos this week. Their changelog says

  * pkg-config: use the correct replacements for libdir/includedir

but clearly that's been screwed up. THeir previous pkg-config
file had

   Cflags: -I/usr/include

The new one has

   Cflags: -I${includedir}/tirpc

>From upstream POV  libtirpc always installed into $include/tirpc,
but suse explicitly undo that to put it in the root.

In thier tumbleweed specfile they munged the .pc file to match,
but i guess that got lost in 15.5 branch :-(

  https://build.opensuse.org/package/view_file/openSUSE:Factory/libtirpc/libtirpc.spec?expand=1

If we want to avoid broken CI upstream after the contaniers get rebuilt
this weekend, we'll need to set CFLAGS="-Wno-missing-include-dirs" in
the suse job

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 :|
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx




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

  Powered by Linux