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 <<a href=3D"mailto:[email protected]">[email protected]</a>> 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'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--