Pushed to git. Thanks for the review.
Dne 8.7.2010 18:07, Denys Vlasenko napsal(a):
EXECUTABLE The file with the main entry point of the application. Might be a JAR archive, a script file, a bytecode file, or the interpreter. It should be a real file on the filesystem, so ABRT can check the origin of the file (source package). If the file is not known, the most appropriate is to shortly describe the reason why it is not known in the field, e.g. "Exception raised from Python shell".
Should we suggest to not save such crashes? They are hardly useful: someone experiments in interactive python or perl, and BOOM - we get completely useless crash dumps, possibly many of them while the experimenting continues.
Yes, the suggestion added.
ANALYZER The name of an analyzer plugin which should be run during the analysis.
add something like: "...It is used to choose appropriate method of post processing for the crash dump. Sensible names are "python", "perl", "java" etc."
Done.
BASENAME Short string indicating the origin of the event. It is used as a part of directory name in /var/spool/abrt. Use "python" for Python exceptions, "ruby" for ruby interpreter, "java" for JRE.
I'd rephrase it, showing how exactly it is going to be used: "It is used as a part of directory name: /var/spool/abrt/BASENAME-suffix"
Agreed. Done.
BACKTRACE The backtrace in a format that is commonly used by the interpreter.
(1) the phrase is unclear. Maybe: "The backtrace at the moment of the crash, if it is available. May include other useful information. For example, for ccpp crashes, it includes register dump." (2) Is it mandatory?
All fields are mandatory now. Added this information.
(3) What to supply if it is not available?
I don't know, the crash is not useful without the backtrace. Any suggestions?
Here is an example of a crash report from Python, as it is written to the socket: PID=563\0 EXECUTABLE=/usr/bin/python\0 ANALYZER=Python\0 BASENAME=pyhook\0 REASON=CCMainWindow.py:1:<module>:ZeroDivisionError: integer division or modulo by zero\0 BACKTRACE=%s\0 DONE\0
A better BACKTRACE= value is needed here.
Fixed.