Sage library fails to build on Cygwin

[kcrisman]

The file sage/rings/factorint.pyx needs to add libraries=['gmp'] as a dependency now.

CC: @mwhansen @dimpase @williamstein

Component: porting: Cygwin

Author: Karl-Dieter Crisman

Reviewer: Reg Burgess

Merged: sage-4.7.2.alpha2

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

[kcrisman]

Author: Karl-Dieter Crisman

[kcrisman]

comment:3

This apparently still isn't quite the right fix.

[kcrisman]

Attachment: trac_11499.patch.gz

[kcrisman]

comment:5

This needs review. Given the troubles building on Win7, checking that this doesn't break our 'normal' builds and one other person checking on XP should be sufficient to review.

[kcrisman]

comment:7

I've now checked this a number of times on Cygwin in the course of building Sage. It would be really nice for someone to make sure this doesn't break anything on 'normal' systems, and hopefully to check it on at least one Windows box - if only so that I don't have to manually fix it whenever I get to the middle of building the Sage spkg!

[kcrisman]

comment:8

User "RegB" was able to use this successfully for this - see this sage-windows thread.

So this should have positive review. Hopefully I can get a "real" name for this person.

[kcrisman]

comment:9

Reg confirmed this attribution by real name was ok in a private email. Welcome to the Sage community, Reg!

[kcrisman]

Reviewer: Reg Burgess

[jdemeyer]

Merged: sage-4.7.2.alpha2