Re: [PATCH v2 01/10] t: add helper to convert object IDs to paths
From: Johannes Schindelin <hidden>
Date: 2019-06-17 19:05:16
From: Johannes Schindelin <hidden>
Date: 2019-06-17 19:05:16
Hi brian, On Sun, 16 Jun 2019, brian m. carlson wrote:
diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh index 8270de74be..11a6abca2e 100644 --- a/t/test-lib-functions.sh +++ b/t/test-lib-functions.sh@@ -1314,6 +1314,12 @@ test_oid () { eval "printf '%s' \"\${$var}\"" } +# Insert a slash into an object ID so it can be used to reference a location +# under ".git/objects". For example, "deadbeef..." becomes "de/adbeef..". +test_oid_to_path () { + echo "$1" | sed -e 's!^..!&/!' +}
I guess it does not *really* matter all that much, but this does spawn a
new process (and I think it actually spawns 4 on Windows, for reasons, and
spawning processes is super expensive on Windows).
We might actually want to think about using something like this instead
(which admittedly looks a bit like gobbledygook to the uninitiated, but it
definitely avoids any spawned process):
test_oid_to_path () {
echo "${1%${1#??}}/${1#??}"
}
Ciao,
Dscho