• Bug#1105769: unblock: xmlrpc-c/1.59.03-10

    From Guillem Jover@21:1/5 to Alexandre Detiste on Wed May 14 20:00:02 2025
    XPost: linux.debian.devel.release

    Hi!

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

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Paul Gevers@21:1/5 to Alexandre Detiste on Thu May 15 11:00:01 2025
    XPost: linux.debian.devel.release
    To: 1105769@bugs.debian.org

    This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --------------O6auoUKehQMk0sUZJFjX5pi1
    Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: base64

    SGksDQoNCk9uIDE0LTA1LTIwMjUgMTY6NDQsIEFsZXhhbmRyZSBEZXRpc3RlIHdyb3RlOg0K PiBUaGlzIGlzIGEgcHJlLWFwcHJvdmFsIHJlcXVlc3QuDQoNCg0KSnVzdCB3b25kZXJpbmcg KGFuZCBub3QgYSBibG9ja2VyKSwgd2h5IGRpZCB5b3UgZmlsZSB0aGlzIGFzIGEgDQpwcmUt YXBwcm92YWw/IFRoZSBwYWNrYWdlIGlzIG9uIHRoZSBhdXRvcmVtb3ZhbCBsaXN0IGFuZCBk ZWxheWluZyB0aGUgDQp1cGxvYWQgcmVkdWNlcyB0aGUgdGltZSBmb3IgdGVzdGluZyB0aGUg Zml4IGJ5IG90aGVycy4NCg0KPiBbIFJlYXNvbiBdDQo+IHhtbHJwYy1jLzEuNTkuMDMtMTAg Zml4IHRoZSBGVEJGUyBvZiB0aGUgcmV2ZXJzZSBkZXBlbmRlbmNpZXMNCj4gd2hvbSBmb3Ig c29tZSBvdGhlciByZWFzb25zIGVuZC11cCBkZXBlbmRlbmluZyBvbiAncGtnY29uZicNCg0K DQpJIHVuZGVyc3RhbmQgZnJvbSB0aGUgcmVwbHkgZnJvbSBHdWlsbGVtIHRoYXQgdGhpcyBk b2Vzbid0IGFmZmVjdCB0aGUgDQpEZWJpYW4gYXJjaGl2ZS4NCg0KPiBJIHJldnVpbGQgdGhl IHJldmVyc2UgZGVwZW5kZW5jaWVzIGFnYWluIGp1c3QgZmluZS4NCj4gDQo+ICAgIFJldmVy c2UtQnVpbGQtRGVwZW5kcw0KPiAgICA9PT09PT09PT09PT09PT09PT09PT0NCj4gICAgKiBm bG93Z3JpbmQgICAgICAgICAgICAgICAgICAgICAoZm9yIGxpYnhtbHJwYy1jb3JlLWMzLWRl dikNCj4gICAgKiBydG9ycmVudCAgICAgICAgICAgICAgICAgICAgICAoZm9yIGxpYnhtbHJw Yy1jb3JlLWMzLWRldikNCj4gICAgKiBydHBlbmdpbmUgICAgICAgICAgICAgICAgICAgICAo Zm9yIGxpYnhtbHJwYy1jb3JlLWMzLWRldikNCj4gICAgKiB0bGYgICAgICAgICAgICAgICAg ICAgICAgICAgICAoZm9yIGxpYnhtbHJwYy1jb3JlLWMzLWRldikNCg0KDQpUaGFua3MgZm9y IHRlc3RpbmcgdGhhdC4gRGlkIHlvdSBkbyBhbnkgY2hlY2tpbmcgdGhhdCB0aGUgYnVpbGQg d2FzIGFsc28gDQpjb3JyZWN0PyBJZiBzbywgd2hhdCBkaWQgeW91IGNoZWNrPw0KDQo+IHRo ZSByZW1haW5pbmcgZGViZGlmZiBpcyBzbWFsbA0KDQoNCkkgdW5kZXJzdGFuZCB0aGF0IHRo ZSByZWFsIGV4cGF0IGZpeCBpcyB0aGUgZm9sbG93aW5nOg0KDQo+IGRpZmYgLS1naXQgYS9k ZWJpYW4vY29udHJvbCBiL2RlYmlhbi9jb250cm9sDQo+IGluZGV4IGM3ZDkwNDEuLmVmMDAw YzcgMTAwNjQ0DQo+IC0tLSBhL2RlYmlhbi9jb250cm9sDQo+ICsrKyBiL2RlYmlhbi9jb250 cm9sDQo+IEBAIC02MCw2ICs2MCw3IEBAIEFyY2hpdGVjdHVyZTogYW55DQo+ICAgRGVwZW5k czoNCj4gICAgbGliYzYtZGV2LA0KPiAgICBsaWJjdXJsNC1vcGVuc3NsLWRldiB8IGxpYmN1 cmw0LWdudXRscy1kZXYsDQo+ICsgbGliZXhwYXQxLWRldiwNCj4gICAgbGlieG1scnBjLWNv cmUtYzN0NjQgKD0gJHtiaW5hcnk6VmVyc2lvbn0pLA0KPiAgICBsaWJ4bWxycGMtdXRpbC1k ZXYsDQo+ICAgICR7bWlzYzpEZXBlbmRzfSwNCj4gLS0tIGEvZGViaWFuL3BhdGNoZXMveG1s cnBjLWMtMS41OS4wMy11c2Utc3lzdGVtLWV4cGF0LnBhdGNoDQo+ICsrKyBiL2RlYmlhbi9w YXRjaGVzL3htbHJwYy1jLTEuNTkuMDMtdXNlLXN5c3RlbS1leHBhdC5wYXRjaA0KPiBAQCAt MTIwLDYgKzEyMCwxNiBAQCBTdWJqZWN0OiBbUEFUQ0hdIFVzZSBzeXN0ZW0gbGliZXhwYXQg cmF0aGVyIHRoYW4gYnVuZGxlZCBsaWIvZXhwYXQvIGZvcg0KPiAgICBlbmRpZg0KPiAgICAN Cj4gICAgJChMSUJYTUxSUENfTU9EUzolPSUubykgXA0KPiArQEAgLTMzOSw3ICszMzcsOCBA QA0KPiArICAgICAgIEBlY2hvICdEZXNjcmlwdGlvbjogWG1scnBjLWMgYmFzaWMgWE1MLVJQ QyBsaWJyYXJ5JyAgICAgICAgICAgICAgICA+PiRADQo+ICsgICAgICAgQGVjaG8gJ1ZlcnNp b246ICAgICAkKFhNTFJQQ19WRVJTSU9OX1NUUklORyknICAgICAgICAgICAgICAgICAgICAg ID4+JEANCj4gKyAgICAgICBAZWNobyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgPj4kQA0KPiArLSAgICAgIEBlY2hvICdS ZXF1aXJlczogeG1scnBjX3V0aWwgJChYTUxfUEtHQ09ORklHX1JFUSknICAgICAgICAgICAg ICAgICA+PiRADQo+ICsrICAgICAgQGVjaG8gJ1JlcXVpcmVzOiB4bWxycGNfdXRpbCcgICAg ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgID4+JEANCj4gKysgICAgICBAZWNo byAnUmVxdWlyZXMucHJpdmF0ZTogJChYTUxfUEtHQ09ORklHX1JFUSknICAgICAgICAgICAg ICAgICAgICAgPj4kQA0KPiArICAgICAgIEBlY2hvICdMaWJzOiAgICAgLUwkJHtsaWJkaXJ9 IC1seG1scnBjJyAgICAgICAgICAgICAgICAgICAgICAgICAgICA+PiRADQo+ICsgICAgICAg QGVjaG8gJ0NmbGFnczogICAtSSQke2luY2x1ZGVkaXJ9JyAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgID4+JEANCj4gKw0KDQpJIG11c3QgYWRtaXQgSSBkb24ndCBvdmVyc2Vl IGFsbCB0aGUgY29uc2VxdWVuY2VzIGhlcmUsIHNvIEkgaG9wZSB0aGF0IA0Kb25lIG9mIHRo ZSBvdGhlciBSVCBtZW1iZXJzIGNhbiBkb3VibGUgY2hlY2ssIGJ1dCBJIHRoaW5rIHlvdSBz aG91bGQgDQp1cGxvYWQgdG8gdW5zdGFibGUgYWxyZWFkeS4NCg0KV2h5IHdlcmUgdGhlIGhh cmRlbmluZyBjaGFuZ2VzIGRyb3BwZWQgZWFybGllcj8NCg0KUGF1bA0KDQo=

    --------------O6auoUKehQMk0sUZJFjX5pi1--

    -----BEGIN PGP SIGNATURE-----

    wsC7BAABCABvBYJoJasBCRCcXJnrBb11CkcUAAAAAAAeACBzYWx0QG5vdGF0aW9u cy5zZXF1b2lhLXBncC5vcmduNCKfZso9HLuPcoCLk+b71+vkfS9GIqnVA4U8OdMu fBYhBFi2bUhza+k7BS3mcpxcmesFvXUKAADNswgAkPZeyYz74vQ89Gi4IwsP4aBp 6nRqdfq17kjNaZ/5pEqKFxKg87qZoEYGcwWwX1gjz9UzZ5VIXZ8QjZRvo/UBc5FE 6pBkCxbFy+Lh4ipGbfNGD4RfkWFBKapo66yWzZbXgasSXwxYJ77prlJCOvqp4ssa Uo+TOO3r8R2GBwDKDJ2JCk4uRiIewB2AugovtQGKoANIAnUNZ3Ita7ChfeD6Fnd8 s8sn87jZUXuYZGaBTG9Tc8z8U/roGjKYbdmVdHlIsSiKiKo2Vv8eh7QDxiRQCT1d 5aukQh/iMAJ0IaTDCKw9MVTsdeHZ7n2UPvCheYBqV/fdy82kemNTjdhg0tmmvA==
    =1kF0
    -----END PGP SIGNATURE-----

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Sebastian Ramacher@21:1/5 to Paul Gevers on Thu May 15 11:30:01 2025
    XPost: linux.debian.devel.release

    On 2025-05-15 10:51:13 +0200, Paul Gevers wrote:
    Hi,

    On 14-05-2025 16:44, Alexandre Detiste wrote:
    This is a pre-approval request.


    Just wondering (and not a blocker), why did you file this as a pre-approval? The package is on the autoremoval list and delaying the upload reduces the time for testing the fix by others.

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


    I understand from the reply from Guillem that this doesn't affect the Debian archive.

    I revuild the reverse dependencies again just fine.

    Reverse-Build-Depends
    =====================
    * flowgrind (for libxmlrpc-core-c3-dev)
    * rtorrent (for libxmlrpc-core-c3-dev)
    * rtpengine (for libxmlrpc-core-c3-dev)
    * tlf (for libxmlrpc-core-c3-dev)


    Thanks for testing that. Did you do any checking that the build was also correct? If so, what did you check?

    the remaining debdiff is small


    I understand that the real expat fix is the following:

    diff --git a/debian/control b/debian/control
    index c7d9041..ef000c7 100644
    --- a/debian/control
    +++ b/debian/control
    @@ -60,6 +60,7 @@ Architecture: any
    Depends:
    libc6-dev,
    libcurl4-openssl-dev | libcurl4-gnutls-dev,
    + libexpat1-dev,
    libxmlrpc-core-c3t64 (= ${binary:Version}),
    libxmlrpc-util-dev,
    ${misc:Depends},
    --- a/debian/patches/xmlrpc-c-1.59.03-use-system-expat.patch
    +++ b/debian/patches/xmlrpc-c-1.59.03-use-system-expat.patch
    @@ -120,6 +120,16 @@ Subject: [PATCH] Use system libexpat rather than bundled lib/expat/ for
    endif
    $(LIBXMLRPC_MODS:%=%.o) \
    +@@ -339,7 +337,8 @@
    + @echo 'Description: Xmlrpc-c basic XML-RPC library' >>$@
    + @echo 'Version: $(XMLRPC_VERSION_STRING)' >>$@
    + @echo >>$@
    +- @echo 'Requires: xmlrpc_util $(XML_PKGCONFIG_REQ)' >>$@
    ++ @echo 'Requires: xmlrpc_util' >>$@
    ++ @echo 'Requires.private: $(XML_PKGCONFIG_REQ)' >>$@
    + @echo 'Libs: -L$${libdir} -lxmlrpc' >>$@
    + @echo 'Cflags: -I$${includedir}' >>$@
    +

    I must admit I don't oversee all the consequences here, so I hope that one
    of the other RT members can double check, but I think you should upload to unstable already.

    I would be more comfortable with the dependency change only. Changing
    pkgconfig files at this stage of the freeze is IMHO too late. The
    potential overlinking is handled by -Wl,--as-needed anyway.

    Cheers
    --
    Sebastian Ramacher

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Paul Gevers@21:1/5 to Alexandre Detiste on Sun May 18 08:30:01 2025
    XPost: linux.debian.devel.release
    To: 1105769@bugs.debian.org
    To: gjover@sipwise.com (Guillem Jover)

    This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --------------E7nP7yzUNxUhHuArcFYK0nzl
    Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: base64

    SGkgQWxleGFuZHJlLA0KDQpPbiAxNi0wNS0yMDI1IDEzOjAzLCBBbGV4YW5kcmUgRGV0aXN0 ZSB3cm90ZToNCj4+IFRoYW5rcyBmb3IgdGVzdGluZyB0aGF0LiBEaWQgeW91IGRvIGFueSBj aGVja2luZyB0aGF0IHRoZSBidWlsZCB3YXMgYWxzbw0KPj4gY29ycmVjdD8gSWYgc28sIHdo YXQgZGlkIHlvdSBjaGVjaz8NCj4gDQo+IEkgY2hlY2tlZCB0aGUgZGlmZiBmcm9tIDEuNTku MDMtOSBpbiBUZXN0aW5nIHdpdGggZGViZGlmZiAmIGRpZmZvc2NvcGUuDQoNCg0KQWNrLg0K DQo+IEZvciBzb21lICRyZWFzb24gcmVwb3J0YnVnIGRvZXMgbm90IGF1dG8tc3Vic2NyaWJl IG1lIHRvIHRoaXMgUlQgYnVnIC4uLg0KDQpJJ20gbm90IGF3YXJlIHRoYXQgcmVwb3J0YnVn IGNhbiBkbyB0aGF0LiBUaGUgYnRzIGZvciBzdXJlIGRvZXNuJ3QgZG8gDQp0aGF0LiBZb3Ug YWx3YXlzIG5lZWQgdG8gYXNrIGV4cGxpY2l0bHkgKHdoaWNoIGhhcyBiZWVuIHRvcGljIGZv ciBsb3RzIA0Kb2YgZGlzY3Vzc2lvbnMpLg0KDQo+IEknbSBvayB0byByZXZlcnQgaGFsZiBv ZiB0aGUgY2hhbmdlcyBhbmQgb25seSBrZWVwaW5nIHRoZSAxICBsaW5lDQo+IGNoYW5nZSBp biBkL2NvbnRyb2wuDQoNCg0KWWVzLCBsZXQncyBkbyB0aGF0Lg0KDQpQYXVsDQoNCg==

    --------------E7nP7yzUNxUhHuArcFYK0nzl--

    -----BEGIN PGP SIGNATURE-----

    wsC7BAABCABvBYJoKXyZCRCcXJnrBb11CkcUAAAAAAAeACBzYWx0QG5vdGF0aW9u cy5zZXF1b2lhLXBncC5vcmcaaKcfwDA9gcQL2Bd6ezyV7WJpYnuVq2Z2p1ceOuV9 ZxYhBFi2bUhza+k7BS3mcpxcmesFvXUKAAAVzgf/WsiIRH6UW/pHQeOgHfBcT3kY I5PPGMpUuwJcOOlZhbwA5e5yyEvZs6mKsUrMJtWO5edOtUt75KuK/CEJrDdu/1kd ORGP4Xu5fU89YcOz6unbbIK3eH7+Wxf6qwmJgyFgPWQiqtDoohkbifVmDG9NBBkQ xuYJja3JSLquYAVHRPeNvGM2CMKb8vyImMf/83e62aRlRjiGssEv9Zd4/Ynylbh+ NL4Sp4okw4pTk52tdg9KSlEMVesl59WdrTwIJ2Y2tgewB5R5O+wUi8XguZYXbT0J ijqK1ksmY1GyWBTUtwHtmqJqZhZlJ49aBbpUTXp0J/E2/qNAj60nRgpgTmLVuA==
    =nNvs
    -----END PGP SIGNATURE-----

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)