#11727: Even more minor Cygwin fixes for FLINT
---------------------------------------------------+------------------------
Reporter: kcrisman | Owner: tbd
Type: defect | Status:
positive_review
Priority: major | Milestone:
sage-4.7.2
Component: cygwin | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Dima Pasechnik, Karl-Dieter Crisman | Author:
Karl-Dieter Crisman, Dima Pasechnik
Merged: | Dependencies:
---------------------------------------------------+------------------------
Old description:
> 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].
New description:
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].
--
Comment(by leif):
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).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11727#comment:15>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
