todo/remove remaining perl usageetckeeperhttp://etckeeper.branchable.com/todo/remove_remaining_perl_usage/etckeeperikiwiki2021-10-17T13:52:28Zcomment 1http://etckeeper.branchable.com/todo/remove_remaining_perl_usage/comment_1_c901ff31a0cddf4912123c6443876327/joey2021-10-17T13:52:28Z2015-06-10T19:50:30Z
<p>Reviewing this patch, I noticed a bug in the first hunk. lsscripts
currently limits the files it finds to those matching a regexp. IIRC, this
is needed, amoung other things, to avoid .dpkg-old files being run.</p>
<p>Looking at the second hunk, I suspect that calling <code>id</code> twice per file is
going to be significantly slower than the perl implementation.</p>
<p>There also seems to be missing escaping of single quotes in filenames in
the second hunk.</p>