Opened 10 years ago

Closed 10 years ago

#13416 closed enhancement (fixed)

Fix GAP dependencies

Reported by: jdemeyer Owned by: GeorgSWeber
Priority: minor Milestone: sage-5.4
Component: build Keywords:
Cc: Merged in: sage-5.4.beta1
Authors: Jeroen Demeyer Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

According to spkg/standard/deps, GAP depends on SAGE. But there is nothing in the GAP sources which refers to Sage, gap_reset_worksheet() is a Sage libary function.

Attachments (1)

13416_gap_deps.patch (827 bytes) - added by jdemeyer 10 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 10 years ago by vbraun

GAP does use readline if available, and we do link against it:

(sage-sh) vbraun@volker-desktop:hg$ ldd $SAGE_LOCAL/gap/latest/bin/x86_64-unknown-linux-gnu-gcc-default64/gap =>  (0x00007fff78fff000) => /lib64/ (0x00000036c4800000) => /home/vbraun/opt/sage-5.3.rc1/local/lib/ (0x00007f2291759000) => /lib64/ (0x00000036da400000) => /lib64/ (0x00000036d5800000) => /lib64/ (0x00000036c5000000) => /lib64/ (0x00000036d6800000) => /lib64/ (0x00000036c4400000)
	/lib64/ (0x00000036c4000000)

We don't need the Sage library, at least gap-4.5.5 doesn't build the workspace cache as part of the spkg. Adding TERMCAP to the deps sounds good to me.

comment:2 Changed 10 years ago by jdemeyer

You must be using a different version of GAP:

jdemeyer@arcanis:/usr/local/src/sage-5.3.rc0$ ./sage --sh -c 'ldd $SAGE_LOCAL/gap/latest/bin/x86_64-unknown-linux-gnu-gcc-default64/gap'
ldd: /usr/local/src/sage-5.3.rc0/local/gap/latest/bin/x86_64-unknown-linux-gnu-gcc-default64/gap: No such file or directory


jdemeyer@arcanis:/usr/local/src/sage-5.3.rc0$ ./sage --sh -c 'ldd $SAGE_LOCAL/lib/gap-4.4.12/bin/x86_64-unknown-linux-gnu-gcc/gap' =>  (0x00007fffe05f5000) => /lib/ (0x00007f40c9311000) => /lib/ (0x00007f40c910d000) => /lib/ (0x00007f40c8db1000)
        /lib64/ (0x00007f40c9594000)

comment:3 Changed 10 years ago by jdemeyer

OK, I'm having a look at #13211 now...

comment:4 Changed 10 years ago by jdemeyer

GAP-4.5.5 links against readline indeed. So the person who wrote these dependencies must have had a crystal ball to predict that GAP would be using readline.

Changed 10 years ago by jdemeyer

comment:5 Changed 10 years ago by jdemeyer

  • Description modified (diff)
  • Status changed from new to needs_review

Kept READLINE dependency in anticipation of #13211, needs review.

comment:6 Changed 10 years ago by vbraun

  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

Looks good to me!

comment:7 Changed 10 years ago by jdemeyer

  • Merged in set to sage-5.4.beta1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.