Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix #190 - Document the requirement for uuid-ossp PostgreSQL extension #219

Merged
merged 1 commit into from
Mar 3, 2014
Merged

Fix #190 - Document the requirement for uuid-ossp PostgreSQL extension #219

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 7 additions & 4 deletions docs/contributing.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,13 +74,16 @@ Finally you can setup the project:

$ # Create a Database
$ createdb warehouse

$ # Install the CIText extension
$ psql warehouse -c "CREATE EXTENSION IF NOT EXISTS citext"

$ psql warehouse -c 'CREATE EXTENSION IF NOT EXISTS citext'

$ # Install the UUID extension
$ psql warehouse -c 'CREATE EXTENSION IF NOT EXISTS "uuid-ossp"'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is uuid needed anymore? IIRC it was originalyl added for linehaul.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol, just realized you need it for teh forwards migration which you need so that table can be deleted.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Techincally? No. Actually? Yes.

The migrations won't run successfully without it installed because the linehaul migrations are still there (I just added another migration to remove the table). However I plan on using UUID's other places anyways so it's not a big deal that it's "needed" but not really used at the moment.


$ # Migrate the database to the latest schema
$ warehouse -c dev/config.yml migrate upgrade head

$ # Serve Warehouse at http://localhost:9000/
$ warehouse -c dev/config.yml serve

Expand Down