Skip to content
This repository has been archived by the owner on Jan 23, 2019. It is now read-only.

Specify the database in case a different user is creating it #104

Closed
wants to merge 1 commit into from

Conversation

rosenfeld
Copy link
Contributor

No description provided.

@phlipper
Copy link
Owner

@rosenfeld thank you for this contribution!

Can you tell me the issue you were running in to that prompted this change? I would think checking the existence of the DB with the user that creates it should work properly. If not, would it make more sense to make the change to the not_if guard here and just set the user directly there?

@rosenfeld
Copy link
Contributor Author

postgres is not the user but the database to connect to. If you don't specify it will try to connect to the database named as the user running the command, which usually won't exist unless you create it explicitly.

@phlipper
Copy link
Owner

@rosenfeld thanks for the explanation, I misread it the first time through. I have rebased this on top of the latest master and merged it in as #108. Thanks again for the contribution!

@phlipper phlipper closed this Jun 21, 2014
@rosenfeld
Copy link
Contributor Author

Thanks :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants