Ticket #1170 (closed defect: fixed)
[patch] On windows 'tg-admin sql create' with sqlalchemy cause 'database not found' message
| Reported by: | fredlin | Owned by: | anonymous |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.0 |
| Component: | SQLAlchemy | Version: | 1.0b1 |
| Severity: | critical | Keywords: | sqlalchemy tg-admin |
| Cc: |
Description
Windows user use sqlalchemy with
tg-admin sql create
will cause 'cannot find database location' message.
The reason is 'current_dir_uri' variable in config.py will add redondent '/' sign in front of the uri.
The patch need the user that use Other OS to confirm if it this change is save for both OS users.
Change History
comment:1 Changed 5 years ago by jorge.vargas
either your patch or title is wrong cause your saying SA and SO :)
I guess this is with SA since I haven't seen this on windows/so
comment:2 Changed 5 years ago by fredlin
I guess this is with SA since I haven't seen this on windows/so
I think this is because a wild patch of config.py, which will add an additional slash(/).
By this patch both SO/SA works well on windows.(need confirm on other OS)
comment:4 Changed 5 years ago by alberto
- Milestone changed from 1.1 to __unclassified__
Batch moved into unclassified from 1.1 to properly track progress on the later
comment:6 Changed 5 years ago by paj
- Summary changed from On windows 'tg-admin sql create' with sqlalchemy cause 'database not found' message to [patch] On windows 'tg-admin sql create' with sqlalchemy cause 'database not found' message
comment:7 Changed 4 years ago by paj
- Status changed from new to closed
- Resolution set to fixed
Since this ticket was submitted, the quickstart template has been corrected. It may be prudent to remove the two lines in config.py the patch suggests; they seem Unix specific. However, for the sake of minimal changes I'm going to leave it be.
