Re: Use of 'cp -r' in CREATE DATABASE

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Postgresql Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Use of 'cp -r' in CREATE DATABASE
Date: 2003-12-12 15:39:24
Message-ID: 3FD9E12C.8050501@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:

>Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
>
>
>>but my BSD/OS manual only documents 'cp -R' and mentions:
>>I think we should switch to -R in our code.
>>
>>
>
>And break the code on who knows how many other systems? No thanks.
>
>If we want to do anything at all with this code, we should eliminate the
>use of system("cp") entirely in favor of doing the recursive copy logic
>ourselves. We already have the beginnings of same in the Windows port,
>and I think we'll be forced down that path anyway for tablespaces.
>
>
>

That seems cleaner to me.

IIRC we don't copy anything but plain files and directories - no special
files, symlinks or fifos, so the -R/-r differences shouldn't affect us
anyway, should they? Also, that should make the implementation of an
internal recursive copy much simpler - far fewer cases to consider.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2003-12-12 17:04:15 Re: Use of 'cp -r' in CREATE DATABASE
Previous Message Bruce Momjian 2003-12-12 15:07:12 Re: pljava revisited