| 1 |
-----BEGIN PGP SIGNED MESSAGE----- |
| 2 |
Hash: SHA1 |
| 3 |
|
| 4 |
Petteri Räty wrote: |
| 5 |
> Jason Stubbs wrote: |
| 6 |
> |
| 7 |
>>>Not sure if hiding the exception away (traceback) is a good thing; |
| 8 |
>>>wrapping it with purdy output, sure, but swallowing it? |
| 9 |
>> |
| 10 |
>> |
| 11 |
>>If it's failing to import, there's something pretty wrong with it. I'm sure |
| 12 |
>>that module authors will know enough to remove the except block. Is there |
| 13 |
>>really anything useful gained from dumping the traceback? |
| 14 |
>> |
| 15 |
>>-- |
| 16 |
>>Jason Stubbs |
| 17 |
> |
| 18 |
> |
| 19 |
> Well emerge does have a --debug flag under which to dump everything you |
| 20 |
> want. Tracebacks might also be included under the --verbose flag. IMHO |
| 21 |
> tracebacks should in most cases be replaced by user understandable |
| 22 |
> errors messages in such a way that the tracebacks are still easily |
| 23 |
> available if wanted. |
| 24 |
> |
| 25 |
> Regards, |
| 26 |
> Petteri |
| 27 |
|
| 28 |
I would prefer both tbh, which sucks because it's more work :P You have |
| 29 |
to generally catch the exception, print an error message, and then raise |
| 30 |
it again ( or just print the tb, your choice ). That way the user knows |
| 31 |
that we probably guessed that exception would be thrown at one point, |
| 32 |
and we can provide actual instructions while still having a traceback to |
| 33 |
tell us what the hell is going on. |
| 34 |
|
| 35 |
Woe is me when they report the bug and the code is just like |
| 36 |
|
| 37 |
writemsg("!!! Error portage sucks!") |
| 38 |
sys.exit() |
| 39 |
|
| 40 |
....Great, that message probably isn't unique and now their problem |
| 41 |
could be in 5 different places in the code, and I get to guess where :/ |
| 42 |
|
| 43 |
sys.exit-- |
| 44 |
|
| 45 |
- -Alec Warner (antarus) |
| 46 |
-----BEGIN PGP SIGNATURE----- |
| 47 |
Version: GnuPG v1.4.2 (GNU/Linux) |
| 48 |
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org |
| 49 |
|
| 50 |
iQIVAwUBQ6wQzmzglR5RwbyYAQI8BQ/+MGK4GjFsNqBJp9cnBgDIUxqbtU/RNuCH |
| 51 |
4/o449JXWpwc0ndqf93SG/EeyB4FxhiSFcLlmvedimKF+8eLpDzgulVlO44Dl1UK |
| 52 |
QpssiQmjJstkecGrO+IoqEZr47PR4EIMXxjE5FqFQBfsAApYwcGf6Ksyt5YVbWl3 |
| 53 |
bRXoi5FKMKYKKS/RXA7FO/4PItF3DapK7GtF31kjP4RkSRbH8h2gWtbmcC2zIVVp |
| 54 |
Xyd+zetEuPxJwgJ0fqym7okWjLdx5tc7d7nfYgeKluXhzpEcix2MRY6OEzvomJ3k |
| 55 |
VuUBV9m279JqFzPcEUFeUx1/a976HF0o5dGreI0qI+A20iDs7yFYxB8ZXvNg5O2r |
| 56 |
pcOX13okBU/C17ZoCVvmeDwJI1NlSdR+pUHfPaGN54V1BgTaNNS+T4ZftXpLOSja |
| 57 |
A1aaoipqNq215bsKuIWOAJZvnU9BLAc3KjjDEqcIuEoddSlEaS1v6grp1aE0NdSV |
| 58 |
V+iyhAU/Y58gwiCQu86jIwozpGkVEQ42/agPbVYpuPE3+F9bICN9rrmscN9iQv60 |
| 59 |
M9uapRED7VDN9UtWxLqQRQTU0W1RjPeoscda1cHKuwTNCDlEhd6aUwNNrSHgVl1B |
| 60 |
6wl+1Mx/Okr+Zcg/8BfEs7EDa07eB8W/V1++v8vDngNG8DC55ihQpq0+KpVdSsuu |
| 61 |
eT0KTxWT7pQ= |
| 62 |
=KUP3 |
| 63 |
-----END PGP SIGNATURE----- |
| 64 |
-- |
| 65 |
gentoo-portage-dev@g.o mailing list |
Archives