Re: [HACKERS] [patch] pg_ctl init extension

2009-12-09 Thread Peter Eisentraut
On ons, 2009-12-09 at 15:18 +0100, Zdenek Kotala wrote: > Robert Haas píše v st 09. 12. 2009 v 08:56 -0500: > > On Dec 9, 2009, at 8:32 AM, Zdenek Kotala wrote: > > > > > > > > Ahh, It is somethink what I want to do, but it is not ready yet in > > > this > > > patch, but I already documented i

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-09 Thread Zdenek Kotala
Robert Haas píše v st 09. 12. 2009 v 08:56 -0500: > On Dec 9, 2009, at 8:32 AM, Zdenek Kotala wrote: > > > > Ahh, It is somethink what I want to do, but it is not ready yet in > > this > > patch, but I already documented it. Idea is to install initdb and > > postgres into libexecdir and packag

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-09 Thread Robert Haas
On Dec 9, 2009, at 8:32 AM, Zdenek Kotala wrote: Greg Smith píše v út 08. 12. 2009 v 22:44 -0500: Zdenek Kotala wrote: thanks for your useful comments. I attached new doc patch version. I removed example changes and add link to create database cluster (I hope) everywhere. Please, look on

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-09 Thread Zdenek Kotala
Greg Smith píše v út 08. 12. 2009 v 22:44 -0500: > Zdenek Kotala wrote: > > thanks for your useful comments. I attached new doc patch version. I > > removed example changes and add link to create database cluster (I hope) > > everywhere. Please, look on it and let me know if there is still > > som

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-08 Thread Greg Smith
Zdenek Kotala wrote: thanks for your useful comments. I attached new doc patch version. I removed example changes and add link to create database cluster (I hope) everywhere. Please, look on it and let me know if there is still something what should be changed. That looks much better. There

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-07 Thread Zdenek Kotala
Greg, thanks for your useful comments. I attached new doc patch version. I removed example changes and add link to create database cluster (I hope) everywhere. Please, look on it and let me know if there is still something what should be changed. Thanks Zdenek Greg Smith píše v ne 06.

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-06 Thread Tom Lane
Greg Smith writes: > The biggest problem is that all of the places that used to say > "" when talking about creating a cluster now just say > "database cluster initialization"--with no link to a section covering > that topic. That's not a good forward step. The part I'm more > favorable towa

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-06 Thread Greg Smith
I just noticed that there was an updated patch here that never made its way onto the CommitFest app listing. I just fixed that and took a quick look at it. I was in favor of this code change, but I have to say even I don't really like how it ended up getting documented--and I'm sure there are

Re: [HACKERS] [patch] pg_ctl init extension

2009-12-04 Thread Zdenek Kotala
I attached updated patch and doc patch. Zdenek Peter Eisentraut píše v so 21. 11. 2009 v 13:19 +0200: > On lör, 2009-11-14 at 14:50 +0100, Zdenek Kotala wrote: > > Peter Eisentraut píše v so 14. 11. 2009 v 10:41 +0200: > > > On tor, 2009-09-17 at 21:43 +0200, Zdenek Kotala wrote

Re: [HACKERS] [patch] pg_ctl init extension

2009-11-21 Thread Peter Eisentraut
On lör, 2009-11-14 at 14:50 +0100, Zdenek Kotala wrote: > Peter Eisentraut píše v so 14. 11. 2009 v 10:41 +0200: > > On tor, 2009-09-17 at 21:43 +0200, Zdenek Kotala wrote: > > > Attached patch extends pg_ctl command with init option. > > > > > > pg_ctl -D /var/lib/postgres [-s] init > > > > > >

Re: [HACKERS] [patch] pg_ctl init extension

2009-11-14 Thread Tom Lane
"Kevin Grittner" writes: > Peter Eisentraut wrote: >> The patch is pretty straightforward, >> but does anyone else actually want this? Comments? > I agree that the initdb name seems odd next to the other executable > names, but the functionality seems a little out of place to me in > pg_ctl. Th

Re: [HACKERS] [patch] pg_ctl init extension

2009-11-14 Thread Kevin Grittner
Peter Eisentraut wrote: > The patch is pretty straightforward, > but does anyone else actually want this? Comments? I agree that the initdb name seems odd next to the other executable names, but the functionality seems a little out of place to me in pg_ctl. The other options all correspond (mo

Re: [HACKERS] [patch] pg_ctl init extension

2009-11-14 Thread Zdenek Kotala
Peter Eisentraut píše v so 14. 11. 2009 v 10:41 +0200: > On tor, 2009-09-17 at 21:43 +0200, Zdenek Kotala wrote: > > Attached patch extends pg_ctl command with init option. > > > > pg_ctl -D /var/lib/postgres [-s] init > > > > This should replace usage of initdb command which has problematic nam

Re: [HACKERS] [patch] pg_ctl init extension

2009-11-14 Thread Peter Eisentraut
On tor, 2009-09-17 at 21:43 +0200, Zdenek Kotala wrote: > Attached patch extends pg_ctl command with init option. > > pg_ctl -D /var/lib/postgres [-s] init > > This should replace usage of initdb command which has problematic name > as we already discussed several times. Initdb binary will be st

Re: [HACKERS] [patch] pg_ctl init extension

2009-09-17 Thread Zdenek Kotala
Bernd Helmle píše v čt 17. 09. 2009 v 23:26 +0200: > > --On 17. September 2009 23:00:12 +0300 Peter Eisentraut > wrote: > > > f the name is a problem, why not change the name? What you are > > proposing above is effectively a very elaborate name change, so why not > > go for a simpler one? >

Re: [HACKERS] [patch] pg_ctl init extension

2009-09-17 Thread Zdenek Kotala
Peter Eisentraut píše v čt 17. 09. 2009 v 23:00 +0300: > On tor, 2009-09-17 at 21:43 +0200, Zdenek Kotala wrote: > > Attached patch extends pg_ctl command with init option. > > > > pg_ctl -D /var/lib/postgres [-s] init > > > > This should replace usage of initdb command which has problematic na

Re: [HACKERS] [patch] pg_ctl init extension

2009-09-17 Thread Bernd Helmle
--On 17. September 2009 23:00:12 +0300 Peter Eisentraut wrote: f the name is a problem, why not change the name? What you are proposing above is effectively a very elaborate name change, so why not go for a simpler one? I don't like the solution by using -o "" to push down command line

Re: [HACKERS] [patch] pg_ctl init extension

2009-09-17 Thread Peter Eisentraut
On tor, 2009-09-17 at 21:43 +0200, Zdenek Kotala wrote: > Attached patch extends pg_ctl command with init option. > > pg_ctl -D /var/lib/postgres [-s] init > > This should replace usage of initdb command which has problematic name > as we already discussed several times. Initdb binary will be st

[HACKERS] [patch] pg_ctl init extension

2009-09-17 Thread Zdenek Kotala
Attached patch extends pg_ctl command with init option. pg_ctl -D /var/lib/postgres [-s] init This should replace usage of initdb command which has problematic name as we already discussed several times. Initdb binary will be still there, but it can be renamed and move into execlib dir in the fu