Changes between Version 354 and Version 419 of Ticket #22626


Ignore:
Timestamp:
12/19/18 16:32:57 (3 years ago)
Author:
embray
Comment:

I understand the issue with compiled packages now, and it is still broken, as nthiery reported originally at https://github.com/markuspf/gap/issues/1 (why on Markus's fork I'm not sure).

This is really more a problem with the build system for the individual compiled packages though, and I'm not sure it's something really within the scope of this ticket (unless there are some generic build package tools in GAP itself that we need to fix, which might be possible; I haven't checked). The shared libraries for compiled packages really need to be linked against libgap so that the loader can find GAP symbols from libgap.

I'm still slightly surprised by this and would appreciate a detailed explanation if someone can give one. I'd have assumed that if libgap.so had already been loaded, that the loader would still be able to resolve symbols from it when dlopen()-ing the GAP package. But it seems that in this case a DT_NEEDED entry is still needed for some subtle reason that I'm not quite getting.


New commits:

8404d4dinstead of disabling autoloading of default packages, just install all default packages as a standard part of the GAP SPKG
4d56e38more fixes to the GAP SPKG installation

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #22626

    • Property Commit changed from 15b7c7855d9097143e2b582b4831d2fb50294f53 to 4d56e389afc507efc047ebb9a6308b0372f06fe0
    • Property Work issues changed from fix libgap workspace loading, etc, and much more work... to ironing out the last kinks
  • Ticket #22626 – Description

    v354 v419  
    9090true
    9191}}}
    92 ~~This should be fixed once GAP's gap binary is built on top of libgap.
    93 See: https://github.com/markuspf/gap/issues/1~~ I believe this is fixed, but there are still some problems with the way this ticket is "installing" GAP for `$SAGE_LOCAL`.
     92This should be fixed once GAP's gap binary is built on top of libgap.
     93See: https://github.com/markuspf/gap/issues/1 I believe this is fixed, but there are still some problems with the way this ticket is "installing" GAP for `$SAGE_LOCAL`.
    9494
    9595Note: