Re: bug in copy-directory

[Michael Albinus]

Thierry Volpiatto <address@hidden> writes:

> Stefan Monnier <address@hidden> writes:
>
>>> In the shell, of course, you just do "cp -r /foo/* /bar".  That's
>>> because cp acts on both files and directories; copy-directory only
>>> copies directories.  This implies that giving copy-directory cp-like
>>> semantics might be a mistake.
>>
>> OK.
> If you want to keep the ability to do "cp -r /foo/* /bar" you can just
> add a new arg copy-contents-only (or whatever name) to copy-directory:
>
> (defun copy-directory (directory newname &optional keep-time parents 
> copy-contents-only)

I'm in favor of this proposal (compared with the recently introduced
`copy-as-subdir'). Thierry's proposal keeps the semantics of
`copy-directory', and the changed semantics are activated by the
optional argument.

> NOTE:
> - this feature would not work with tramp names without changes to
>   tramp.
>   However (copy-directory "/foo" "/existing/directory/bar")
>   will work without changes.

Changes in Tramp would be simple.

Best regards, Michael.