Add prepare-dev.py to pre 1.1 version of Reviewboard

Review Request #1177 — Created Oct. 23, 2009 and discarded

Information

Review Board
release-1.0.5.1

Reviewers

Added prepare-dev.py to pre 1.1 version (viz. release-1.0.5.1) of Reviewboard. The tool for setting up a quick test/dev server with RB 1.1 is really easy to use. I missed it when I tried to go to a stable version for some testing/hacking.

I've simply copied the template and script for setting up a quick dev server. I suspect the extra 1.1 template settings are not appropriate so this review uses a completely different name. I do no expect this change to be accepted as is!

Main diffs are:

1) adds prepare-dev.py from 1.1 BUT with some tweaks so that it works for 1.0.5.1
2) updated (well renamed) template file, defaults to sqlite, etc. May well have some 1.1 specific settings in it that should not be there
3) I ended up removing the site.sync_database(allow_input=True) call - under 1.1 I found this did not work and a manual syncdb call was needed.

If the approach is approved, I'm happy to update and remove the commented out code and update the template (with content from 1.1).
git clone git://github.com/reviewboard/reviewboard.git
cd reviewboard
git checkout release-1.0.5.1
# branch....

# hack attached diffs

python contrib/internal/prepare-dev.py
reviewboard/manage.py syncdb
./contrib/internal/devserver.sh
CL
  1. 
      
  2. contrib/internal/prepare-dev.py (Diff revision 1)
     
     
     
    NOTE hacked up template name - should not be submitted like this!
    
    1. Have to wonder how important it is to really keep a different name. is this a problem today where we have settings files that aren't compatible between 1.0.x and 1.1?
  3. contrib/internal/prepare-dev.py (Diff revision 1)
     
     
     
     
     
     
     
    These lines are different compared with Reviewboard 1.1
    
  4. contrib/internal/prepare-dev.py (Diff revision 1)
     
     
     
    Even under 1.1 this does not work properly (for me with headrevs as of yesterday). It completes successfully but actually using the server results in the "use syncdb" message in the browser.
    
    1. Hrm. I'd like to find out why and fix this if at all possible. It's the same version of Django.
  5. contrib/internal/prepare-dev.py (Diff revision 1)
     
     
    explicit call syncdb message
  6. 
      
CL
Review request changed

Change Summary:

Copied 1.1 template over 1.0.5.1 template. I hope the two are compatible ;-) I've had no problems using a 1.1 template with my 1.0.5.1 testing environment.

Attempted to address review comments.

NOTE see review http://reviews.reviewboard.org/r/1183/ which is related to the "this doesn't work under 1.1" comment. I suspect after http://reviews.reviewboard.org/r/1183 has been reviewed this review will also need an update. E.g. remove the commented out code.

Diff:

Revision 2 (+131 -9)

Show changes

Loading...