Hi!

On Wed, 2025-05-14 at 16:44:46 +0200, Alexandre Detiste wrote:
> Package: release.debian.org
> Severity: normal
> X-Debbugs-Cc: xmlrp...@packages.debian.org, Guillem Jover <gjo...@sipwise.com>
> Control: affects -1 + src:xmlrpc-c
> User: release.debian....@packages.debian.org
> Usertags: unblock

> This is a pre-approval request.

Thanks for fixing and handling this!

> ----
> [ Reason ]
> xmlrpc-c/1.59.03-10 fix the FTBFS of the reverse dependencies
> whom for some other reasons end-up dependening on 'pkgconf'

Ah sorry, should have been more clear and probably give more details
which are not obvious, on the bug report!

This does not seem to currently affect any package in the archive,
but it breaks (external) projects that use pkgconf/pkg-config to get
its build flags.

> [ Impact ]
> That is not exactly clear to me, but I'm the one _learning_
> from all my previous & current interractions with Guillem;
> so I trust he's judgement.

*blush* :D

Here's the details I incorrectly obviated from the report. With the
version in trixie right now:

  ,---
  $ xmlrpc-c-config --libs | sed -e 's/^/STDOUT: /'; echo EXIT:$?
  Package expat was not found in the pkg-config search path.
  Perhaps you should add the directory containing `expat.pc'
  to the PKG_CONFIG_PATH environment variable
  Package 'expat', required by 'virtual:world', not found
  STDOUT: -L/usr/lib/x86_64-linux-gnu   -lxmlrpc   -lxmlrpc_util -lpthread 
  EXIT:0
  `---

Which is what I think most packages are currently using in Debian. But
the problem is with:

  ,---
  $ pkgconf --libs xmlrpc | sed -e 's/^/STDOUT: /'; echo EXIT:$?
  Package expat was not found in the pkg-config search path.
  Perhaps you should add the directory containing `expat.pc'
  to the PKG_CONFIG_PATH environment variable
  Package 'expat', required by 'xmlrpc', not found
  EXIT:1
  `---

So this does not ignore the errors, and then produces no output, so
library users end up failing to link against the library, where expat
is an implementation detail, that should only be needed when statically
linking.

With the fixed libxmlrpc-core-c3-dev package, which pulls in
libexpat1-dev, we should get instead:

  ,---
  $ pkgconf --libs xmlrpc | sed -e 's/^/STDOUT: /'; echo EXIT:$?
  STDOUT: -lxmlrpc -lxmlrpc_util 
  EXIT:0
  `---

  ,--
  $ pkgconf --static --libs xmlrpc | sed -e 's/^/STDOUT: /'; echo EXIT:$?
  STDOUT: -lxmlrpc -lxmlrpc_util -lexpat -lm 
  EXIT:0
  `---

Thanks,
Guillem

Reply via email to