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:

Status badges

Description (last modified by mkoeppe)

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 mkoeppe

  • 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 mkoeppe

  • Cc dimpase jdemeyer added
  • Commit set to bf44c21a8fdea0e4ca5c84681dec11a9c68df5cf
  • Description modified (diff)
  • Status changed from new to needs_review

New commits:

​bf44c21#21474: Patch Boehm-Weiser gc so that ecl builds on 32-bit Linux Docker images

comment:3 Changed 4 years ago by mkoeppe

  • Cc embray fbissey added

comment:4 Changed 4 years ago by fbissey

So the fix is in master but not in any release yet?

comment:5 Changed 4 years ago by mkoeppe

Yes, as far as I can see.

comment:6 Changed 4 years ago by fbissey

  • 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 vbraun

  • 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
Note: See TracTickets for help on using tickets.