#11550: Singular does not build on Cygwin on Windows 7
-----------------------------------+----------------------------------------
Reporter: kcrisman | Owner: kcrisman
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: cygwin | Keywords: cygwin singular
Work_issues: | Upstream: N/A
Reviewer: Karl-Dieter Crisman | Author: Mike Hansen, Karl-Dieter
Crisman
Merged: | Dependencies:
-----------------------------------+----------------------------------------
Changes (by kcrisman):
* status: needs_info => needs_review
Comment:
I just realized this whole discussion about variables is a red herring.
In the spkg-install, we have
{{{
if [ "$UNAME" = "CYGWIN" ]; then
# Fine to make this patch on any system, because it is code that
is onl$
cp patches/sing_win.cc src/Singular/
cp patches/IntegerProgramming-Makefile.in
src/IntegerProgramming/Makefi$
cp patches/Minor.h src/Singular/
fi
}}}
which includes our patch.
Compare the relevant part of the diff:
{{{
#!diff
diff -r 0825382212aa spkg-install
a b
114 114 # Fine to make this patch on any system, because
it is code that is only compiled on Windows.
115 115 cp patches/sing_win.cc src/Singular/
116 116 cp patches/IntegerProgramming-Makefile.in
src/IntegerProgramming/Makefile.in
117 cp patches/Minor.h src/Singular/
117 118 fi
118 119
119 120 # parallel build patches
}}}
so I see why it was confusing! But even with the comment (legacy comment,
not mine), this is still all just included on Cygwin.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11550#comment:13>
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.
