Issue 604875
Created on 2002-09-05 03:53 by stefan, last changed 2002-09-05 03:53 by stefan.
msg324 |
Author: [hidden] (stefan) |
Date: 2002-09-05 03:53 |
|
if dbinit.py contains errors (which is quite possible
as that file is supposed to be edited by the tracker
admin),
'roundup-admin initialise' will generate a bogus error
saying
" 'module' object has no attribute 'init' "
It would be far more useful to catch load errors and report
correctly what went wrong, instead of initializing the
'module' with a dummy placeholder object.
|
msg325 |
Author: [hidden] (richard) |
Date: 2002-09-05 04:02 |
|
Logged In: YES
user_id=6405
Yeah, the try/except in the instance __init__ has always been evil. I
guess it's time to actually try to come up with a better solution.
|
msg326 |
Author: [hidden] (stefan) |
Date: 2002-09-05 04:06 |
|
Logged In: YES
user_id=764
what errors can occure here ? Why are you catching exceptions
in the first place, instead of just letting the caller deal
with them ?
If you want the convenience of a clear error message (and no
backtrace), you should probably wrap the imp.load_package()
in a try block, and then issue a sensible error message and
abort...
|
msg327 |
Author: [hidden] (richard) |
Date: 2002-09-05 04:19 |
|
Logged In: YES
user_id=6405
I'm going to move this discussion to the mailing list where it's easier.
|
|
Date |
User |
Action |
Args |
2002-09-05 03:53:50 | stefan | create | |
|