> > > > The postgresql-?.?.?/contrib/linux/postgres.init is meant to start your
> > > > postmaster at boot time and stop it at halt/reboot. Excelent.
> > > > But it is made for postgres account running tcsh. I know nothing about tchs
> > > > and my postgres account defaults to bash. So (thanks to Steve "Stevers!"
> > > > Coile) I changed it to bash:
> >
> > OK, but _I_ don't run bash. So someone else is now maintaining this
> > file? Why didn't we keep both forms in the file, with one commented out?
> > What are we trying to accomplish here??
>
> bash/sh is the standard, expecially because the top of the file has
> #!/bin/sh.
Sorry, the top of the init script and the postgres shell aren't related.
The init script is run at startup out of the root account. The line
which was changed is run under the postgres account, which can have a
different shell from the root shell.
imho it is not a step forward to break some code to help others. How
about asking the person submitting the patch to document what they did
differently? The original code at least had a line saying what had to be
changed; the patch doesn't fix the comment or suggest how to support an
alternate shell.
Or, have two files in contrib, postgres.init.tcsh and postgres.init. Or
at least something which is a step forward rather than sideways...
- Tom
