Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GAP.jl broken? #291

Closed
thofma opened this issue Oct 2, 2019 · 4 comments · Fixed by #293
Closed

GAP.jl broken? #291

thofma opened this issue Oct 2, 2019 · 4 comments · Fixed by #293
Assignees

Comments

@thofma
Copy link
Contributor

thofma commented Oct 2, 2019

Fresh julia installation.

julia> versioninfo()
Julia Version 1.2.0
Commit c6da87ff4b (2019-08-20 00:03 UTC)
Platform Info:
  OS: Linux (x86_64-pc-linux-gnu)
  CPU: Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.1 (ORCJIT, skylake)
Environment:
  JULIA_DEPOT_PATH = /tmp/new

(v1.2) pkg> add GAP
  Updating registry at `/tmp/new/registries/General`
  Updating git-repo `https://github.com/JuliaRegistries/General.git`
 Resolving package versions...
 Installed GAPTypes ─ v1.0.0
 Installed GAP ────── v0.2.1
  Updating `/tmp/new/environments/v1.2/Project.toml`
  [c863536a] + GAP v0.2.1
  Updating `/tmp/new/environments/v1.2/Manifest.toml`
  [c863536a] + GAP v0.2.1
  [cd4130c4] + GAPTypes v1.0.0
  Building GAP  `/tmp/new/packages/GAP/JMFDD/deps/build.log`

julia> using GAP
[ Info: Precompiling GAP [c863536a-3901-11e9-33e7-d5cd0df7b904]
Adding path /tmp/new/packages/GAP/JMFDD/gap/.libs to DL_LOAD_PATH
Panic in src/julia_gc.c:756: GAPTypes.jl is not available

Fails also with the same error on julia 1.1. I am pretty sure this used to work.

@ThomasBreuer
Copy link
Member

@thofma The problem arises because of a bug in gap-system/gap#3669, in the GAP master branch.

@thofma
Copy link
Contributor Author

thofma commented Oct 2, 2019

Thanks. Any way to work around this? Is the GAP version in GAP.jl fixed or does it pull in the latest master branch of GAP?

@fingolfin
Copy link
Member

I'll take care of this shortly. Thomas made me aware if this before, but I didn't get to it yesterday

@fingolfin
Copy link
Member

The problem is a missing change on the GAP.jl side, plus a lack of proper integration testing in the GAP CI tests, which do have some minimal tests for the Julia integration, but do not test against GAP.jl yet. I'll look into that, too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants