Skip to content

Commit bac2f24

Browse files
committed
tests: only override sort & find if there are usable ones in /usr/bin/
The idea is to allow running the test suite on MinGit with BusyBox installed in /mingw64/bin/sh.exe. In that case, we will want to exclude sort & find (and other Unix utilities) from being bundled. Signed-off-by: Johannes Schindelin <[email protected]>
1 parent 389b904 commit bac2f24

File tree

2 files changed

+28
-14
lines changed

2 files changed

+28
-14
lines changed

git-sh-setup.sh

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -332,13 +332,20 @@ create_virtual_base() {
332332
# Platform specific tweaks to work around some commands
333333
case $(uname -s) in
334334
*MINGW*)
335-
# Windows has its own (incompatible) sort and find
336-
sort () {
337-
/usr/bin/sort "$@"
338-
}
339-
find () {
340-
/usr/bin/find "$@"
341-
}
335+
if test -x /usr/bin/sort
336+
then
337+
# Windows has its own (incompatible) sort; override
338+
sort () {
339+
/usr/bin/sort "$@"
340+
}
341+
fi
342+
if test -x /usr/bin/find
343+
then
344+
# Windows has its own (incompatible) find; override
345+
find () {
346+
/usr/bin/find "$@"
347+
}
348+
fi
342349
# git sees Windows-style pwd
343350
pwd () {
344351
builtin pwd -W

t/test-lib.sh

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1343,13 +1343,20 @@ yes () {
13431343
uname_s=$(uname -s)
13441344
case $uname_s in
13451345
*MINGW*)
1346-
# Windows has its own (incompatible) sort and find
1347-
sort () {
1348-
/usr/bin/sort "$@"
1349-
}
1350-
find () {
1351-
/usr/bin/find "$@"
1352-
}
1346+
if test -x /usr/bin/sort
1347+
then
1348+
# Windows has its own (incompatible) sort; override
1349+
sort () {
1350+
/usr/bin/sort "$@"
1351+
}
1352+
fi
1353+
if test -x /usr/bin/find
1354+
then
1355+
# Windows has its own (incompatible) find; override
1356+
find () {
1357+
/usr/bin/find "$@"
1358+
}
1359+
fi
13531360
# git sees Windows-style pwd
13541361
pwd () {
13551362
builtin pwd -W

0 commit comments

Comments
 (0)