• Bug#1104515: blends fails to reproduce due to emtpy apt cache (future F

    From Jochen Sprickerhof@21:1/5 to All on Thu May 1 14:40:02 2025
    This is a multi-part MIME message sent by reportbug.


    Source: blends
    Version: 0.7.10
    Severity: normal
    Tags: ftbfs patch
    User: debian-qa@lists.debian.org
    Usertags: apt-cache

    Hi,

    blends uses the apt cache during testing. This breaks on reproduce.d.n
    as it has an empty apt cache:

    https://reproduce.debian.net/all/#blends-common

    The attached patch avoids running the test in that case.

    In addition the sbuild team plans to clean the apt cache before building
    the package by default in forky which would make this package FTBFS.
    This would also be fixed with the patch.

    Cheers Jochen

    diff --git a/debian/rules b/debian/rules
    index 478475a..99c34ec 100755
    --- a/debian/rules
    +++ b/debian/rules
    @@ -17,10 +17,12 @@ override_dh_auto_build:

    override_dh_auto_test:
    ifeq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
    +ifneq (,$(wildcard /var/lib/apt/lists/*Packages))
    pytest-3 --doctest-modules
    rm -f education # clean up test remnants
    find . -type d -name __pycache__ | xargs rm -rf
    endif
    +endif

    override_dh_installchangelogs:
    for pkgnews in $(DEB_SOURCE)-common $(DEB_SOURCE)-dev ; do \

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Jochen Sprickerhof@21:1/5 to All on Sun Jun 1 18:10:02 2025
    Hi,

    I plan to clean the apt cache by default for forky and implement that
    already in the trixie sbuild. This would make blends FTBFS unless the
    proposed patch (or something similar) is applied in forky. Please speak
    up if there is any problem with this plan. You can test the new
    behaviour with:

    sbuild --starting-build-commands='apt-get distclean' -d unstable blends

    Cheers Jochen

    * Jochen Sprickerhof <jspricke@debian.org> [2025-05-01 14:37]:
    Source: blends
    Version: 0.7.10
    Severity: normal
    Tags: ftbfs patch
    User: debian-qa@lists.debian.org
    Usertags: apt-cache

    Hi,

    blends uses the apt cache during testing. This breaks on reproduce.d.n
    as it has an empty apt cache:

    https://reproduce.debian.net/all/#blends-common

    The attached patch avoids running the test in that case.

    In addition the sbuild team plans to clean the apt cache before building
    the package by default in forky which would make this package FTBFS.
    This would also be fixed with the patch.

    Cheers Jochen

    diff --git a/debian/rules b/debian/rules
    index 478475a..99c34ec 100755
    --- a/debian/rules
    +++ b/debian/rules
    @@ -17,10 +17,12 @@ override_dh_auto_build:

    override_dh_auto_test:
    ifeq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
    +ifneq (,$(wildcard /var/lib/apt/lists/*Packages))
    pytest-3 --doctest-modules
    rm -f education # clean up test remnants
    find . -type d -name __pycache__ | xargs rm -rf
    endif
    +endif

    override_dh_installchangelogs:
    for pkgnews in $(DEB_SOURCE)-common $(DEB_SOURCE)-dev ; do \


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

    iQIzBAABCgAdFiEEc7KZy9TurdzAF+h6W//cwljmlDMFAmg8evUACgkQW//cwljm lDPKAw//bQiivmz5xXACYXGVNOCl5KhdVaYjETnEgiUUJywpXR+ghJqtWTEX/MK6 XTG266Ov193+2ETAryRjsFF2T9YTbD4pRyb6Oqs/tOUxDzZ+zVJQnio06nqlGRcK nfFCQ4jVXLcJdIuQcds+WzWe2lGc20JT8G6mPD/0GsApOA+tgXjtXEKwbVY1q72P btP2/u52AvjJ1qD+Of90gWK4fVlcdORBIYCzIFUEvTAjmzHE6gw0PjohYw16hLXD uOTdsZwSCwcQPPNyUg5yGHz/u0464F0P/mLF5VAT2G1PPNGGoIle+wSpscmhJhzG CSI4+QIGNCE2XbEF+Rlf+1uOMzecjOU54P2/8H51vUlAzvBWs6RF1DUB/lLPrNBe An/W8eoBMyJOm+1ZjUcuWHSgCSJN/xTUOCZRSq69ZuNRRS1C6N/miYs3cTibGMrb EuOPjR8ooI4UNi/bBF1cgYBVAjZpTKV58E2l8NnTv/gWSsJpHkKp+hoLDOKk9W06 4ieYSI/COG3L46L0NfguB5ViJLkoteLQrqeVtYbol4UL90SBuhZQTaxfsgdSIKZx 5pNjQi4z6tJ/FhjWuDXI3xLtDQ8xvxM6c7Y/grHjCf7OiAX901Otpd0sAfIQ3uZ2 iLPhUa1h8jltBnkQWlsX2jxL9vh/kq4XfFcjB6xJkis/ItnIB5c=
    =ENwQ
    -----END PGP SIGNATURE-----

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Jochen Sprickerhof@21:1/5 to All on Mon Jun 2 10:40:01 2025
    Hi Ole,

    * Ole Streicher <olebole@debian.org> [2025-06-02 10:26]:
    The provided patch just disables the build time tests for the package,
    which seems not a good solution to me.

    The problem is that the test explicitly opens the apt cache and I did
    not find a way to mock that in a doctest. We could convert the tests to
    unit tests and use mock then.

    Also changing Trixie sbuild at
    this stage seems a bit contradicting to the goal of the current Trixie
    hard freeze, not?

    That's up to the release team in #1107108 but I think there is still
    time to do it.

    I'd propose to keep this as-is for Trixie.

    "it" as in blends? Yes, totally. This is all work for forky.

    After its release, couldn't
    we just add a "apt update" to re-fill the cache before running the
    tests? Or does this already count as "network access"?

    That's network access.

    Cheers Jochen

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

    iQIzBAABCgAdFiEEc7KZy9TurdzAF+h6W//cwljmlDMFAmg9YncACgkQW//cwljm lDNILA/+O/c5GHnVUKShuMwJhxFBncI0gohVGZZjlbNB67/iYVX953u3QnxNlNrJ B5SVHJptaUxj35kKRhxH2lLBKCPVRmPvV+CQhJwWx19ioUIyx54qaE1c715aMNzV hEHdYkApFCuhdzqGoWapJFfaSsxGUaxw/H+4GpVdBbv7IRY8BqGnJ0o8KcO19RlZ iqgw+If55m/skPsyA015Dzffr1F4+Ge0PjKT9Nf8WtC6JkmYUzpiohCm+e/285Sp iSKTvKfP2/GFOpcaDfiKC0mnskh4RrikFxCDSa9Eh3SBKJm391U6z0UfZXo+GiVM y+w+lrSTbIVV+jLDiZdFWgU34RlxnSDSkhApkKPFCJfjQUQDhCPexvV5tzk4u0fJ aF+DuNjFmbUg6TzVdXLY8nez371owv0JQteArJqrpTOQpJQ9ZEUVWhADLkVehWY/ gV2k2lPSHxNeknmi5afDXLUtLDOWuq7nAy3YsDOfihjb9Npenq6L4y821wzO0RFE aFAONa8S8qyy3+bmUF+wyAzbyn6fmfWSb5PiU0qLnAdMkK9Vfg31GdTHE+apQGT5 rBB0RM+kQM7/j1BfPXGRu7hJBilYz9MEY6RZr9v9pB9OERDFDvpE4OMsC+jXsxby 3ZVouopPgv1+mawio4eQDo6n924EUomfSKKtHhxo2SX9ajRk86A=
    =tbtF
    -----END PGP SIGNATURE-----

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Jochen Sprickerhof@21:1/5 to All on Tue Jun 3 14:00:01 2025
    * Jochen Sprickerhof <jspricke@debian.org> [2025-06-02 10:36]:
    The problem is that the test explicitly opens the apt cache and I did
    not find a way to mock that in a doctest. We could convert the tests
    to unit tests and use mock then.

    An other option would be to run the tests as autopkgtest and add the needs-internet restriction.

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

    iQIzBAABCgAdFiEEc7KZy9TurdzAF+h6W//cwljmlDMFAmg+4fIACgkQW//cwljm lDN7kw//f/07KPHColsi7lcAQrOHYimXVl70COXWLn0LlgWjJCrjeEBuGtzrMDBv QpurZzjO0sZOyLwmdgO/32gCdtQYqhR9x7/ZWhABkH4fpW8k8zM+kdjW0SmyjMxg TFwHBAzAT4Xqum13I711KjqgFNReYNUM7mslPCTUdx+EigUYSFlV3aR2RYxHFQvg vrl/rvZS16alhKmaOjwYy/WR1qLYVdX5IPDPsr6pBGzEfnQTN9RlXERjFiOWnSdG yIU5fjbeavKdXx/9+B49ZKVC/bBuIA0E6IaRSU5+s26/aKQaPliyAAaVCw41m4Dg Sj/8qQLBVonjE2saA/ocmT4Jq3AGNo4Hds15xlm5sT3zmrPes3JZpudWkszHkqdB xp76Dulv6X1EM3iQgCN2jOAIZvMO7Rnx6wj2sAmW5NlbS4e0kc/JyE/rn8+Me23Z tfEXsCks8+PRgA2CEYo6DRGBz4gCbSYPolwhvCgSk1AjKDaGNtOZKIbEKFh7RBOk mp//1MK9+hcawmLyY71BYwnHyh0WkXTVEHZVPwA8UGPxVjNIYa/AOXeXoOUvRtRS YCKwylfSce3b9WymcSMoY+JaPnI9rxHfoPAEtTMqxA6PG6yaIU18qn59ueC7QsnS eUH8Lw0XRpuVr3t6mGdKPEHXtN+D0TyvOhsbvdI9uCEYqHMxaSA=
    =idCw
    -----END PGP SIGNATURE-----

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