Opened 4 years ago
Closed 4 years ago
#21474 closed defect (fixed)
Patch Boehm-Weiser gc so that ecl builds on 32-bit Linux Docker images
Reported by: | mkoeppe | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | sage-7.4 |
Component: | packages: standard | Keywords: | |
Cc: | dimpase, jdemeyer, embray, fbissey | Merged in: | |
Authors: | Matthias Koeppe | Reviewers: | François Bissey |
Report Upstream: | Fixed upstream, but not in a stable release. | Work issues: | |
Branch: | bf44c21 (Commits, GitHub, GitLab) | Commit: | bf44c21a8fdea0e4ca5c84681dec11a9c68df5cf |
Dependencies: | Stopgaps: |
Description (last modified by )
I am building Sage on a 32-bit Linux Docker (https://github.com/mkoeppe/docker-images/tree/master/sagemath-buildenv-32bit).
Building ecl fails with this error: getcontext failed: Use another register retrieval method?
This is solved with a patch that is a backport from the patch mentioned here https://github.com/ivmai/bdwgc/issues/133 (branch here: https://github.com/mkoeppe/bdwgc/tree/gc7_2f_sage)
Change History (7)
comment:1 Changed 4 years ago by
- Branch set to u/mkoeppe/patch_boehm_weiser_gc_so_that_ecl_builds_on_32_bit_linux_docker_images
comment:2 Changed 4 years ago by
- Cc dimpase jdemeyer added
- Commit set to bf44c21a8fdea0e4ca5c84681dec11a9c68df5cf
- Description modified (diff)
- Status changed from new to needs_review
comment:3 Changed 4 years ago by
- Cc embray fbissey added
comment:4 Changed 4 years ago by
So the fix is in master but not in any release yet?
comment:5 Changed 4 years ago by
Yes, as far as I can see.
comment:6 Changed 4 years ago by
- Report Upstream changed from N/A to Fixed upstream, but not in a stable release.
- Reviewers set to François Bissey
- Status changed from needs_review to positive_review
I have no issue with adding a patch approved upstream. So long as it doesn't cause anything else to break we are all good as far as I concerned.
comment:7 Changed 4 years ago by
- Branch changed from u/mkoeppe/patch_boehm_weiser_gc_so_that_ecl_builds_on_32_bit_linux_docker_images to bf44c21a8fdea0e4ca5c84681dec11a9c68df5cf
- Resolution set to fixed
- Status changed from positive_review to closed
New commits:
#21474: Patch Boehm-Weiser gc so that ecl builds on 32-bit Linux Docker images