Merge pull request oclint#283 from ryuichis/master

Match compiler visibility settings with LLVM/Clang
yinfan198741 · Jan 17, 2016 · bf70aa2 · bf70aa2
2 parents 5964951 + ca5288f
commit bf70aa2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/oclint-core/cmake/OCLintConfig.cmake b/oclint-core/cmake/OCLintConfig.cmake
@@ -24,8 +24,8 @@ ELSE()
 ENDIF()
 
 IF(APPLE)
-    SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -mmacosx-version-min=10.10")
-    SET(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -mmacosx-version-min=10.10")
+    SET(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fvisibility-inlines-hidden -mmacosx-version-min=10.11")
+    SET(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -mmacosx-version-min=10.11")
 ENDIF()
 
 IF(OCLINT_BUILD_TYPE STREQUAL "Release")

diff --git a/oclint-scripts/clang b/oclint-scripts/clang
@@ -98,7 +98,7 @@ def setup_prebuilt_binary():
     path.cd(build_path)
     process.call('curl -o prebuilt.tar.xz ' + binary_url)
     path.mkdir_p(install_path)
-    process.call('tar xvf prebuilt.tar.xz -C ' + install_path + ' --strip-components=1')
+    process.call('tar xf prebuilt.tar.xz -C ' + install_path + ' --strip-components=1')
 
     # A workaround for mac prebuilt binary
     if environment.is_darwin: