Even more minor Cygwin fixes for FLINT

[Karl-Dieter Crisman]

I'm not quite sure how this happened, but apparently in newer Cygwins patch won't accept certain things.

FLINT now doesn't actually work on one of my Cygwins, even after #11547, because of an extra line that crept into one of the patches.

--- src/mpn_extras.h        2009-09-23 18:03:27.000000000 +0800
+++ patches/mpn_extras.h        2011-04-25 22:36:33.000000000 +0800
@@ -22,7 +22,6 @@
 #ifndef MPN_EXTRAS_H
 #define MPN_EXTRAS_H
 
-#include "flint.h"
 #include "ZmodF_poly.h"
 
 #include "longlong_wrapper.h"

Note the newline at the end; removing this solves the problem. I guess this didn't cause trouble on other systems, since these patches are applied universally. I can't even figure out where this change happened.

See also #9858 and #10328 for other updates to FLINT needed.

---

Spkg at ​http://sage.math.washington.edu/home/kcrisman/flint-1.5.0.p9.spkg.

[Dima Pasechnik]

I suppose here hg did not see the file being opened in Unix mode, and saved in DOS mode. If I open it in vim I see

--- src/mpn_extras.h    2009-09-23 18:03:27.000000000 +0800
+++ patches/mpn_extras.h        2011-04-25 22:36:33.000000000 +0800
@@ -22,7 +22,6 @@
 #ifndef MPN_EXTRAS_H^M
 #define MPN_EXTRAS_H^M
 ^M
-#include "flint.h"^M
 #include "ZmodF_poly.h"^M
 ^M
 #include "longlong_wrapper.h"^M

these ^M are DOS LF/CR symbols. You can go over the spkg leading to this one, to track where exactly this got in. I can only say that in p6 spkg I created back in April it was still OK.

What you need to do is to save this file in Unix mode and re-package...

[Keshav Kini]

The CR characters were introduced to this file in p7, as you can see by doing hg log -p patches/mpn_extras.h.patch in the spkg directory.

[Dima Pasechnik]

Replying to kini:

The CR characters were introduced to this file in p7, as you can see by doing hg log -p patches/mpn_extras.h.patch in the spkg directory.

in fact hg log -p mpn_extras.h.patch shows this change, done by Jeroen, so this came in p7. I cc to him. He didn't notice, apparently. The whole src/ directory is in DOS mode there...

[Karl-Dieter Crisman]

Okay, see ​http://sage.math.washington.edu/home/kcrisman/flint-1.5.0.p9.spkg for a fix. Can someone very quickly just check that this has fixed it? (I can't see this in pico/nano.)

I think that if it works for me, and you review it, then that would be sufficient for positive review.

[Karl-Dieter Crisman]

This fixes the problem. I don't see any crud in the spkg, either.

cc:ing Jeroen as apparently he did it as well. I'd appreciate a quick review for folks, if possible :)

[Dima Pasechnik]

Replying to kcrisman:

Okay, see ​http://sage.math.washington.edu/home/kcrisman/flint-1.5.0.p9.spkg for a fix. Can someone very quickly just check that this has fixed it? (I can't see this in pico/nano.)

I think that if it works for me, and you review it, then that would be sufficient for positive review.

patches/mpn_extras.h.patch is in DOS mode. As well, some files in src/are in DOS mode, too. Please convert all the files in the spkg into Unix mode. This is in fact just on-liner (assuming dos2unix installed):

flint-1.5.0.p9$ find . -type f -exec dos2unix '{}' \;

New package, with this done, at ​http://boxen.math.washington.edu/home/dima/packages/flint-1.5.0.p9.spkg I checked in on Linux and MacOSX only - please check on Cygwin.

[Karl-Dieter Crisman]

Thanks. I have no idea what is going on with this - I only use Mac for developing (don't create patches/spkgs on Cygwin) so I am very unfamiliar with all this. My editors don't show these things.

(I'll try to check this on Cygwin soon - having some problems getting new builds started post 4.7.1, something might have changed in prereq or spkg/standard/deps.)

[Karl-Dieter Crisman]

Okay, your version did not change SPKG.txt (maybe mine didn't either?) so I did that, and then even added a tag for the p9 (thanks, ​Leif!) Still at ​http://sage.math.washington.edu/home/kcrisman/flint-1.5.0.p9.spkg.

[Dima Pasechnik]

Replying to kcrisman:

Thanks. I have no idea what is going on with this - I only use Mac for developing (don't create patches/spkgs on Cygwin) so I am very unfamiliar with all this. My editors don't show these things.

well, on every Mac there is vim, which you can run in terminal window - and it will show you the encoding...

[Karl-Dieter Crisman]

This still fixes the patch problem on Cygwin, builds ok. Positive review. Thanks, Dima, for catching the precise cause of this.

[Leif Leonhardy]

How's the storm? Did all machines survive?

---

If you can tell me what DOS file endings are... ;) (DOS actually used ^Z to indicate the end of a text file.)

Hopefully this will be the last spkg in the FLINT 1.5.0 series, so I can finally rebase my 1.5.2 one on it.

[Dima Pasechnik]

Replying to leif:

How's the storm? Did all machines survive?

---

If you can tell me what DOS file endings are... ;) (DOS actually used ^Z to indicate the end of a text file.)

yeah, "line endings" meant, indeed.

[Karl-Dieter Crisman]

How's the storm? Did all machines survive?

Turned out to be more of a non-event here, fortunately. Elsewhere on the US East Coast, not so much.

Hopefully this will be the last spkg in the FLINT 1.5.0 series, so I can finally rebase my 1.5.2 one on it.

Yes, let's hope!

[Leif Leonhardy]

Please don't put tabs into the ticket's description, as trac (at least our current version) doesn't properly handle these (i.e., doesn't escape them when exporting).