BIND 10 #3095: Generic traceback handling
BIND 10 Development
do-not-reply at isc.org
Thu Sep 12 13:37:22 UTC 2013
#3095: Generic traceback handling
-------------------------------------+-------------------------------------
Reporter: vorner | Owner:
Type: enhancement | vorner
Priority: medium | Status:
Component: Unclassified | reviewing
Keywords: | Milestone:
Sensitive: 0 | Sprint-20130917
Sub-Project: Core | Resolution:
Estimated Difficulty: 4 | CVSS Scoring:
Total Hours: 0 | Defect Severity: N/A
| Feature Depending on Ticket:
| Add Hours to Ticket: 0
| Internal?: 0
-------------------------------------+-------------------------------------
Changes (by muks):
* owner: muks => vorner
Comment:
Hi Michal
Here is my review of the branch:
* It seems to pass `make distcheck`.
* In `util_messages.mes`, the following comment must be fixed:
{{{
+# No namespace declaration - these constants go in the global namespace
+# of the libddns_messages python module.
}}}
* I think it's better to rename `UNHANDLED_EXCEPTION` message id to
`PYTHON_UNHANDLED_EXCEPTION`, as this message id is used specifically
for Python components.
* Do we want to use this traceback handler for `dbutil` and `loadzone`
too? I'm just raising this point; I don't have a strong preference.
* I have pushed some minor commits which I didn't feel needed more
discussing. Please check if they are okay.
Apart from these, the branch is perfect, so please go ahead and merge
after addressing the above.
--
Ticket URL: <http://bind10.isc.org/ticket/3095#comment:9>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list