Thomas Rast <trast@xxxxxxxxxxxxxxx> writes: > [dropped Dieter as this really goes off on an internal tangent] > > Junio C Hamano <gitster@xxxxxxxxx> writes: > >> If somebody is writing a script using "git add" (which is not recommended >> to begin with) > > Can we still stick to that stance? Our tests are increasingly using > 'git add' instead of 'git update-index --add': > > $ git grep 'git[ -]add' t/ | wc -l > 1540 > $ git grep 'git[ -]update-index --add' t/ | wc -l > 269 > $ git grep 'git[ -]update-index --add' v1.6.0 t/ | wc -l > 251 > $ git grep 'git[ -]add' v1.6.0 t/ | wc -l > 705 Stop being silly. Have you actually looked at these usage? Some of them are genuinely testing if "git add" works correctly, so it is out of the scope of this discussion, but others that could be "git update-index" are feeding the paths known to the script to exist (and we want 'git add' to error out if that is not the case). More generally, scripts in t/ directories are "scripts", but it is totally different from the kind of "user facing script that behaves as if it is a complete command, taking its own command line arguments, passing them through to the underlying plumbing commands". -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html