Opened 7 years ago

Closed 7 years ago

#14030 closed enhancement (fixed)

Switch order in automatic backtrace

Reported by: vbraun Owned by: jdemeyer
Priority: major Milestone: sage-5.9
Component: scripts Keywords:
Cc: jdemeyer Merged in: sage-5.9.beta5
Authors: Volker Braun Reviewers: Jeroen Demeyer
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #13889 Stopgaps:

Description

Same order for C and Cython backtrace.

Attachments (1)

trac_14030_reverse_order.patch (2.0 KB) - added by vbraun 7 years ago.
Updated patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by vbraun

  • Authors set to Volker Braun
  • Cc jdemeyer added
  • Component changed from c_lib to scripts
  • Status changed from new to needs_review

comment:2 Changed 7 years ago by jdemeyer

Does sage_crash_IAI1or.log look right to you? Why are there only 4 frames in the Cython backtrace?

And (this is unrelated to this ticket) why is there a message

Exception:  <type 'exceptions.TypeError'> must be string, not None
Last edited 7 years ago by jdemeyer (previous) (diff)

comment:3 Changed 7 years ago by vbraun

That means that the Python script running in GDB encountered an error. Unfortunately it doesn't do automatic backtraces. I've updated the patch to manually print a backtrace. Can you give it a try? This should tell us what is wrong on your system...

comment:4 Changed 7 years ago by jdemeyer

Exception:  <type 'exceptions.TypeError'> must be string, not None


warning: Could not load shared library symbols for linux-vdso.so.1.
Do you need "set solib-search-path" or "set sysroot"?
warning: File "/home/jdemeyer/local/lib64/libstdc++.so.6.0.16-gdb.py" auto-loading has been declined by your `auto-load safe-path' set to "$debugdir:$datadir/auto-load".
Traceback (most recent call last):
  File "<string>", line 118, in <module>
  File "<string>", line 90, in invoke
  File "<string>", line 98, in newest_first_order
  File "<string>", line 69, in print_stackframe
TypeError: must be string, not None

Changed 7 years ago by vbraun

Updated patch

comment:5 Changed 7 years ago by vbraun

Ok, updated patch should fix your problem.

comment:6 Changed 7 years ago by jdemeyer

  • Reviewers set to Jeroen Demeyer
  • Status changed from needs_review to positive_review

comment:7 Changed 7 years ago by jdemeyer

  • Merged in set to sage-5.9.beta5
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.