Opened 9 years ago
Closed 9 years ago
#15624 closed enhancement (fixed)
Support "make jN"
Reported by:  Jeroen Demeyer  Owned by:  

Priority:  minor  Milestone:  sage6.2 
Component:  build  Keywords:  
Cc:  Merged in:  
Authors:  Jeroen Demeyer  Reviewers:  Peter Bruin 
Report Upstream:  N/A  Work issues:  
Branch:  u/pbruin/15624parallelmake (Commits, GitHub, GitLab)  Commit:  43b3cd6bcb924ae8291e8d456d7e7e2bf5604ed8 
Dependencies:  #15596, #15580  Stopgaps: 
Description (last modified by )
This should be supported by the toplevel Makefile
:
$ make j4
The branch on this ticket implements this for submake
s. For uses of $MAKE
outside of make
itself (like building the Sage documentation or Sage library), the value of j
given to make
cannot be determined so this cannot work.
Change History (8)
comment:1 Changed 9 years ago by
Branch:  → u/jdemeyer/ticket/15624 

Created:  Jan 3, 2014, 7:31:01 AM → Jan 3, 2014, 7:31:01 AM 
Modified:  Jan 3, 2014, 7:31:01 AM → Jan 3, 2014, 7:31:01 AM 
comment:2 Changed 9 years ago by
Authors:  → Jeroen Demeyer 

Commit:  → edd3b144ee734b54efa2e5a5c55e8065928a8d60 
Dependencies:  → #15596, #15580 
Description:  modified (diff) 
comment:3 Changed 9 years ago by
Commit:  edd3b144ee734b54efa2e5a5c55e8065928a8d60 → 8e1dabdb31da8a11748b2a07c80e881a002923f7 

Branch pushed to git repo; I updated commit sha1. This was a forced push. Recent commits:
8e1dabd  Support toplevel make jN

0255a2a  Allow sage sdist without building Sage

68ac77f  Merge branch 'u/jdemeyer/ticket/15596' of git://trac.sagemath.org/sage into ticket/15580

c7c0106  sagesdist: copy upstream tarballs using sagespkg

143e5e4  Don't use cp p in sageclonesource

comment:4 Changed 9 years ago by
Description:  modified (diff) 

Status:  new → needs_review 
comment:5 Changed 9 years ago by
Description:  modified (diff) 

comment:6 Changed 9 years ago by
Milestone:  sage6.1 → sage6.2 

comment:7 Changed 9 years ago by
Branch:  u/jdemeyer/ticket/15624 → u/pbruin/15624parallelmake 

Commit:  8e1dabdb31da8a11748b2a07c80e881a002923f7 → 43b3cd6bcb924ae8291e8d456d7e7e2bf5604ed8 
Reviewers:  → Peter Bruin 
Status:  needs_review → positive_review 
I used this to test #15765 and it appears to work well. Just changing the branch because of a trivial merge conflict.
comment:8 Changed 9 years ago by
Resolution:  → fixed 

Status:  positive_review → closed 
Note: See
TracTickets for help on using
tickets.
New commits:
Support toplevel make jN