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.

Sign up for GitHub

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

Jump to bottom

Several modules in sagelib don't link on Cygwin #22323

Closed
embray opened this issue Feb 7, 2017 · 9 comments
Closed

Several modules in sagelib don't link on Cygwin #22323

embray opened this issue Feb 7, 2017 · 9 comments
Labels
c: porting: cygwin p: major / 3 t: bug
Milestone
sage-7.6

Comments

@embray
Copy link
Contributor

embray commented Feb 7, 2017

There are a few Cython modules in sage that are not compiling on Cygwin due to missing libraries at link time. Rather than provide an exhaustive list here, I will just provide a patch that fixes all the current cases I've found so far.

Component: porting: Cygwin

Keywords: windows cygwin sagelib

Author: Erik Bray

Branch/Commit: f8bb028

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/22323
@embray embray added this to the sage-7.6 milestone Feb 7, 2017
@embray
Copy link
Contributor Author

embray commented Feb 7, 2017

Branch: u/embray/cygwin/ticket-22323

@embray
Copy link
Contributor Author

embray commented Feb 7, 2017

Commit: f8bb028

@embray
Copy link
Contributor Author

embray commented Feb 7, 2017

Author: Erik Bray

@embray
Copy link
Contributor Author

embray commented Feb 7, 2017

comment:1

Patch.

New commits:

f8bb028Add some link flags needs to build some modules on Cygwin

@embray
Copy link
Contributor Author

embray commented Feb 7, 2017

Changed keywords from none to windows cygwin sagelib

@jdemeyer
Copy link

jdemeyer commented Feb 7, 2017

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link

jdemeyer commented Feb 7, 2017

comment:2

Makes sense.

@embray
Copy link
Contributor Author

embray commented Feb 9, 2017

comment:3

Thanks!

@vbraun
Copy link
Member

vbraun commented Feb 11, 2017

Changed branch from u/embray/cygwin/ticket-22323 to f8bb028

@vbraun vbraun closed this as completed in 0039d8a Feb 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: porting: cygwin p: major / 3 t: bug
Projects
None yet
Milestone
sage-7.6
Development

No branches or pull requests
3 participants
@embray @vbraun @jdemeyer