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
Allow NTL to build on Solaris with Sun or GNU linker #6380
Comments
comment:1
Positive review, though I didn't test it with the Sun linker, it looks very sensible and doesn't break things. An spkg with the changes checked into the repo and a typo fixed is now here: http://sage.math.washington.edu/home/wstein/patches/ntl-5.4.2.p8.spkg |
comment:2
Unfortunately, another ticket already got "p8", and the changes here need to be reapplied to that spkg. |
comment:3
Replying to @rlmill:
That spkg is available here: http://sage.math.washington.edu/home/rlmill/ntl-5.4.2.p8.spkg |
Reviewer: was |
comment:4
OK, I've made the changes. Please see: http://sage.math.washington.edu/home/kirkby/Solaris-fixes/ntl-5.4.2.p9/ (different directory to last one) I created a patchfile which shows the differences from the last copy of 'mfile' otherwise the patch would be huge and uncomprehsible, as mfile has been changed so many times. There is actually a patch in the 'patches' directory. I just noticed I probably put the original and new files in the wrong order, as it looks like I've added stuff to the 'mfile' not taken it away. I have in fact just removed -Wl,-soname,libcat DIRNAME.so Formally the line was: now it is |
comment:6
Replying to @sagetrac-drkirkby:
The NTL spkg above contains some junk and changes were not checked in. I've checked in changes in David Kirkby's name. The updated spkg is available at http://sage.math.washington.edu/home/mvngu/patch/ntl-5.4.2.p9.spkg |
Changed reviewer from was to William Stein |
comment:7
After installing the NTL package at http://sage.math.washington.edu/home/mvngu/patch/ntl-5.4.2.p9.spkg and running doctests on all of the Sage library, I got this:
The funny thing is that it reports doctest failures, with "0 doctests failed". I reinstalled |
Author: David Kirkby |
comment:8
Just to let people know, this has been merged in sage-4.1.1-alpha0. I can't close this ticket because I don't have the privilege to do so. Sorry, folks :-( |
Merged: sage-4.1.1.alpha0 |
Although the ntl-5.4.2.p7 package would build on Solaris 10 with gcc 4.4.0 if the gcc was configured to use the GNU linker from binutils, the package would not build with the gcc if the compiler was configured to use the Sun linker, with the following options:
--with-ld=/usr/ccs/bin/ld --without-gnu-ld
The part of the makefile executed when building a shared library would fail if the Sun linker was used. The makefile specified the same output filename twice, but in a way the Sun linker would not tolerate.
This patch simply removes a very small bit of code (just "-Wl,-soname,lib
cat DIRNAME
.so "), which allows NTL to build properly, irrespective of the linker that is being used.Please see
http://sage.math.washington.edu/home/kirkby/Solaris-fixes/ntl/
Component: porting: Solaris
Keywords: solaris ntl makefile GNUism
Author: David Kirkby
Reviewer: William Stein
Merged: sage-4.1.1.alpha0
Issue created by migration from https://trac.sagemath.org/ticket/6380
The text was updated successfully, but these errors were encountered: