[Bug 239682] Default to devel/llvm90 when libLLVM/libclang are required or if /usr/bin/clang is not enough

[email protected]
Newsgroups gmane.os.freebsd.devel.x11
Message-ID <[email protected]/bugzilla/>
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=239682

Henry Hu <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|maintainer-feedback?(henry. |maintainer-feedback+
                   |[email protected])            |

--- Comment #13 from Henry Hu <[email protected]> ---
I tested cquery with newer version of LLVM, and "cquery --test-unit" and
"cquery --check <sample c++ file>" seem to work fine. We can make it use the
default LLVM version with this patch:

diff --git a/devel/cquery/Makefile b/devel/cquery/Makefile
index 4ebab81..a91ea4a 100644
--- a/devel/cquery/Makefile
+++ b/devel/cquery/Makefile
@@ -12,8 +12,8 @@ LICENSE=      MIT

 BROKEN_powerpc64=      fails to build: Checking for 'clang++' (C++ compiler):
not found

-BUILD_DEPENDS= llvm-config60:devel/llvm60
-LIB_DEPENDS=   libclang.so:devel/llvm60
+BUILD_DEPENDS= llvm-config${LLVM_DEFAULT}:devel/llvm${LLVM_DEFAULT}
+LIB_DEPENDS=   libclang.so:devel/llvm${LLVM_DEFAULT}

 USES=          compiler:c++14-lang waf

@@ -28,7 +28,7 @@ GH_TUPLE=    
miloyip:rapidjson:daabb88:rapidjson/third_party/rapidjson \

 PLIST_FILES=   bin/cquery

-CONFIGURE_ARGS=        --variant=system
--llvm-config=${LOCALBASE}/bin/llvm-config60
+CONFIGURE_ARGS=        --variant=system
--llvm-config=${LOCALBASE}/bin/llvm-config${LLVM_DEFAULT}
 MAKE_ARGS=     --variant=system

 post-install:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-x11
To unsubscribe, send any mail to "[email protected]"
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.