Re: Use of 'cp -r' in CREATE DATABASE - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: Use of 'cp -r' in CREATE DATABASE
Date
Msg-id 3FD9E12C.8050501@dunslane.net
Whole thread Raw
In response to Re: Use of 'cp -r' in CREATE DATABASE  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Use of 'cp -r' in CREATE DATABASE
List pgsql-hackers
Tom Lane wrote:

>Bruce Momjian <pgman@candle.pha.pa.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



pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: pljava revisited
Next
From: Tom Lane
Date:
Subject: Re: Use of 'cp -r' in CREATE DATABASE