Re: [HACKERS] Linux LSB init script

2009-09-30 Thread Kevin Grittner
Robert Haas robertmh...@gmail.com wrote: Peter Eisentraut pete...@gmx.net wrote: On Sun, 2009-09-20 at 22:54 -0400, Robert Haas wrote: It seems like there is some support for what this patch is trying to do, but much disagreement about the details of how to get there. Where do we go from

Re: [HACKERS] Linux LSB init script

2009-09-30 Thread Peter Eisentraut
On Wed, 2009-09-30 at 11:58 -0500, Kevin Grittner wrote: Right. It seems that, in addition to the above, there also remains some disagreement about: (1) how much checking the script should do to provide error messages and exit codes which target the specific problems versus generic I'm

Re: [HACKERS] Linux LSB init script

2009-09-27 Thread Robert Haas
On Mon, Sep 21, 2009 at 3:20 AM, Peter Eisentraut pete...@gmx.net wrote: On Sun, 2009-09-20 at 22:54 -0400, Robert Haas wrote: It seems like there is some support for what this patch is trying to do, but much disagreement about the details of how to get there. Where do we go from here? I

Re: [HACKERS] Linux LSB init script

2009-09-21 Thread Peter Eisentraut
On Sun, 2009-09-20 at 22:54 -0400, Robert Haas wrote: It seems like there is some support for what this patch is trying to do, but much disagreement about the details of how to get there. Where do we go from here? I think the next step would be to outline what changes would be necessary in

Re: [HACKERS] Linux LSB init script

2009-09-20 Thread Robert Haas
On Thu, Sep 17, 2009 at 4:52 PM, Robert Haas robertmh...@gmail.com wrote: On Thu, Sep 17, 2009 at 4:18 PM, Peter Eisentraut pete...@gmx.net wrote: On tor, 2009-09-17 at 11:59 -0500, Kevin Grittner wrote: Well, in such cases it may be useful to add an option such as --oknodo to select the

Re: [HACKERS] Linux LSB init script

2009-09-17 Thread Robert Haas
On Thu, Sep 17, 2009 at 1:30 AM, Peter Eisentraut pete...@gmx.net wrote: Well, in such cases it may be useful to add an option such as --oknodo to select the idempotent behavior. It took me about 60 seconds to figure out what I thought you were going for there, so I submit that's not a good

Re: [HACKERS] Linux LSB init script

2009-09-17 Thread Kevin Grittner
Peter Eisentraut pete...@gmx.net wrote: I think it's important to consider what the conforming behavior really achieves in practice. The INFO section is essential nowadays for correct functioning on a large portion of distributions. The exit codes are relatively uninteresting but

Re: [HACKERS] Linux LSB init script

2009-09-17 Thread Greg Smith
On Thu, 17 Sep 2009, Peter Eisentraut wrote: On ons, 2009-09-16 at 12:05 -0500, Kevin Grittner wrote: Well, with differences in behavior, of course, like attempting a fast shutdown, and escalating to an immediate shutdown if that doesn't succeed in the allowed time. Right, but if you think

Re: [HACKERS] Linux LSB init script

2009-09-17 Thread Peter Eisentraut
On tor, 2009-09-17 at 09:26 -0400, Robert Haas wrote: On Thu, Sep 17, 2009 at 1:30 AM, Peter Eisentraut pete...@gmx.net wrote: Well, in such cases it may be useful to add an option such as --oknodo to select the idempotent behavior. It took me about 60 seconds to figure out what I thought

Re: [HACKERS] Linux LSB init script

2009-09-17 Thread Peter Eisentraut
On tor, 2009-09-17 at 11:59 -0500, Kevin Grittner wrote: Well, in such cases it may be useful to add an option such as --oknodo to select the idempotent behavior. I found that confusing (as did Robert); how about --lsm-conforming? s/lsm/lsb/ I'm not so sure that I would label it as LSB,

Re: [HACKERS] Linux LSB init script

2009-09-17 Thread Peter Eisentraut
On tor, 2009-09-17 at 14:21 -0400, Greg Smith wrote: I've implemented exactly the same logic Kevin describes for a past consulting customer and for multiple forms of shutdown scripts at Truviso. I would suggest the situation here is that it's so easy to script a custom solution that the

Re: [HACKERS] Linux LSB init script

2009-09-17 Thread Robert Haas
On Thu, Sep 17, 2009 at 4:18 PM, Peter Eisentraut pete...@gmx.net wrote: On tor, 2009-09-17 at 11:59 -0500, Kevin Grittner wrote: Well, in such cases it may be useful to add an option such as --oknodo to select the idempotent behavior. I found that confusing (as did Robert); how about

Re: [HACKERS] Linux LSB init script

