A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

Started by Ryan Kirkpatrickalmost 25 years ago22 messages

pgsql@rkirkpat.net

almost 25 years ago

3 attachment(s)

Just a few random bits and pieces that might be useful to someone
else. They are:

postgres-7-sparc.patch -> This is the same one I posted earlier to
hackers and ports, forgot to post it to patches. This fixes the resultmap
for sparc-linux so that all regression tests pass.

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

pg_ctl.patch -> Minor patch required by postgresql script above.
It basically just allows you to specify the -o option (postmaster options)
more than once and all of them will be concatenated together. Makes it
easier to do '-o -F -o -i', then to try and shell escape quotes in a shell
script. Should be harmless, and might be handy.

Anyway, just wanted to post them in case some one was interested in them.
TTYL.

---------------------------------------------------------------------------
| "For to me to live is Christ, and to die is gain." |
| --- Philippians 1:21 (KJV) |
---------------------------------------------------------------------------
| Ryan Kirkpatrick | Boulder, Colorado | http://www.rkirkpat.net/ |
---------------------------------------------------------------------------

Attachments:

pg_ctl.patchtext/plain; CHARSET=US-ASCII; NAME=pg_ctl.patchDownload

diff -uNr postgresql-7.1beta3/src/bin/pg_ctl/pg_ctl.sh postgresql-7.1beta3-fixed/src/bin/pg_ctl/pg_ctl.sh
--- postgresql-7.1beta3/src/bin/pg_ctl/pg_ctl.sh	Sat Jan  6 19:03:36 2001
+++ postgresql-7.1beta3-fixed/src/bin/pg_ctl/pg_ctl.sh	Sat Jan 27 13:24:45 2001
@@ -141,7 +141,7 @@
 	    ;;
 	-o)
 	    shift
-	    POSTOPTS="$1"
+	    POSTOPTS="$POSTOPTS $1"
 	    ;;
 	-*)
 	    echo "$CMDNAME: invalid option: $1" 1>&2

postgres-7.0-sparc.patchtext/plain; CHARSET=US-ASCII; NAME=postgres-7.0-sparc.patchDownload

diff -uNr postgresql-7.1beta3/src/test/regress/resultmap postgresql-7.1beta3-sparc/src/test/regress/resultmap
--- postgresql-7.1beta3/src/test/regress/resultmap	Sat Jan  6 19:03:42 2001
+++ postgresql-7.1beta3-sparc/src/test/regress/resultmap	Thu Jan 25 21:59:53 2001
@@ -28,6 +28,7 @@
 geometry/powerpc-unknown-linux-gnu=geometry-powerpc-linux-gnulibc1
 geometry/powerpc.*-aix4=geometry-powerpc-aix4
 geometry/sparc-sun-solaris=geometry-solaris-precision
+geometry/sparc.*-linux-gnu=geometry-solaris-precision
 geometry/alpha.*-linux-gnu=geometry-solaris-precision
 geometry/.*-beos=geometry-intel-beos
 horology/.*-aix4=horology-1947-PDT

