Re: [PATCH 1/3] Call extended-semantics commands through variables.
From: Jason Riedy <hidden>
Date: 2016-06-15 22:42:18
And Junio C Hamano writes: - The use of FIND or CPIO in git clone does not need -0 (and the - code does not use -0, nor your patch adds -0), so you should not - have to override them this way. I'm not sure what's up with cpio, but git causes pkgsrc's default cpio to segfault on my Solaris machines. It's easier to point CPIO at a different cpio than debug a utility I've never really used. ;) I thought it was git-clone breaking in the tests, but it could have been git-merge. I'll check again when I get a chance. And I worry about using different programs in different scripts, so I just changed all of them. - (BTW, you got count-objects one wrong; there is a leftover - GIT_FIND there). Friday afternoon patching, sorry. That also means either that count-objects has no test cases or that branch of it is not exercised by tests. - The places we _do_ use -0 currently should be converted - with something like your patch to use -0 capable version of the - tool. Again, I'm not very comfortable using different finds or xargs in different places. But if you want, I'll re-do the patch with just those locations changed, and I'll double- check which cpio invocation is breaking. - [...] but nobody should be using rsync transfer anyway, - so... Is there a better way of grabbing all the tags now? I haven't kept track, as I haven't had to do that in a while. Jason