From 48f5ea4b37157040edba01e098a81c7bb78cbebe Mon Sep 17 00:00:00 2001 From: David Majda Date: Fri, 15 Jan 2016 15:30:51 +0100 Subject: [PATCH] tools/impact: Tweak the uname call It turns out that OS X doesn't support long options for uname and it doesn't support -o/--operating-system at all. Let's tweak uname's options into something POSIX-compatible which still gives reasonable results. The new "uname -mrs" call results in the following: OS uname -mrs ----------------------------------------------- OS X Mavericks Darwin 15.2.0 x86_64 Ubuntu 14.04 Linux 3.13.0-32-generic x86_64 --- tools/impact | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/impact b/tools/impact index 9848b26..da2b0e9 100755 --- a/tools/impact +++ b/tools/impact @@ -61,7 +61,7 @@ print_results() { echo - echo "(Measured by /tools/impact with Node.js $(node --version) on $(uname --operating-system --machine).)" + echo "(Measured by /tools/impact with Node.js $(node --version) on $(uname -mrs).)" } print_usage() {