Opened 20 years ago
Closed 20 years ago
#3055 closed enhancement (fixed)
Make trac-admin a bit more userfriendly during initenv
| Reported by: | Owned by: | Christian Boos | |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.10 |
| Component: | admin/console | Version: | devel |
| Severity: | minor | Keywords: | initenv |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
trac-admin currently generates a traceback when the directory passed to trac-admin's initenv is exists.
The attached patch solves this by checking if the directory exists and skipping the creation call. The patch furthermore checks if the passed directory already has files in them, checks these to see if it might be a Subversion repository (this happens from time to time according to alect) and reporting that we do not install into non-empty directories.
All cases tested.
Attachments (1)
Change History (3)
by , 20 years ago
| Attachment: | trac-dir.diff added |
|---|
comment:1 by , 20 years ago
| Keywords: | initenv added |
|---|---|
| Milestone: | 0.9.6 → 0.10 |
| Owner: | changed from to |
We can do something like this for 0.10. I'll skip the Subversion parts, as we don't want to be too much tied to a specific VersioningSystemBackend. Simply aborting on non-empty directories will also ensure such a mistake can't be made.
comment:2 by , 20 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Something similar to your suggestion has been done in r3213.
See ticket description