Ticket #11070 (closed defect: fixed)
Update prereq so that SAGE_PORT does not need setting on Solaris x86
| Reported by: | drkirkby | Owned by: | drkirkby |
|---|---|---|---|
| Priority: | major | Milestone: | sage-4.7 |
| Component: | porting: Solaris | Keywords: | |
| Cc: | jhpalmieri | Work issues: | |
| Report Upstream: | N/A | Reviewers: | John Palmieri |
| Authors: | David Kirkby | Merged in: | sage-4.7.alpha4 |
| Dependencies: | Stopgaps: |
Description (last modified by drkirkby) (diff)
The environment variable SAGE_PORT
http://www.sagemath.org/doc/installation/source.html
has to be set on systems such as AIX and HP-UX where Sage is not supported. Currently this is also so on Solaris x86, despite the fact Sage build fine and is fully supported on both OpenSolaris x86 and Solaris 10 x86:
http://wiki.sagemath.org/SupportedPlatforms
As such, the file
$SAGE_ROOT/spkg/base/prereq-0.7-install
needs to be changed to remove this specific test for Solaris on x86. There is also a minor typo with an "x" missing on a line where a test for Solaris 2.6 is performed.
The new script will be installed as:
$SAGE_ROOT/spkg/base/prereq-0.8-install
and the old one removed.
Output on a Solaris x86 system
Here's the output shown with sage-4.7.alpha1 on a Solaris x86 system with the environment variable SAGE_PORT not set.
Starting prerequisite check. Machine: SunOS hawk 5.11 snv_134 i86pc i386 i86pc Building or using Sage on non-Sparc Solaris is tricky and not supported at the moment. It is possible, but you should be well aware that some things do not work. Support for Solaris on non-SPARC hardware is actively being worked on. More information can be found about Sage on Solaris on the Wiki at http://wiki.sagemath.org/solaris To get past this message, export the variable SAGE_PORT to something non-empty. make[1]: *** [installed/prereq-0.7] Error 1 make[1]: Leaving directory `/export/home/drkirkby/sage-4.7.alpha1/spkg' real 0m0.024s user 0m0.011s sys 0m0.017s Error building Sage. make: *** [build] Error 1
Here's the output of sage-4.7.alpha2, which has the changes applied. This time the prereq script gets run, without Sage exiting.
Starting prerequisite check. Machine: SunOS hawk 5.11 snv_134 i86pc i386 i86pc prereq-0.8/ prereq-0.8/install-sh prereq-0.8/aclocal.m4 prereq-0.8/Makefile.am <snip> checking for sqrt in -lm... yes checking for sqrtl in -lm... yes configure: creating ./config.status config.status: creating Makefile config.status: creating config.h config.status: config.h is unchanged config.status: executing depfiles commands
Notes for reviewer
You only need to check the changes between the two scripts - ignore the tar file that the script opens. That is the subject of another ticket #9978.
When testing, you will need to copy the file prereq-0.8.tar attached to #9978 to $SAGE_ROOT/spkg/base/.
Both the script and the tar file must have the same version number (0.8).
Notes for the release manager
This ticket depends on #9978. This and #9978 must be updated together. Both have positive review.
- The tar file prereq-0.8.tar, attached to #9978 should be installed as $SAGE_ROOT/spkg/base/prereq-0.8.tar. It is listed in .hgignore,so there's no need to change the repository for this.
- The tar file $SAGE_ROOT/spkg/base/prereq-0.7.tar should be removed.
- Apply trac_11070-base-repo.patch to the repository in spkg/base.
Attachments
Change History
comment:1 Changed 2 years ago by drkirkby
- Cc jhpalmieri added
- Status changed from new to needs_review
Changed 2 years ago by drkirkby
-
attachment
prereq-0.8-install
added
To be installed as $SAGE_ROOT/spkg/base/prereq-0.8-install
Changed 2 years ago by drkirkby
-
attachment
prereq-install-changes.diff
added
Differences between the old script prereq-0.7-install and the new one prereq-0.8-install
comment:6 Changed 2 years ago by jhpalmieri
Hi Dave,
In principle this looks good. One very minor change to the patch: add a period at the end of
echo "Sage is not supported on any version of Solaris earlier than 10"
(You're deleting the following line, which has the period for that sentence.)
More importantly, though, I think that the file prereq-0.8-install should be added to the spkg/base repo, it should be executable, and prereq-0.7-install should be removed from the repo. (I guess actually you should use hg to rename the 0.7 version to the 0.8 version, and then patch it.) The .hgignore file should be modified, too, to include the new tar file, and I suppose it doesn't need to mention the old tar file.
I'm attaching a mercurial patch which does these things, and I've committed it in your name. Let me know if you have any objections. Please take a look at the patch, and if you're happy with it, give the ticket a positive review. (I'm happy with your changes, I just want to make sure you're happy with my implementation of them.)
Changed 2 years ago by jhpalmieri
-
attachment
trac_11070-base-repo.patch
added
apply to spkg/base repo
comment:8 Changed 2 years ago by drkirkby
- Status changed from needs_review to positive_review
- Reviewers set to John Palmieri
Thank you John.
Yes, your changes look fine to me. I was not aware exactly how to use Mercurial to rename the files. But I applied the Mercurial patch you created and it did the right things.
I still need to get a positive review on the other half if this update of "prereq", which is on ticket #9978. If you have time, perhaps you could take a look at that.
comment:10 Changed 2 years ago by jdemeyer
- Status changed from positive_review to closed
- Resolution set to fixed
- Merged in set to sage-4.7.alpha4
