#9799: Running "make" in SAGE_ROOT returns the wrong exit code, leading to all
kinds of confusion
------------------------+---------------------------------------------------
Reporter: was | Owner: GeorgSWeber
Type: defect | Status: needs_review
Priority: critical | Milestone: sage-4.6
Component: build | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------+---------------------------------------------------
Comment(by leif):
Replying to [comment:8 jhpalmieri]:
> I've tested this a little bit, and it basically seems to work for me. I
also have no objections to renaming it {{{Makefile}}}.
Fine.
> (If it does get renamed, then #9433 will need to be changed
accordingly.)
Yes, that's already recorded there.
> I'm attaching version 3; this just adds Leif's requested phony targets,
Thanks.
> plus changes some tabs to spaces. With the tabs, I see this at the
start of the build:
{{{
(sage-4.5.3) [10:58]$ make
# Note that (currently) "tee" will be run in the directory cd'ed to
# in pipestatus' first argument, i.e. "spkg/":
spkg/pipestatus "cd spkg && ./install all 2>&1" "tee -a ../install.log"
}}}
Well, that's (just) informational and does no harm... ;-)
> With spaces instead, the lines starting with "#" do not appear.
I'm currently not that sure that would be very portable; instead, you
could also prepend {...@}}} (to {{{#}}}, and leave the tabs). The comments
were actually ''shell'' comments (i.e., part of the receipt) rather than
Makefile comments.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9799#comment:9>
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.
