Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #1170 (closed defect: fixed)

Opened 13 years ago

Last modified 9 years ago

[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.

Attachments

win.diff Download (950 bytes) - added by fredlin 13 years ago.

Change History

Changed 13 years ago by fredlin

comment:1 Changed 13 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 13 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:3 Changed 12 years ago by alberto

  • Milestone changed from 1.0b2 to 1.1

comment:4 Changed 12 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:5 Changed 12 years ago by paj

  • Component changed from tg-admin (non-toolbox) to SQLAlchemy

comment:6 Changed 12 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 12 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.

comment:8 Changed 9 years ago by chrisz

  • Milestone changed from __unclassified__ to 1.0
Note: See TracTickets for help on using tickets.