postgresqltext/plain; CHARSET=US-ASCII; NAME=postgresqlDownload

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Ryan Kirkpatrick (#1)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

Just a few random bits and pieces that might be useful to someone
else. They are:

postgres-7-sparc.patch -> This is the same one I posted earlier to
hackers and ports, forgot to post it to patches. This fixes the resultmap
for sparc-linux so that all regression tests pass.

Already applied.

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

Added to /contrib as init.d.

pg_ctl.patch -> Minor patch required by postgresql script above.
It basically just allows you to specify the -o option (postmaster options)
more than once and all of them will be concatenated together. Makes it
easier to do '-o -F -o -i', then to try and shell escape quotes in a shell
script. Should be harmless, and might be handy.

This doesn't seem like a good idea. It makes -o handling different for
different utilities.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

Ryan Kirkpatrick

pgsql@rkirkpat.net

almost 25 years ago

In reply to: Bruce Momjian (#2)

1 attachment(s)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

On Tue, 30 Jan 2001, Bruce Momjian wrote:

postgres-7-sparc.patch -> This is the same one I posted earlier to
hackers and ports, forgot to post it to patches. This fixes the resultmap
for sparc-linux so that all regression tests pass.

Already applied.

Great! Just never saw a confirmation (may have missed it) of my
first post. Also verified it is in beta4, and all regression
tests now pass on Sparc/Linux. :)

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

Added to /contrib as init.d.

Please replace with the one attached to this email message. The
previous version depended on the below patch, this one does not. It is
also a bit cleaner and has a few more helpful comments.

pg_ctl.patch -> Minor patch required by postgresql script above.
It basically just allows you to specify the -o option (postmaster options)
more than once and all of them will be concatenated together. Makes it
easier to do '-o -F -o -i', then to try and shell escape quotes in a shell
script. Should be harmless, and might be handy.

This doesn't seem like a good idea. It makes -o handling different for
different utilities.

Hmm... Didn't know other utils used that option. Maybe use the
'--' option for pg_ctl passing postmaster options, i.e. 'pg_ctl start --
-F -i'. Of course that is a bit more major of a change to the code that I
will leave as an exercise to others. With the revised version of the
init.d script attached, the pg_ctl patch is immaterial.
Thanks and TTYL.

Thomas Lockhart

lockhart@alumni.caltech.edu

almost 25 years ago

In reply to: Ryan Kirkpatrick (#3)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.dscript....

Great! Just never saw a confirmation (may have missed it) of my
first post. Also verified it is in beta4, and all regression
tests now pass on Sparc/Linux. :)

What version of Linux? I'm updating the ports listing...

- Thomas

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Ryan Kirkpatrick (#3)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

Applied.

On Tue, 30 Jan 2001, Bruce Momjian wrote:

postgres-7-sparc.patch -> This is the same one I posted earlier to
hackers and ports, forgot to post it to patches. This fixes the resultmap
for sparc-linux so that all regression tests pass.

Already applied.

Great! Just never saw a confirmation (may have missed it) of my
first post. Also verified it is in beta4, and all regression
tests now pass on Sparc/Linux. :)

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

Added to /contrib as init.d.

Please replace with the one attached to this email message. The
previous version depended on the below patch, this one does not. It is
also a bit cleaner and has a few more helpful comments.

pg_ctl.patch -> Minor patch required by postgresql script above.
It basically just allows you to specify the -o option (postmaster options)
more than once and all of them will be concatenated together. Makes it
easier to do '-o -F -o -i', then to try and shell escape quotes in a shell
script. Should be harmless, and might be handy.

This doesn't seem like a good idea. It makes -o handling different for
different utilities.

Hmm... Didn't know other utils used that option. Maybe use the
'--' option for pg_ctl passing postmaster options, i.e. 'pg_ctl start --
-F -i'. Of course that is a bit more major of a change to the code that I
will leave as an exercise to others. With the revised version of the
init.d script attached, the pg_ctl patch is immaterial.
Thanks and TTYL.

---------------------------------------------------------------------------
| "For to me to live is Christ, and to die is gain." |
| --- Philippians 1:21 (KJV) |
---------------------------------------------------------------------------
| Ryan Kirkpatrick | Boulder, Colorado | http://www.rkirkpat.net/ |
---------------------------------------------------------------------------

Content-Description:

[ Attachment, skipping... ]

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

Ryan Kirkpatrick

pgsql@rkirkpat.net

almost 25 years ago

In reply to: Thomas Lockhart (#4)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.dscript....

On Thu, 1 Feb 2001, Thomas Lockhart wrote:

Great! Just never saw a confirmation (may have missed it) of my
first post. Also verified it is in beta4, and all regression
tests now pass on Sparc/Linux. :)

What version of Linux? I'm updating the ports listing...

Debian GNU/Linux 2.2, with 2.2.15 (stock kernel for this
distribution). For a bit more details, I tested it on a Sparc 20 w/64MB
of RAM, running the stock gcc 2.95.2 and glibc 2.1.3.
Is that sufficient? :)

Noname

doj@wwws2.redaex.de

almost 25 years ago

In reply to: Thomas Lockhart (#4)

1 attachment(s)

julian day function

Hello Developers,

I have written a function to calculate the julian day from any gregorian day.
You can include this code in the contrib directory or whatever place seems
applicable in future releases of posgresql.

See the attachment for code and examples.

--
--
---> doj@redaex.de

Thomas Lockhart

lockhart@alumni.caltech.edu

almost 25 years ago

In reply to: Ryan Kirkpatrick (#6)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.dscript....

Is that sufficient? :)

Yes ;)

- Thomas

Thomas Lockhart

lockhart@alumni.caltech.edu

