Skip to content

Commit

Permalink
t: make PIPE a standard test prerequisite
Browse files Browse the repository at this point in the history
The 'PIPE' test prerequisite was already defined identically by t9010
and t9300, therefore it makes sense to make it a predefined
prerequisite.

Signed-off-by: Adam Spiers <[email protected]>
Signed-off-by: Junio C Hamano <[email protected]>
  • Loading branch information
aspiers authored and gitster committed Apr 12, 2013
1 parent 59a7714 commit 2007327
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 16 deletions.
5 changes: 5 additions & 0 deletions t/README
Original file line number Diff line number Diff line change
Expand Up @@ -613,6 +613,11 @@ use these, and "test_set_prereq" for how to define your own.
The process retains the same pid across exec(2). See fb9a2bea for
details.

- PIPE

The filesystem we're on supports creation of FIFOs (named pipes)
via mkfifo(1).

- SYMLINKS

The filesystem we're on supports symbolic links. E.g. a FAT
Expand Down
8 changes: 0 additions & 8 deletions t/t9010-svn-fe.sh
Original file line number Diff line number Diff line change
Expand Up @@ -54,14 +54,6 @@ text_no_props () {

>empty

test_expect_success 'setup: have pipes?' '
rm -f frob &&
if mkfifo frob
then
test_set_prereq PIPE
fi
'

test_expect_success PIPE 'empty dump' '
reinit_git &&
echo "SVN-fs-dump-format-version: 2" >input &&
Expand Down
8 changes: 0 additions & 8 deletions t/t9300-fast-import.sh
Original file line number Diff line number Diff line change
Expand Up @@ -49,14 +49,6 @@ echo "$@"'

>empty

test_expect_success 'setup: have pipes?' '
rm -f frob &&
if mkfifo frob
then
test_set_prereq PIPE
fi
'

###
### series A
###
Expand Down
5 changes: 5 additions & 0 deletions t/test-lib.sh
Original file line number Diff line number Diff line change
Expand Up @@ -727,6 +727,11 @@ test_i18ngrep () {
fi
}

test_lazy_prereq PIPE '
# test whether the filesystem supports FIFOs
rm -f testfifo && mkfifo testfifo
'

test_lazy_prereq SYMLINKS '
# test whether the filesystem supports symbolic links
ln -s x y && test -h y
Expand Down

0 comments on commit 2007327

Please sign in to comment.