2009-09-16 Thread Peter Eisentraut
OK, I have a number of comments on this proposal. What does it buy us, in general, and compared to the existing (non-LSB) script? This becomes clearer when you consider what LSB actually entails, which is: - INFO header - standardized exit codes - functions from /lib/lsb/init-functions My

Re: [HACKERS] Linux LSB init script

2009-09-16 Thread Kevin Grittner
Peter Eisentraut pete...@gmx.net wrote: OK, I have a number of comments on this proposal. Thanks for looking at it. What does it buy us, in general, and compared to the existing (non-LSB) script? A conforming script for use in LSB conforming implementations. what LSB actually

Re: [HACKERS] Linux LSB init script

2009-09-16 Thread Peter Eisentraut
On ons, 2009-09-16 at 12:05 -0500, Kevin Grittner wrote: what LSB actually entails, which is: - INFO header - standardized exit codes - functions from /lib/lsb/init-functions As well as a standardized set of actions and standard semantics for them, if we're only talking about the

Re: [HACKERS] Linux LSB init script

2009-09-15 Thread Peter Eisentraut
On Wed, 2009-09-02 at 15:06 -0500, Kevin Grittner wrote: Wolfgang Wilhelm wolfgang20121...@yahoo.de wrote: if [ $# -lt 1 -o $1 = ] ] ; then Oops. Fixed patch attached. Thanks! The commitfest lists this as the last patch, but there was some discussion after this. Could you/we

Re: [HACKERS] Linux LSB init script

2009-09-15 Thread Kevin Grittner
Peter Eisentraut pete...@gmx.net wrote: The commitfest lists this as the last patch, but there was some discussion after this. Could you/we clarify what is actually proposed for inclusion now? I have seen proposals for: - Linux LSB init script - Linux non-LSB init script - SUSE

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Kevin Grittner
Peter Eisentraut pete...@gmx.net wrote: On tis, 2009-09-01 at 12:04 -0500, Kevin Grittner wrote: I wouldn't expect a packaged SuSE build to cater to all of that; but it would be nice if they donated their init script to the PostgreSQL project, so that those of us who have a reason to build

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Andrew Dunstan
Kevin Grittner wrote: (2) We don't have the distribution's PostgreSQL package installed on any of our machines, so I'd have to install it and risk breaking something to even have a look at it. Umm, no, you could either install the SRPM in a build directory, and look there, or

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Greg Stark
On Wed, Sep 2, 2009 at 3:13 PM, Kevin Grittnerkevin.gritt...@wicourts.gov wrote: # Copyright (c) 2006 SUSE Linux Products GmbH, Nuernberg, Germany. # All rights reserved. and that I would be violating that copyright by copying it to PostgreSQL.  Am I wrong? The above is just a statement of

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Kevin Grittner
Greg Stark gsst...@mit.edu wrote: Kevin Grittnerkevin.gritt...@wicourts.gov wrote: # Copyright (c) 2006 SUSE Linux Products GmbH, Nuernberg, Germany. # All rights reserved. and that I would be violating that copyright by copying it to PostgreSQL. Am I wrong? The above is just a statement

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Andrew Dunstan
Kevin Grittner wrote: I tracked down the license text. It includes these sections, which leave me disinclined to copy the init script to the PostgreSQL contrib directory: The Software is a collective work of Novell. You must acquire a license for each installation of the Software and for

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Kevin Grittner
Andrew Dunstan and...@dunslane.net wrote: Kevin Grittner wrote: [SuSE Linux Enterprise Server license] Novell reserves all rights not expressly granted to You. You may not: (2) transfer the Software or Your license rights under this Agreement, in whole or in part. here is what's in

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Andrew Dunstan
Kevin Grittner wrote: Andrew Dunstan and...@dunslane.net wrote: Kevin Grittner wrote: [SuSE Linux Enterprise Server license] Novell reserves all rights not expressly granted to You. You may not: (2) transfer the Software or Your license rights under this Agreement,

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Devrim GÜNDÜZ
On Wed, 2009-09-02 at 10:21 -0400, Andrew Dunstan wrote: Umm, no, you could either install the SRPM in a build directory, and look there, or extract the script from the built RPM using rpm2cpio. ... actually I am maintaining SuSE spec and patches unofficially in RPM repository:

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Tom Lane
Andrew Dunstan and...@dunslane.net writes: Kevin Grittner wrote: So while OpenSuse seems a little less restrictive, it's still not something we can copy into the PostgreSQL distribution, right? right. Right. Your clean-room approach seems to have been the right thing.

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Peter Eisentraut
On ons, 2009-09-02 at 09:13 -0500, Kevin Grittner wrote: We don't have the distribution's PostgreSQL package installed on any of our machines, so I'd have to install it and risk breaking something to even have a look at it. Using Midnight Commander is the canonical way to peek inside

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Wolfgang Wilhelm
...@mit.edu; pgsql-hackers@postgresql.org Gesendet: Montag, den 31. August 2009, 21:06:22 Uhr Betreff: Re: [HACKERS] Linux LSB init script Andrew Dunstan and...@dunslane.net wrote: cvsutils That allowed me to use 'cvsdo add' and 'cvs diff -cN' to generate the attached. This contains a couple minor fixes

Re: [HACKERS] Linux LSB init script

2009-09-02 Thread Kevin Grittner
Wolfgang Wilhelm wolfgang20121...@yahoo.de wrote: if [ $# -lt 1 -o $1 = ] ] ; then Oops. Fixed patch attached. Thanks! -Kevin Index: contrib/start-scripts/linux-lsb === RCS file: contrib/start-scripts/linux-lsb diff -N

Re: [HACKERS] Linux LSB init script

2009-09-01 Thread Peter Eisentraut
On mån, 2009-08-31 at 20:54 -0400, Greg Smith wrote: On Mon, 31 Aug 2009, Kevin Grittner wrote: My counter-argument to that would be that the SuSE distribution's version of PostgreSQL is so out-of-date that we don't install it. Given that it's also RPM based, is it possible to get SuSE

Re: [HACKERS] Linux LSB init script

2009-09-01 Thread Peter Eisentraut
On mån, 2009-08-31 at 15:17 -0500, Kevin Grittner wrote: My counter-argument to that would be that the SuSE distribution's version of PostgreSQL is so out-of-date that we don't install it. It also doesn't enable debug info or integer date times. So we switched to build from source before we

Re: [HACKERS] Linux LSB init script

2009-09-01 Thread Kevin Grittner
Peter Eisentraut pete...@gmx.net wrote: On mån, 2009-08-31 at 15:17 -0500, Kevin Grittner wrote: the SuSE distribution's version of PostgreSQL is so out-of-date that we don't install it. It also doesn't enable debug info or integer date times. Fixes and help are welcome, btw. We're

Re: [HACKERS] Linux LSB init script

2009-09-01 Thread Peter Eisentraut
On tis, 2009-09-01 at 12:04 -0500, Kevin Grittner wrote: We're using SLES10. If we installed the latest from the SuSE site, we would be at PostgreSQL version 8.1. Now, to their credit, that is 8.1.17; but we're looking at when to upgrade from 8.3 to 8.4, not at whether to go back to 8.1.

Re: [HACKERS] Linux LSB init script

2009-09-01 Thread Tom Lane
Peter Eisentraut pete...@gmx.net writes: On tis, 2009-09-01 at 12:04 -0500, Kevin Grittner wrote: And unless I'm remembering incorrectly, the configure options are not what we would want. I don't see any reason the packaged build shouldn't be with --enable-debug on a platform where that has

Re: [HACKERS] Linux LSB init script

2009-09-01 Thread David E. Wheeler
On Sep 1, 2009, at 6:43 PM, Tom Lane wrote: Debatable, but it's not upstream default, so why should it be downstream default? FWIW, that particular issue is invariably a matter of distro policy; they could care less what upstream's default is. Well, they could, but do they? /me offers

Re: [HACKERS] Linux LSB init script

2009-09-01 Thread Andrew Dunstan
Kevin Grittner wrote: We're using SLES10. If we installed the latest from the SuSE site, we would be at PostgreSQL version 8.1. Now, to their credit, that is 8.1.17; but we're looking at when to upgrade from 8.3 to 8.4, not at whether to go back to 8.1. I recently build 8.4 RPMs for

Re: [HACKERS] Linux LSB init script

2009-08-31 Thread Kevin Grittner
Peter Eisentraut pete...@gmx.net wrote: If it's a new file, it's pointless to send a patch anyway. If people are OK with just sending the new file, that's cool with me. From the other posts, it appears that I need to have my own copy of the repository to do it as a patch, or download a tool.

Re: [HACKERS] Linux LSB init script

2009-08-31 Thread Kevin Grittner
Andrew Dunstan and...@dunslane.net wrote: cvsutils That allowed me to use 'cvsdo add' and 'cvs diff -cN' to generate the attached. This contains a couple minor fixes to what I posted in new file form. I was holding off on the CommitFest entry until I sorted out the format; I'll link here as

Re: [HACKERS] Linux LSB init script

2009-08-31 Thread Greg Smith
On Mon, 31 Aug 2009, Kevin Grittner wrote: Is the LSB standard sufficiently widely adopted to omit the older format? I'm not sure, and I'm similarly not sure just what the benefits of a LSB compatible script are here given that several major distributions like RHEL/Debian have their own

Re: [HACKERS] Linux LSB init script

2009-08-31 Thread Kevin Grittner
Greg Smith gsm...@gregsmith.com wrote: I'm similarly not sure just what the benefits of a LSB compatible script are here given that several major distributions like RHEL/Debian have their own thing they're doing and are unlikely to change. I don't know about other platforms, but on SuSE

Re: [HACKERS] Linux LSB init script

2009-08-31 Thread Peter Eisentraut
On mån, 2009-08-31 at 12:07 -0500, Kevin Grittner wrote: Is the LSB standard sufficiently widely adopted to omit the older format? I was concerned that there might be Linux versions we wanted to support which wouldn't have a /lib/lsb/init-functions file to source. If that's not an issue, I

Re: [HACKERS] Linux LSB init script

2009-08-31 Thread Kevin Grittner
Peter Eisentraut pete...@gmx.net wrote: While the major distributions support LSB, the major distributions also have PostgreSQL packages available and so will likely not need the init script shipped in the source. My counter-argument to that would be that the SuSE distribution's version of

Re: [HACKERS] Linux LSB init script

2009-08-31 Thread Greg Smith
On Mon, 31 Aug 2009, Kevin Grittner wrote: My counter-argument to that would be that the SuSE distribution's version of PostgreSQL is so out-of-date that we don't install it. Given that it's also RPM based, is it possible to get SuSE in sync so that it shares the same init script as RHEL?

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread Kevin Grittner
I wrote: But before I spend a lot of time fine-tuning it, I wanted to post this as a proof-of-concept draft and see if people think it's on the right track. I chose to take the lack of response as an indication that nobody who cares about this thought there was anything seriously wrong

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread David Fetter
On Fri, Aug 28, 2009 at 04:41:47PM -0500, Kevin Grittner wrote: I wrote: But before I spend a lot of time fine-tuning it, I wanted to post this as a proof-of-concept draft and see if people think it's on the right track. I chose to take the lack of response as an indication that

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread Kevin Grittner
David Fetter da...@fetter.org wrote: cvs diff if you're on CVS. If you're using git, commit to your local repository and git-diff. I tried that. When I just did it at the top level, it ignored the new file. When I specified the file: kgri...@kgrittn-desktop:~/pg/pgsql$ cvs diff -c -N

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread David Fetter
On Fri, Aug 28, 2009 at 04:57:24PM -0500, Kevin Grittner wrote: David Fetter da...@fetter.org wrote: cvs diff if you're on CVS. If you're using git, commit to your local repository and git-diff. I tried that. When I just did it at the top level, it ignored the new file. When I

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread Greg Stark
On Fri, Aug 28, 2009 at 10:57 PM, Kevin Grittnerkevin.gritt...@wicourts.gov wrote: David Fetter da...@fetter.org wrote: cvs diff if you're on CVS.  If you're using git, commit to your local repository and git-diff. I tried that.  When I just did it at the top level, it ignored the new file.

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread David Fetter
On Fri, Aug 28, 2009 at 11:57:07PM +0100, Greg Stark wrote: On Fri, Aug 28, 2009 at 10:57 PM, Kevin Grittnerkevin.gritt...@wicourts.gov wrote: David Fetter da...@fetter.org wrote: cvs diff if you're on CVS.  If you're using git, commit to your local repository and git-diff. I tried

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread Peter Eisentraut
On fre, 2009-08-28 at 16:57 -0500, Kevin Grittner wrote: David Fetter da...@fetter.org wrote: cvs diff if you're on CVS. If you're using git, commit to your local repository and git-diff. I tried that. When I just did it at the top level, it ignored the new file. When I specified

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread Greg Stark
On Fri, Aug 28, 2009 at 11:59 PM, David Fetterda...@fetter.org wrote: You have to cvs add the file That only works if you have write permissions to the central repo.  I don't. True. The only workable way to use cvs that i found was to rsync the repository and then check out from that. If you

Re: [HACKERS] Linux LSB init script

2009-08-28 Thread Andrew Dunstan
David Fetter wrote: On Fri, Aug 28, 2009 at 11:57:07PM +0100, Greg Stark wrote: On Fri, Aug 28, 2009 at 10:57 PM, Kevin Grittnerkevin.gritt...@wicourts.gov wrote: David Fetter da...@fetter.org wrote: cvs diff if you're on CVS. If you're using git, commit to your local

[HACKERS] Linux LSB init script

2009-08-26 Thread Kevin Grittner
Here's a first rough cut of Linux script which attempts to launch PostgreSQL as a somewhat LSB conforming application. It's very lightly tested and I haven't gone through to confirm that every corner case is handled exactly right; in fact on my kubuntu workstation (the only place I've tested so