Opened 19 years ago
Closed 19 years ago
#3829 closed (fixed)
Windows setup failed
| Reported by: | Owned by: | Adrian Holovaty | |
|---|---|---|---|
| Component: | Tools | Version: | 0.96 |
| Severity: | Keywords: | ||
| Cc: | dev@… | Triage Stage: | Accepted |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
I could be doing something daft, but I had to change the following line in setup.py:
package = dirpath[len_root_dir:].lstrip('/').replace('/', '.')
to:
package = dirpath[len_root_dir:].lstrip('
').replace('
', '.')
to make it install under Windows.
Attachments (1)
Change History (8)
comment:1 by , 19 years ago
| Cc: | added |
|---|---|
| Component: | Uncategorized → Tools |
| Owner: | changed from to |
comment:2 by , 19 years ago
If it is a problem, this would be better (assuming os.path is imported):
package = dirpath[len_root_dir:].lstrip(os.path.sep).replace(os.path.sep, '.')
comment:3 by , 19 years ago
Or better:
package = dirpath[len_root_dir:].lstrip(os.path.sep).replace(os.path.sep, os.path.curdir)
by , 19 years ago
comment:4 by , 19 years ago
| Has patch: | set |
|---|---|
| Patch needs improvement: | set |
| Triage Stage: | Unreviewed → Accepted |
Just noticed that this line also uses lstrip() which isn't available in Python 2.3, is it?
comment:6 by , 19 years ago
| Patch needs improvement: | unset |
|---|
Cool, I was confusing with rsplit. I don't have access to a windows machine, would someone who does like to test the patch?
comment:7 by , 19 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
This was fixed in a different way in [4912].
Hmm. That does look like we're assuming a *nix enviroment. Can someone else who runs windows confirm this?