Modify ↓
#1807 closed defect (fixed)
unexpected exception by checking existane of plugins directory
| Reported by: | Owned by: | Christopher Lenz | |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.9 |
| Component: | general | Version: | devel |
| Severity: | minor | Keywords: | |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
The line 71 of loader.py causes exception by {{{listdir()}}
if 'plugins' directory does not exist.
I think os.path.exists() should be used for this purpose.
--- loader.py (revision 2009)
+++ loader.py (working copy)
@@ -68,7 +68,7 @@
except ImportError, e:
env.log.error('Component module %s not found',
module, exc_info=True)
- elif os.listdir(os.path.join(env.path, 'plugins')):
+ elif os.path.exists(os.path.join(env.path, 'plugins')):
self.env.warning('setuptools is required for plugin deployment')
# Load default components
Many users are trapped here, I guess.
Attachments (0)
Change History (2)
comment:1 by , 20 years ago
| Milestone: | → 0.9 |
|---|---|
| Owner: | changed from to |
| Severity: | major → minor |
| Status: | new → assigned |
comment:2 by , 20 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
I confirmed this issue is fixed, so I close by hand because commit action of [2010] didn't close it automaticaly.
Note:
See TracTickets
for help on using tickets.



You're right, I'll fix that.