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

opencdk ignoring SAGE64 except on OS X #7817

Closed
sagetrac-drkirkby mannequin opened this issue Jan 2, 2010 · 5 comments
Closed

opencdk ignoring SAGE64 except on OS X #7817

sagetrac-drkirkby mannequin opened this issue Jan 2, 2010 · 5 comments

Comments

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Jan 2, 2010

Like many packages, opencdk has code which adds -m64 on OS X if SAGE64 is set to yes. It is being ignored on other platforms, with the result the build fails - see below.

/home/drkirkby/sage-4.3/local/lib/libgcrypt.so /export/home/drkirkby/sage-4.3/local/lib/libgpg-error.so -lz -lc 
ld: fatal: file /export/home/drkirkby/sage-4.3/local/lib/libgcrypt.so: wrong ELF class: ELFCLASS64
ld: fatal: file processing errors. No output written to .libs/libopencdk.so.10.0.6
collect2: ld returned 1 exit status
make[4]: *** [libopencdk.la] Error 1
make[4]: Leaving directory `/export/home/drkirkby/sage-4.3/spkg/build/opencdk-0.6.6.p2/src/src'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/export/home/drkirkby/sage-4.3/spkg/build/opencdk-0.6.6.p2/src'
make[2]: *** [all] Error 2
make[2]: Leaving directory `/export/home/drkirkby/sage-4.3/spkg/build/opencdk-0.6.6.p2/src'
Failed to build OpenCDK

Component: porting

Author: David Kirkby

Reviewer: Jaap Spies

Merged: sage-4.3.1.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/7817

@sagetrac-drkirkby sagetrac-drkirkby mannequin added this to the sage-4.3.1 milestone Jan 2, 2010
@sagetrac-drkirkby sagetrac-drkirkby mannequin self-assigned this Jan 2, 2010
@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Jan 2, 2010

Author: David Kirkby

@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Jan 2, 2010

comment:1

I've updated spkg-install so SAGE64 is used on any platform.

Changes have been checked in.

http://boxen.math.washington.edu/home/kirkby/portability/opencdk-0.6.6.p3/

@jaapspies
Copy link
Member

comment:2

Looks good to me. Tested it on Open Solaris and Fedora 11 and 12.

So positive review.

Jaap

@jaapspies
Copy link
Member

Reviewer: Jaap Spies

@mwhansen
Copy link
Contributor

mwhansen commented Jan 4, 2010

Merged: sage-4.3.1.alpha0

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

No branches or pull requests

2 participants