New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ilegal 'grep -o' causes problems installing optional packages on Solaris #8509
Comments
Attachment: sage-spkg.gz Revised sage-spkg removing the -o option |
comment:1
Thanks to John Palmieri who suggested this solution, which does work. |
Author: David Kirkby, John Palmieri |
comment:2
sage-spkg should be under revision control: either run Mercurial from the directory SAGE_ROOT/local/bin, or while running sage, use "hg_scripts" (instead of "hg_sage"). |
comment:3
Oops, my mistake. I was not aware of that. I'm attaching a Mercurial patch |
Attachment: sage-spkg.patch.gz Mercurial patch (replaces the earlier one, which was just from 'diff') |
comment:4
Replying to @sagetrac-drkirkby:
Dave, $SAGE_ROOT/local/bin is not under hg. That's the only way to make it into the release, AFAIK. |
comment:5
Replying to @dimpase:
It is, actually; as I said before, it's the "scripts" repository discussed in the Sage developer's guide. |
comment:6
I created that while running Mercurial from $SAGE_ROOT/local/bin which is what you said John. I did not use 'hg_scripts'. (I tend to prefer using 'hg' as it means one can apply patches to Sage before even building it. I'll try use 'hg_scripts' later, but for now I need to do something more pressing. I'm in the middle of decorating and my wife will be coming home in a few days expecting to see it done. Like all these things, it takes a lot longer than one thinks. So this patch is going to have to wait a bit. If anyone want to replace my patch with one more suitable, feel free, but otherwise I'll sort this out when I've got the more important jobs out of the way. Dave |
comment:7
Running "hg_scripts" from within Sage is, as far as I know, equivalent to running "hg" or "sage -hg" from the command line while in the directory $SAGE_ROOT/local/bin. So you don't need to produce a new patch. |
comment:8
Thank you John. That makes sense. I think I just need to put a note for the release manager to sage what repository it goes in, but apart from that, I think it will be ok. All we need now is someone to review it! Since it was your idea, and I tested and wrote it, then neither of us can review it. Dave |
comment:9
** Note to the release manager - this patch is for the sage shell scripts repository.** |
comment:10
Replying to @sagetrac-drkirkby:
Works on t2 (and on a Linux install, just to check it doesn't break anything badly). |
comment:11
Merged "sage-spkg.patch" into 4.4.alpha0. |
Merged: sage-4.4.alpha0 |
Reviewer: Dmitrii Pasechnik |
Hardware
== Software ==
The problem
Despite the fact that Sage builds and pass all doctests (including the long ones), installing optional packages is problematic, as it would appear something is calling 'grep' with the '-o' option which is not POSIX compatible
http://www.opengroup.org/onlinepubs/9699919799/utilities/grep.html
This causes the problems below:
CC: @jhpalmieri
Component: porting: Solaris
Author: David Kirkby, John Palmieri
Reviewer: Dmitrii Pasechnik
Merged: sage-4.4.alpha0
Issue created by migration from https://trac.sagemath.org/ticket/8509
The text was updated successfully, but these errors were encountered: