[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#68504: [PATCH] Add copy-on-write support to scm_copy_file.
From: |
Tomas Volf |
Subject: |
bug#68504: [PATCH] Add copy-on-write support to scm_copy_file. |
Date: |
Wed, 24 Jan 2024 20:19:51 +0100 |
On 2024-01-24 11:26:56 +0100, Ludovic Courtès wrote:
>
> The patch looks great (and very useful) to me, modulo one issue:
>
> > -SCM_API SCM scm_copy_file (SCM oldfile, SCM newfile);
> > +SCM_API SCM scm_copy_file (SCM oldfile, SCM newfile, SCM rest);
>
> Since this is a public interface, we cannot change this function’s
> signature during the 3.0 stable series.
>
> Thus, I would suggest keeping the public ‘scm_copy_file’ unchanged and
> internally having a three-argument variant. The Scheme-level
> ‘copy-file’ would map to that three-argument variant. (See how
> ‘scm_pipe’ and ‘scm accept’ as examples.)
That is a very good point, which I did not realize at all. Thanks to the
examples you provided, it was not that hard to do (well, assuming I did it
right).
> Could you send an updated patch?
Done. However now that I read it after myself, I overlooked this occurrence of
scm_copy_file in the commit message:
This commit adds support for this feature into our
copy-file (scm_copy_file) procedure. Same as `cp', it defaults to
So I just sent v3 right after v2, sorry for the noise, should have been more
careful.
>
> BTW, copyright assignment to the FSF is now optional but encouraged.
> Please see
> <https://lists.gnu.org/archive/html/guile-devel/2022-10/msg00008.html>.
Since it is optional, I will currently opt into not doing the assignment, I do
not like the concept that much. I will try to find time to actually form an
opinion based on facts.
Have a nice day,
Tomas
--
There are only two hard things in Computer Science:
cache invalidation, naming things and off-by-one errors.
signature.asc
Description: PGP signature