Opened 5 years ago
Closed 5 years ago
#16113 closed defect (fixed)
CtrlC while sage b
Reported by:  vbraun  Owned by:  

Priority:  major  Milestone:  sage6.4 
Component:  build  Keywords:  
Cc:  nthiery, robertwb  Merged in:  
Authors:  Volker Braun  Reviewers:  Jeroen Demeyer 
Report Upstream:  Fixed upstream, in a later stable release.  Work issues:  
Branch:  cf3f073 (Commits)  Commit:  cf3f073dd109eb05905ce245fe672e369d9df7f0 
Dependencies:  #16912  Stopgaps: 
Description (last modified by )
Interrupting (CtrlC) in the parallel builder for the sage extension modules (sage b
) sometimes doesn't work.
Steps to reproduce:
 sage ba
 press CtrlC after the cythonization started
Change History (21)
comment:1 Changed 5 years ago by
 Cc nthiery added
 Component changed from PLEASE CHANGE to build
 Description modified (diff)
 Type changed from PLEASE CHANGE to defect
comment:2 Changed 5 years ago by
 Report Upstream changed from N/A to Reported upstream. No feedback yet.
comment:3 Changed 5 years ago by
 Branch set to u/vbraun/ctrl_c_while_sage__b
comment:4 Changed 5 years ago by
 Commit set to f4d0ff2a7309457e96bb97b4e26249536f6da114
 Description modified (diff)
 Status changed from new to needs_review
New commits:
f4d0ff2  make sage b interruptable

comment:6 Changed 5 years ago by
 Milestone changed from sage6.2 to sage6.3
comment:7 Changed 5 years ago by
 Status changed from needs_review to needs_info
comment:8 Changed 5 years ago by
 Milestone changed from sage6.3 to sage6.4
comment:9 Changed 5 years ago by
 Report Upstream changed from Reported upstream. No feedback yet. to Fixed upstream, in a later stable release.
Should be fixed by Cython 0.21, see #16912.
comment:10 Changed 5 years ago by
#16912 indeed fixes this during the Cythonizing phase, but the GCC phase which uses distutils (I guess?) still exhibits the same problem.
comment:11 Changed 5 years ago by
 Reviewers set to Jeroen Demeyer
I see, you have a patch for setup.py
also.
comment:12 Changed 5 years ago by
 Dependencies set to #16912
comment:13 Changed 5 years ago by
 Branch changed from u/vbraun/ctrl_c_while_sage__b to u/jdemeyer/ticket/16113
 Created changed from 04/09/14 17:26:30 to 04/09/14 17:26:30
 Modified changed from 09/01/14 08:21:16 to 09/01/14 08:21:16
comment:14 Changed 5 years ago by
 Commit changed from f4d0ff2a7309457e96bb97b4e26249536f6da114 to f02ceccc4b0d506628c249d0956b157080775850
comment:15 Changed 5 years ago by
 Status changed from needs_info to needs_review
comment:16 Changed 5 years ago by
 Commit changed from f02ceccc4b0d506628c249d0956b157080775850 to 3d9db043296e865fb644c431aa6350d48d66e5e8
Branch pushed to git repo; I updated commit sha1. New commits:
3d9db04  Remove maxtasksperchild=1

comment:18 Changed 5 years ago by
 Status changed from positive_review to needs_work
Conflicts with the Cython upgrade since you force pushed at #16912..
comment:19 Changed 5 years ago by
 Commit changed from 3d9db043296e865fb644c431aa6350d48d66e5e8 to cf3f073dd109eb05905ce245fe672e369d9df7f0
comment:20 Changed 5 years ago by
 Status changed from needs_work to positive_review
comment:21 Changed 5 years ago by
 Branch changed from u/jdemeyer/ticket/16113 to cf3f073dd109eb05905ce245fe672e369d9df7f0
 Resolution set to fixed
 Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
Upstream Cython ticket: http://trac.cython.org/cython_trac/ticket/827