Remove bsdmainutils from doc/ci #30749

pull fanquake wants to merge 3 commits into bitcoin:master from fanquake:no_bsd_main_utils changing 3 files +3 −3
  1. fanquake commented at 9:24 am on August 29, 2024: member
    This was previously used to install hexdump (for the tests). However that isn’t used by CMake. I’m not aware of any other tools from this package being used.
  2. DrahtBot commented at 9:24 am on August 29, 2024: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK maflcko, hebasto

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  3. hebasto approved
  4. hebasto commented at 9:38 am on August 29, 2024: member

    This was previously used to install hexdump (for the tests).

    (via the bsdextrautils package, at least on Ubuntu 24.04)

    ACK c462276e6eea70efb0dd3a486f50fb32a7934500.

  5. ci: remove bsdmainutils 36ff336d2e
  6. doc: remove bsdmainutils a2b1d2c5ec
  7. maflcko commented at 10:02 am on August 29, 2024: member
     0-- Checking for module 'libzmq>=4'
     1--   Package 'libzmq', required by 'virtual:world', not found
     2CMake Error at /usr/local/share/cmake-3.30/Modules/FindPkgConfig.cmake:645 (message):
     3  The following required packages were not found:
     4
     5   - libzmq>=4
     6
     7Call Stack (most recent call first):
     8  /usr/local/share/cmake-3.30/Modules/FindPkgConfig.cmake:873 (_pkg_check_modules_internal)
     9  CMakeLists.txt:150 (pkg_check_modules)
    10
    11
    12-- Configuring incomplete, errors occurred!
    
  8. fanquake commented at 10:04 am on August 29, 2024: member
    Yea, see my comment here: https://github.com/bitcoin/bitcoin/pull/30740/files#r1735920877. I will just add the proper dependency.
  9. ci: add libzmq3-dev to test-each-commit job 78358ce09d
  10. fanquake force-pushed on Aug 29, 2024
  11. maflcko commented at 10:13 am on August 29, 2024: member
    ACK 78358ce09d2a62a469a544249e02e02f9f7b94cd
  12. DrahtBot requested review from hebasto on Aug 29, 2024
  13. maflcko added the label Bug on Aug 29, 2024
  14. maflcko removed the label Bug on Aug 29, 2024
  15. hebasto approved
  16. hebasto commented at 11:01 am on August 29, 2024: member
    ACK 78358ce09d2a62a469a544249e02e02f9f7b94cd.
  17. fanquake commented at 11:29 am on August 29, 2024: member
    Going to merge this now, to prevent any further unrelated CI failures in other PRs.
  18. fanquake merged this on Aug 29, 2024
  19. fanquake closed this on Aug 29, 2024

  20. fanquake deleted the branch on Aug 29, 2024
  21. Sjors commented at 11:44 am on August 29, 2024: member
    Just ran into this as well, thanks for the fix.

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-10-30 03:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me