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

flint-1.5.2.p0.spkg installation fails on Cygwin #13330

Closed
jpflori opened this issue Aug 3, 2012 · 8 comments
Closed

flint-1.5.2.p0.spkg installation fails on Cygwin #13330

jpflori opened this issue Aug 3, 2012 · 8 comments

Comments

@jpflori
Copy link

jpflori commented Aug 3, 2012

The FLINT 1.5.2 source is a mess with regard to line terminators.

Because of that patch gets confused on Cygwin.
And I found no way to produce patches that apply both on CRLF and LF line terminators systems (i.e. Cygwin and Ubuntu).
Using --binary flag on Cygwin solves the problem.

Here is an updated spkg fixing that while we wait for FLINT2 of ticket #12173:
http://boxen.math.washington.edu/home/jdemeyer/spkg/flint-1.5.2.p1.spkg

Component: porting: Cygwin

Keywords: cygwin flint spkg

Author: Jean-Pierre Flori

Reviewer: Dmitrii Pasechnik

Merged: sage-5.3.beta1

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

@jpflori jpflori added this to the sage-5.3 milestone Aug 3, 2012
@jpflori

This comment has been minimized.

@jpflori
Copy link
Author

jpflori commented Aug 3, 2012

comment:1

The spkg is posted at:
http://perso.telecom-paristech.fr/~flori/sage/flint-1.5.2.p1.spkg

@jpflori
Copy link
Author

jpflori commented Aug 3, 2012

Changed work issues from upload spkg to none

@jpflori
Copy link
Author

jpflori commented Aug 3, 2012

Spkg diff, for review only.

@dimpase
Copy link
Member

dimpase commented Aug 3, 2012

comment:2

Attachment: spkg.diff.gz

Right; we fought with flint people (w00t? they develop, like, on Windows? :–)) for a while regarding these broken line endings, that confuse the hell out of Cygwin.
This way it works. Positive review.

@jdemeyer
Copy link

jdemeyer commented Aug 4, 2012

Reviewer: Dmitrii Pasechnik

@jdemeyer
Copy link

Merged: sage-5.3.beta1

@jdemeyer

This comment has been minimized.

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