From commits-return-7108-archive-asf-public=cust-asf.ponee.io@kudu.apache.org Fri Mar 8 19:44:47 2019 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by mx-eu-01.ponee.io (Postfix) with SMTP id 3C5AF18067C for ; Fri, 8 Mar 2019 20:44:47 +0100 (CET) Received: (qmail 92279 invoked by uid 500); 8 Mar 2019 19:44:46 -0000 Mailing-List: contact commits-help@kudu.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@kudu.apache.org Delivered-To: mailing list commits@kudu.apache.org Received: (qmail 92253 invoked by uid 99); 8 Mar 2019 19:44:46 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 08 Mar 2019 19:44:46 +0000 Received: by gitbox.apache.org (ASF Mail Server at gitbox.apache.org, from userid 33) id C936B87989; Fri, 8 Mar 2019 19:44:45 +0000 (UTC) Date: Fri, 08 Mar 2019 19:44:47 +0000 To: "commits@kudu.apache.org" Subject: [kudu] 02/02: [gutil] fix build on macOS MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit From: alexey@apache.org In-Reply-To: <155207428573.32211.1803887392054890835@gitbox.apache.org> References: <155207428573.32211.1803887392054890835@gitbox.apache.org> X-Git-Host: gitbox.apache.org X-Git-Repo: kudu X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Rev: 701f2f4fc9c0ce002a3f72fcdc537610be4e7ac5 X-Git-NotificationType: diff X-Git-Multimail-Version: 1.5.dev Auto-Submitted: auto-generated Message-Id: <20190308194445.C936B87989@gitbox.apache.org> This is an automated email from the ASF dual-hosted git repository. alexey pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/kudu.git commit 701f2f4fc9c0ce002a3f72fcdc537610be4e7ac5 Author: Alexey Serbin AuthorDate: Fri Mar 8 10:44:10 2019 -0800 [gutil] fix build on macOS The newly-added sysinfo-test consists of scenarios for Linux-specific ParseMaxCpuIndex() function. This function is not implemented on macOS, so the sysinfo-test in its current state makes sense only for Linux Kudu builds. This is a follow-up to feebbb2649a659bbd6b2fdcfc6a347eafbccba2b. Change-Id: I3c954b66534690b2c50e86d15f9c9be9d4ce3ed7 Reviewed-on: http://gerrit.cloudera.org:8080/12703 Reviewed-by: Will Berkeley Tested-by: Kudu Jenkins --- src/kudu/gutil/CMakeLists.txt | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/kudu/gutil/CMakeLists.txt b/src/kudu/gutil/CMakeLists.txt index 5569650..f24a41e 100644 --- a/src/kudu/gutil/CMakeLists.txt +++ b/src/kudu/gutil/CMakeLists.txt @@ -70,4 +70,8 @@ ADD_EXPORTABLE_LIBRARY(gutil add_kudu_test(strings/string_util-test) add_kudu_test(strings/numbers-test) -add_kudu_test(sysinfo-test) +if (NOT APPLE) + # The sysinfo-test consists of scenarios for Linux-specific function + # ParseMaxCpuIndex(): that doesn't exist on macOS. + add_kudu_test(sysinfo-test) +endif()