almost 25 years ago

In reply to: Ryan Kirkpatrick (#3)

Re: julian day function

I have written a function to calculate the julian day from any gregorian day.
You can include this code in the contrib directory or whatever place seems
applicable in future releases of posgresql.

Looks nice!

There is already a j2date()/date2j() subroutine pair in the backend, to
support all of the existing date/time arithmetic. Would you be willing
to use calls to that? Then we wouldn't have two sets of code to maintain
(not that JD calculations change very often ;)

It would likely be helpful to have a, say, "julian" argument to
date_part(), which I'd be happy to add in. There is already access to
Julian date using the to_char(date,'J') function call, and you can get a
numeric type return using

cast(to_char(date,'J') as int)

- Thomas

#10

Ryan Kirkpatrick

pgsql@rkirkpat.net

almost 25 years ago

In reply to: Bruce Momjian (#5)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

On Thu, 1 Feb 2001, Bruce Momjian wrote:

Applied.

On Tue, 30 Jan 2001, Bruce Momjian wrote:

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

Added to /contrib as init.d.

Please replace with the one attached to this email message. The
previous version depended on the below patch, this one does not. It is
also a bit cleaner and has a few more helpful comments.

Thanks for the quick response. Just as nit-pick the
./contrib/README lists ./contirb/init.d as a starting/stopping script as
for 'non-Linux' systems.... Uh... I meant it as non-distribution specific,
but still probably linux specific. Though it is so general that any unix
system that has bash (might even world under sh, never tested) that uses
init.d style scripts should work. Maybe state it as a 'generic unix
init.d script for starting/stopping pgsql....'? Thanks.

#11

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Ryan Kirkpatrick (#10)

Re: A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

New text:

Init.d-style scripts for starting and stopping the PostgreSQL
server by Ryan Kirkpatrick <pgsql@rkirkpat.net

Thanks.

On Thu, 1 Feb 2001, Bruce Momjian wrote:

Applied.

On Tue, 30 Jan 2001, Bruce Momjian wrote:

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

Added to /contrib as init.d.

Please replace with the one attached to this email message. The
previous version depended on the below patch, this one does not. It is
also a bit cleaner and has a few more helpful comments.

Thanks for the quick response. Just as nit-pick the
./contrib/README lists ./contirb/init.d as a starting/stopping script as
for 'non-Linux' systems.... Uh... I meant it as non-distribution specific,
but still probably linux specific. Though it is so general that any unix
system that has bash (might even world under sh, never tested) that uses
init.d style scripts should work. Maybe state it as a 'generic unix
init.d script for starting/stopping pgsql....'? Thanks.

---------------------------------------------------------------------------
| "For to me to live is Christ, and to die is gain." |
| --- Philippians 1:21 (KJV) |
---------------------------------------------------------------------------
| Ryan Kirkpatrick | Boulder, Colorado | http://www.rkirkpat.net/ |
---------------------------------------------------------------------------

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

#12

Peter Eisentraut

peter_e@gmx.net

almost 25 years ago

In reply to: Ryan Kirkpatrick (#1)

Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

Ryan Kirkpatrick writes:

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

I don't see how this can be more independent if it uses

DAEMON=/home/postgres/bin/pg_ctl

and

LOG="/usr/local/pgsql/server.log"

Nor is

su - postgres sh -c "$DAEMON stop >& /dev/null"

an appropriate way to stop the server in a system shutdown situation.

IMHO, rather than accumulating a bunch of versions that somebody liked
better than the existing ones, why not provide actual scripts from actual
distributions? Generic scripts will just lead to generic problems.

--
Peter Eisentraut peter_e@gmx.net http://yi.org/peter-e/

#13

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Peter Eisentraut (#12)

Re: Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

LOG="/usr/local/pgsql/server.log"

Nor is

su - postgres sh -c "$DAEMON stop >& /dev/null"

Also, >& is not standard either.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

#14

Karel Zak

zakkr@zf.jcu.cz

almost 25 years ago

In reply to: Thomas Lockhart (#9)

Re: julian day function

On Fri, 2 Feb 2001, Thomas Lockhart wrote:

I have written a function to calculate the julian day from any gregorian day.
You can include this code in the contrib directory or whatever place seems
applicable in future releases of posgresql.

Looks nice!

There is already a j2date()/date2j() subroutine pair in the backend, to
support all of the existing date/time arithmetic. Would you be willing
to use calls to that? Then we wouldn't have two sets of code to maintain
(not that JD calculations change very often ;)

It would likely be helpful to have a, say, "julian" argument to
date_part(), which I'd be happy to add in. There is already access to
Julian date using the to_char(date,'J') function call, and you can get a
numeric type return using

cast(to_char(date,'J') as int)

... and vice versa:

test=# select to_timestamp('2451946', 'J');
to_timestamp
------------------------
2001-02-05 00:00:00+01
(1 row)

Karel

#15

Ryan Kirkpatrick

ryan@calibration.com

almost 25 years ago

In reply to: Peter Eisentraut (#12)

Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

On Sun, 4 Feb 2001, Peter Eisentraut wrote:

Ryan Kirkpatrick writes:

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

I don't see how this can be more independent if it uses

DAEMON=/home/postgres/bin/pg_ctl

Ooops.... That is my mistake... Should have been
/usr/local/pgsql/bin/pg_ctl. I have /usr/local/pgsql/ symlinked to /home
(where there is more, faster disk space). I can submit a patch, or can
some one just fix it?

LOG="/usr/local/pgsql/server.log"

What is wrong with that? There really is no standard on where to
put the log file, so it is either here or /var/log.

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

an appropriate way to stop the server in a system shutdown situation.

Uh... Isn't that the way you are supposed to stop it? pg_ctl stop?

IMHO, rather than accumulating a bunch of versions that somebody liked
better than the existing ones, why not provide actual scripts from actual
distributions? Generic scripts will just lead to generic problems.

Fine by me... I just put mine up as a suggestion, as the only
other one in contrib was very RH specific. The one included with the
debian package is very debian specific (and over complex in IMHO). I just
submitted mine as a possible generic Linux version that should work with
bash on most installation. If some one has a better idea, then by all
means post it.
Also, from the way I understand contrib, they are user submissions
and are in no way assured to work on any other machine than the user's who
submitted it. I take it as given anything in contrib might need a bit of
editing to fit my needs.
Anyway, just my response as one who has used Slackware, RedHat,
and now Debian over the years, but who in no way claims to be an
uber-Linux or Unix hacker or sysadmin. :)

PS. I do play a part time Linux sysadmin in real though...

#16

Ryan Kirkpatrick

pgsql@rkirkpat.net

almost 25 years ago

In reply to: Ryan Kirkpatrick (#15)

Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

On Sun, 4 Feb 2001, Peter Eisentraut wrote:

Ryan Kirkpatrick writes:

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

I don't see how this can be more independent if it uses

DAEMON=/home/postgres/bin/pg_ctl

LOG="/usr/local/pgsql/server.log"

What is wrong with that? There really is no standard on where to
put the log file, so it is either here or /var/log.

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

an appropriate way to stop the server in a system shutdown situation.

Uh... Isn't that the way you are supposed to stop it? pg_ctl stop?

IMHO, rather than accumulating a bunch of versions that somebody liked
better than the existing ones, why not provide actual scripts from actual
distributions? Generic scripts will just lead to generic problems.

PS. I do play a part time Linux sysadmin in real though...

Import Notes

Resolved by subject fallback

#17

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Ryan Kirkpatrick (#15)

Re: Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

On Sun, 4 Feb 2001, Peter Eisentraut wrote:

Ryan Kirkpatrick writes:

postgresql -> This is a Linux distribution independent (or so I
hope) init.d/rc.d script that makes use of pg_ctl. There is currently a
few in ./contrib/linux of the pgsql source tree, but they are RedHat
specific. This one is simple and self contained. Might be worth adding to
the other scripts.

I don't see how this can be more independent if it uses

DAEMON=/home/postgres/bin/pg_ctl

Ooops.... That is my mistake... Should have been
/usr/local/pgsql/bin/pg_ctl. I have /usr/local/pgsql/ symlinked to /home
(where there is more, faster disk space). I can submit a patch, or can
some one just fix it?

Change made.

LOG="/usr/local/pgsql/server.log"

What is wrong with that? There really is no standard on where to
put the log file, so it is either here or /var/log.

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

Change made.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

#18

Florent Guillaume

efgeor@noos.fr

almost 25 years ago

In reply to: Bruce Momjian (#17)

Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

It won't do what you want. You want '>/dev/null 2>&1'.

Change made.

Hmmm, I don't see this change in cvsweb.

Florent

--
florent.guillaume@mail.com

#19

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Florent Guillaume (#18)

Re: Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

It won't do what you want. You want '>/dev/null 2>&1'.

Yes, I knew he wanted >/dev/null 2>&1. I just fixed it.

Change made.

Hmmm, I don't see this change in cvsweb.

Thanks. Seems I overwrote it with his new version. Done now.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

#20

Ryan Kirkpatrick

pgsql@rkirkpat.net

almost 25 years ago

In reply to: Bruce Momjian (#17)

1 attachment(s)

Re: Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

On Tue, 6 Feb 2001, Bruce Momjian wrote:

DAEMON=/home/postgres/bin/pg_ctl

Ooops.... That is my mistake... Should have been
/usr/local/pgsql/bin/pg_ctl. I have /usr/local/pgsql/ symlinked to /home
(where there is more, faster disk space). I can submit a patch, or can
some one just fix it?

Change made.

Actually I found a few more places where I used /home/postgres
instead of /usr/local/pgsql. To make things clear (and for reasons
below) I have attached a new version of the file.

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

Change made.

Turns out that it has to be '> {dest} 2>&1' for it to work, at
least with bash. So, to reduce confusion on fixes to this file, as I
stated above, a new version is attached. It is tested to work and I don't
see any thing more that is non-standard. Though if you still see problems,
feel free to point them out. :) Thanks and TTYL.

#21

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Ryan Kirkpatrick (#20)

Re: Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

Updated.

On Tue, 6 Feb 2001, Bruce Momjian wrote:

DAEMON=/home/postgres/bin/pg_ctl

Ooops.... That is my mistake... Should have been
/usr/local/pgsql/bin/pg_ctl. I have /usr/local/pgsql/ symlinked to /home
(where there is more, faster disk space). I can submit a patch, or can
some one just fix it?

Change made.

Actually I found a few more places where I used /home/postgres
instead of /usr/local/pgsql. To make things clear (and for reasons
below) I have attached a new version of the file.

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

Change made.

Turns out that it has to be '> {dest} 2>&1' for it to work, at
least with bash. So, to reduce confusion on fixes to this file, as I
stated above, a new version is attached. It is tested to work and I don't
see any thing more that is non-standard. Though if you still see problems,
feel free to point them out. :) Thanks and TTYL.

---------------------------------------------------------------------------
| "For to me to live is Christ, and to die is gain." |
| --- Philippians 1:21 (KJV) |
---------------------------------------------------------------------------
| Ryan Kirkpatrick | Boulder, Colorado | http://www.rkirkpat.net/ |
---------------------------------------------------------------------------

Content-Description:

[ Attachment, skipping... ]

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026

#22

Bruce Momjian

pgman@candle.pha.pa.us

almost 25 years ago

In reply to: Ryan Kirkpatrick (#20)

Re: Re: [PATCHES] A Sparc/Linux patch (for 7.1), and a Linux rc.d/init.d script....

Applied. Thanks.

On Tue, 6 Feb 2001, Bruce Momjian wrote:

DAEMON=/home/postgres/bin/pg_ctl

Ooops.... That is my mistake... Should have been
/usr/local/pgsql/bin/pg_ctl. I have /usr/local/pgsql/ symlinked to /home
(where there is more, faster disk space). I can submit a patch, or can
some one just fix it?

Change made.

Actually I found a few more places where I used /home/postgres
instead of /usr/local/pgsql. To make things clear (and for reasons
below) I have attached a new version of the file.

su - postgres sh -c "$DAEMON stop >& /dev/null"

Hmm... What is wrong here, besides the '>&'? The '>&' can be
replaced with '2>&1 >' if that is more standard.

Change made.

Turns out that it has to be '> {dest} 2>&1' for it to work, at
least with bash. So, to reduce confusion on fixes to this file, as I
stated above, a new version is attached. It is tested to work and I don't
see any thing more that is non-standard. Though if you still see problems,
feel free to point them out. :) Thanks and TTYL.

---------------------------------------------------------------------------
| "For to me to live is Christ, and to die is gain." |
| --- Philippians 1:21 (KJV) |
---------------------------------------------------------------------------
| Ryan Kirkpatrick | Boulder, Colorado | http://www.rkirkpat.net/ |
---------------------------------------------------------------------------

Content-Description:

[ Attachment, skipping... ]

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026