Opened 9 years ago
Closed 9 years ago
#14649 closed defect (fixed)
Cython debug symbols and sage --gdb
Reported by: | vbraun | Owned by: | GeorgSWeber |
---|---|---|---|
Priority: | blocker | Milestone: | sage-5.10 |
Component: | build | Keywords: | |
Cc: | jdemeyer, roed, ohanar, ppurka, kini, mhansen, leif | Merged in: | sage-5.10.rc1 |
Authors: | Volker Braun | Reviewers: | Jeroen Demeyer |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
Debugging got broken by #13031.
Apply
- trac_14649_root_cython_debug.patch to the
SAGE_ROOT
repository - trac_14649_cython_debug.patch to the Sage library
Attachments (2)
Change History (11)
Changed 9 years ago by
comment:1 Changed 9 years ago by
- Description modified (diff)
comment:2 Changed 9 years ago by
- Cc jdemeyer roed ohanar ppurka kini mhansen added
- Status changed from new to needs_review
comment:3 follow-up: ↓ 4 Changed 9 years ago by
The doctest should be
# optional - gdb
comment:4 in reply to: ↑ 3 Changed 9 years ago by
comment:5 follow-up: ↓ 6 Changed 9 years ago by
Shouldn't the default value here be 'yes'
?
if os.environ.get('SAGE_DEBUG', None) != 'no':
I don't think it is recommended to compare None with equality operators.
comment:6 in reply to: ↑ 5 Changed 9 years ago by
comment:7 Changed 9 years ago by
- Cc leif added
comment:8 Changed 9 years ago by
- Priority changed from major to blocker
I think this should be a blocker for sage-5.10. Shipping with broken --gdb just makes debugging needlessly complicated.
comment:9 Changed 9 years ago by
- Merged in set to sage-5.10.rc1
- Resolution set to fixed
- Reviewers set to Jeroen Demeyer
- Status changed from needs_review to closed
Note: See
TracTickets for help on using
tickets.
Initial patch