Re: [PATCH] allow TTY tests to run under recent Mac OS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/13/2014 11:01 PM, Mike Blume wrote:
listed bug doesn't reproduce on Mac OS Yosemite. For now, just enable
TTY on Yosemite and higher

Signed-off-by: Mike Blume <blume.mike@xxxxxxxxx>
---
  t/lib-terminal.sh | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/t/lib-terminal.sh b/t/lib-terminal.sh
index 5184549..1311ce0 100644
--- a/t/lib-terminal.sh
+++ b/t/lib-terminal.sh
@@ -29,7 +29,10 @@ test_lazy_prereq TTY '
  	# After 2000 iterations or so it hangs.
  	# https://rt.cpan.org/Ticket/Display.html?id=65692
  	#
-	test "$(uname -s)" != Darwin &&
+	# Under Mac OS X 10.10.1 and Perl 5.18.2, this problem
+	# appears to be gone.
+	#
+	[[ test "$(uname -s)" != Darwin || test "$(uname -r | cut -d. -f1") -ge 14 ]] &&
This seems to be bash syntax (the "[[" can (and should) be easily avoided)

Another question:
Is this related to perl or Mac OS ?
The commit message is indicating that the combination of Mac OS 10.10.1 and perl 5.18.2
is working, but the code does not reflect this.
Does it make sense to test for the perl version in the code?
Or is it OK to mention

 Under Mac OS X 10.10.1 which ships with Perl 5.18.2, this problem....



perl "$TEST_DIRECTORY"/test-terminal.perl \
  		sh -c "test -t 1 && test -t 2"

--
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




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]