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@lists.libvirt.org
To unsubscribe send an email to devel-le...@lists.libvirt.org

Reply via email to