Re: [PATCH] Use file(COPY) to copy test data instead of spawning a process

Ivan Zhakov <[email protected]> Tue, 12 May 2026 20:18:31 +0200
Newsgroups gmane.comp.apache.apr.devel
Message-ID <CAPZho08sPLewekT1w=Z3Ssx-WQ68TyPC3xFjL=SDpmBO8=x8Yw@mail.gmail.com>
--000000000000a454030651a2e4b5
Content-Type: text/plain; charset="UTF-8"

On Tue, 12 May 2026 at 18:37, Timofei Zhakov <[email protected]> wrote:

> I think this syntax is much simpler, more convenient, and looks
> prettier. According to the documentation [1] cmake will do its best at
> avoiding copying the file twice if the same exists with matching
> timestamps. Not using cmake executable to copy files will also slightly
> improve configuration time because it removes invoking extra tools, if it's
> even remotely close to being anywhere significant.
>
> Committed in r1934148.

Thanks!

-- 
Ivan Zhakov

--000000000000a454030651a2e4b5
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">On Tue, 12 May 2026 at 18:37, Timofei Zha=
kov &lt;<a href=3D"mailto:[email protected]">[email protected]</a>&gt; wrot=
e:</div><div class=3D"gmail_quote gmail_quote_container"><blockquote class=
=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rg=
b(204,204,204);padding-left:1ex"><div dir=3D"ltr"><div>I think this syntax =
is much simpler, more convenient,=C2=A0and looks prettier.=C2=A0According=
=C2=A0to the documentation=C2=A0[1] cmake will do its best at avoiding copy=
ing the file twice if the same exists with matching timestamps. Not using c=
make executable to copy files will also slightly improve configuration time=
 because it removes invoking extra tools, if it&#39;s even remotely close t=
o being anywhere significant.</div><div><br></div></div></blockquote><div>C=
ommitted in r1934148.</div><div><br></div><div>Thanks!</div></div><br><span=
 class=3D"gmail_signature_prefix">-- </span><br><div dir=3D"ltr" class=3D"g=
mail_signature">Ivan Zhakov</div></div>

--000000000000a454030651a2e4b5--