ci: use Debian Bullseye in ARM CI #22994

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:use_bullseye_arm_ci changing 2 files +3 −3
  1. fanquake commented at 8:03 am on September 16, 2021: member

    This works around an issue when trying to use std::filesystem::remove_all with the ARM GCC on Buster. Has been split out of #20744.

    See commentary starting here: #20744 (comment). Also: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93201.

  2. ci: use Debian Bullseye in ARM CI
    This works around an issue when trying to use `std::filesystem::remove_all`
    with the ARM GCC on Buster. Has been split out of #20744.
    
    See comments starting here:
    https://github.com/bitcoin/bitcoin/pull/20744#issuecomment-810279549.
    Also: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93201.
    252d1a70fb
  3. fanquake added the label Tests on Sep 16, 2021
  4. MarcoFalke commented at 8:11 am on September 16, 2021: member
    We currently use gcc 8.4 for the arm guix builds. Should this be bumped to gcc 10.3?
  5. fanquake commented at 8:22 am on September 16, 2021: member

    We currently use gcc 8.4 for the arm guix builds. Should this be bumped to gcc 10.3?

    To match the base GCC in Buster it’d be 10.2, however if we were doing this, I don’t think it should be part of this PR. Maybe another PR separate from #20744 that takes care of the more involved build system changes. There is at least some Guix changes required for the Windows build as well.

  6. MarcoFalke commented at 8:24 am on September 16, 2021: member
    Nvm. Looks like this has been backported to gcc 8 after all: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81091#c9
  7. MarcoFalke commented at 8:25 am on September 16, 2021: member

    cr ACK 252d1a70fb452893efe4ab64298139eb08d8ac98

    Buster is on gcc 8.3, and it doesn’t look like they are going to bump to 8.4 any time soon, so moving to the next debian release seems ok.

  8. fanquake commented at 8:30 am on September 16, 2021: member

    Buster is on gcc 8.3, and it doesn’t look like they are going to bump to 8.4 any time soon, so moving to the next debian release seems ok.

    Yes, GCC-8 on Buster was bumped from 8.2 to 8.3 in Feb 2019, and received it’s last update in April 2019.

  9. hebasto approved
  10. hebasto commented at 8:50 am on September 16, 2021: member
    ACK 252d1a70fb452893efe4ab64298139eb08d8ac98, I have reviewed the code and it looks OK, I agree it can be merged.
  11. MarcoFalke referenced this in commit 87780dfee8 on Sep 16, 2021
  12. MarcoFalke added the label DrahtBot Guix build requested on Sep 16, 2021
  13. MarcoFalke removed the label DrahtBot Guix build requested on Sep 16, 2021
  14. MarcoFalke added this to the milestone Future on Sep 16, 2021
  15. MarcoFalke removed this from the milestone Future on Sep 16, 2021
  16. MarcoFalke commented at 9:07 am on September 16, 2021: member
    This was merged, but not marked as merged by GitHub.
  17. fanquake closed this on Sep 16, 2021

  18. fanquake deleted the branch on Sep 16, 2021
  19. fanquake locked this on Sep 18, 2021

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-09-29 01:12 UTC

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