Update puppet-postgresql and remove unless

I've fixed the issue with the puppet-postgresql module which caused us to add the unless => 'SELECT false' as a workaround. That workaround doesn't seem to work on fresh provisions, or at least not for me.

Here's the pull request for puppet-postgresql:
https://github.com/puppetlabs/puppet-postgresql/pull/144

[Wladimir Palant, 2013-04-03 14:48:34]

LGTM but you might want to use the simpler puppet-postgresql fix.

[Felix Dahlke, 2013-04-03 16:18:41]

On 2013/04/03 14:48:34, Wladimir Palant wrote:
> LGTM but you might want to use the simpler puppet-postgresql fix.

You mean the one suggested in the pull request? Should work for us, but I'm not
sure if it's the proper way to do it. To my terror, the guy reviewing that
doesn't seem sure either...

[Wladimir Palant, 2013-04-03 19:21:47]

Yes, the suggestion to use false rather than :false sounds like the right thing
to do. Not that I know Ruby enough to be sure myself...

[Felix Dahlke, 2013-04-04 02:25:57]

On 2013/04/03 19:21:47, Wladimir Palant wrote:
> Yes, the suggestion to use false rather than :false sounds like the right
thing
> to do. Not that I know Ruby enough to be sure myself...

It's not a Ruby thing, :false is just a keyword (which are arbitrary, like a
string, and can in fact be converted to/from strings). How I implemented it, we
can use any of false, "false" or :false in manifests. I did get the impression
that's how it's supposed to be.

[Felix Dahlke, 2013-06-04 15:11:33]

On 2013/04/04 02:25:57, Felix H. Dahlke wrote:
> On 2013/04/03 19:21:47, Wladimir Palant wrote:
> > Yes, the suggestion to use false rather than :false sounds like the right
> thing
> > to do. Not that I know Ruby enough to be sure myself...
> 
> It's not a Ruby thing, :false is just a keyword (which are arbitrary, like a
> string, and can in fact be converted to/from strings). How I implemented it,
we
> can use any of false, "false" or :false in manifests. I did get the impression
> that's how it's supposed to be.

Well, since that's what the docs say, and since that guy won't merge it
otherwise, and since it works for us anyway, I simplified things. I wasn't able
to figure out enough about booleans in Puppet to know how they're supposed to be
handled.