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

Fix sublinking again #19663

Closed
sagetrac-gouezel mannequin opened this issue Dec 3, 2015 · 13 comments
Closed

Fix sublinking again #19663

sagetrac-gouezel mannequin opened this issue Dec 3, 2015 · 13 comments

Comments

@sagetrac-gouezel
Copy link
Mannequin

sagetrac-gouezel mannequin commented Dec 3, 2015

Sublinking in arb. Moreover, fplll links to gmp and mpfr, so the order of libraries in module_list.py should be changed accordingly.

CC: @jdemeyer

Component: porting: Cygwin

Author: Sebastien Gouezel

Branch/Commit: 0437c18

Reviewer: Travis Scrimshaw

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

@sagetrac-gouezel sagetrac-gouezel mannequin added this to the sage-6.10 milestone Dec 3, 2015
@tscrim
Copy link
Collaborator

tscrim commented Dec 6, 2015

comment:2

Needs rebasing over #19639.

@sagetrac-gouezel
Copy link
Mannequin Author

sagetrac-gouezel mannequin commented Dec 6, 2015

comment:3

Rebased on top of 6.10.beta7


New commits:

0437c18 #19663: fix sublinking in arb and libraries order

@sagetrac-gouezel
Copy link
Mannequin Author

sagetrac-gouezel mannequin commented Dec 6, 2015

Changed branch from u/gouezel/sublinking_20151202 to u/gouezel/sublinking_20151206

@sagetrac-gouezel
Copy link
Mannequin Author

sagetrac-gouezel mannequin commented Dec 6, 2015

Changed dependencies from #19627 to none

@sagetrac-gouezel
Copy link
Mannequin Author

sagetrac-gouezel mannequin commented Dec 6, 2015

Changed commit from 757210b to 0437c18

@tscrim
Copy link
Collaborator

tscrim commented Dec 6, 2015

Attachment: pynac-0.5.2.log

@tscrim

This comment has been minimized.

@tscrim
Copy link
Collaborator

tscrim commented Dec 6, 2015

comment:4

LGTM for arb. However, I think there is an issue with Pynac linking to GMP:

.libs/libpynac_la-numeric.o: In function `ZN5GiNaC7numericD2Ev':
/home/Travis/sage/local/var/tmp/sage/build/pynac-0.5.2/src/ginac/numeric.cpp:621:
 undefined reference to `_imp____gmpq_clear'

Full log has been attached.

@sagetrac-gouezel
Copy link
Mannequin Author

sagetrac-gouezel mannequin commented Dec 6, 2015

comment:5

OK, must be new from beta7. I will fix this once I am back in France, in one week.

@jdemeyer
Copy link

jdemeyer commented Dec 7, 2015

comment:7

You can open a new ticket for the pynac build failure.

@jdemeyer
Copy link

jdemeyer commented Dec 7, 2015

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Dec 7, 2015

comment:8

Done at #19678.

@vbraun
Copy link
Member

vbraun commented Dec 8, 2015

Changed branch from u/gouezel/sublinking_20151206 to 0437c18

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

3 participants