tag 386148 pending
thanks

Hi Bastian,

Bastian Kleineidam [2006-09-05 18:44 +0200]:
> --- /usr/bin/pg_createcluster.orig    2006-09-05 12:14:16.000000000 +0200
> +++ /usr/bin/pg_createcluster 2006-09-05 16:31:47.000000000 +0200
> @@ -304,6 +304,15 @@
>  setup_pg_hba if $newcluster;
>  
>  # configure socket directory
> +if ($socketdir) {
> +    if (-e $socketdir) {
> +        chmod 0750, $socketdir;
> +        chown $owneruid, $ownergid, $socketdir;
> +    }
> +    elsif (system 'install', '-d', '-m', '0750', '-o', $owneruid, '-g', 
> $ownergid, $socketdir) {
> +        error 'could not create socket directory';
> +    }
> +}

I changed this to not touch permissions of an existing directory.
Changing permissions of e. g. /tmp would have grave results on the
system, and in general I feel that existing directory permissions
should be honoured. I also changed the permissions to 0755.


> +If a custom socket directory is given and it does not exist, it is created.
> +The permissions are adjusted to allow write access to the cluster owner and
> +group.

POD updated accordingly with above adjustments.

> -=item B<-e> I<encoding>, B<--encoding> I<encoding>
> +=item B<-e> I<encoding>, B<--encoding=>I<encoding>
> [and similar cases]

Committed.

> --- /usr/bin/pg_dropcluster.orig      2006-09-05 12:29:12.000000000 +0200
> +++ /usr/bin/pg_dropcluster   2006-09-05 16:31:47.000000000 +0200

Committed this change, too.

Thank you!

Martin

-- 
Martin Pitt        http://www.piware.de
Ubuntu Developer   http://www.ubuntu.com
Debian Developer   http://www.debian.org

In a world without walls and fences, who needs Windows and Gates?

Attachment: signature.asc
Description: Digital signature

Reply via email to