#9160: Singular - change timestamp of file and sort out SAGE64 isssue.
------------------------------------------------------+---------------------
Reporter: drkirkby | Owner:
GeorgSWeber
Type: defect | Status: new
Priority: blocker | Milestone:
sage-4.4.4
Component: build | Keywords:
Author: David Kirkby and Willem Jan Palenstijn | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------------------------------------+---------------------
== The problems ==
There were three things wrong with the Singular package, the last of which
can cause build failures.
* The ability to build 64-bit was restricted to the OS X operating
system, as the -m64 flag was only added if both SAGE64 was set to "yes"
and the operating system was OS X with the following line of code.
* The package version was unconventional, with people sometimes updating
a date, rather than incrementing the patch level as is standard Sage
practice - see the developers guide at
http://www.sagemath.org/doc/developer/patching_spkgs.html#overview-of-
patching-spkg-s
* The time stamp of the file {{{src/Singular/libparse.cc}}} was one
second older than the file it was supposed to be created from (
{{{src/Singular/libparse.l}}}), so the build process was trying to use
'flex' to update {{{src/Singular/libparse.cc}}}, which fails if flex is
not available.
{{{
make install in Singular
make[4]: Entering directory
`/export/home/drkirkby/32/sage-4.4.3/spkg/build/singular-3-1-0-4-20100214/src/Singular'
sh flexer.sh -I -Pyylp -t libparse.l >libparse.cc.lmp
flexer.sh: flex: not found
flexer.sh: test: argument expected
make[4]: *** [libparse.cc] Error 1
}}}
See http://groups.google.co.uk/group/sage-
devel/browse_thread/thread/fbf5b7f781c3f523?hl=en for a discussion of
this.
== The solutions ==
Three very minor changes are made.
* The following
{{{
if [ `uname` = "Darwin" -a "$SAGE64" = "yes" ]; then
}}}
was changed to
{{{
if [ "x$SAGE64" = xyes ]; then
}}}
to enable 64-bit builds on any platform when SAGE64 is set to "yes".
* The time stamp of the file {{{src/Singular/libparse.cc}}} made the
current time with:
{{{
touch src/Singular/libparse.cc
}}}
* The package name was set in according with the Sage Developers Guide.
Had this been done properly, it would be patch level 6, so the package was
named singular-3.1.0.4.p6.
== Why a blocker ? ==
The incorrect time stamp has caused build failures on Solaris and could
cause them on other platforms too.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9160>
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.
