Unverified Commit 3491f673 authored by Alyssa Ross's avatar Alyssa Ross Committed by GitHub
Browse files

clang-tools: Respect $CLANGD_FLAGS in wrapper (#498874)

parents 4d183914 b9ea8c22
Loading
Loading
Loading
Loading
+16 −20
Original line number Diff line number Diff line
@@ -50,7 +50,7 @@ stdenv.mkDerivation (finalAttrs: {
    runHook postInstall
  '';

  passthru.tests.smokeOk =
  passthru.tests =
    let
      src = writeText "main.cpp" ''
        #include <iostream>
@@ -61,28 +61,24 @@ stdenv.mkDerivation (finalAttrs: {
      '';

    in
    runCommand "clang-tools-test-smoke-ok" { } ''
    {
      smokeOk = runCommand "clang-tools-test-smoke-ok" { } ''
        ${finalAttrs.finalPackage}/bin/clangd  --check=${src}
        touch $out
      '';
      smokeErr = runCommand "clang-tools-test-smoke-err" { } ''
        (${finalAttrs.finalPackage}/bin/clangd --query-driver='**' --check=${src} 2>&1 || true) \
            | grep 'use of undeclared identifier'

  passthru.tests.smokeErr =
    let
      src = writeText "main.cpp" ''
        #include <iostream>

        int main() {
           std::cout << "Hi!";
        }
        touch $out
      '';

    in
    runCommand "clang-tools-test-smoke-err" { } ''
      (${finalAttrs.finalPackage}/bin/clangd --query-driver='**' --check=${src} 2>&1 || true) \
      environmentErr = runCommand "clang-tools-test-environment-err" { } ''
         (CLANGD_FLAGS="--query-driver='**'" ${finalAttrs.finalPackage}/bin/clangd --check=${src} 2>&1 || true) \
            | grep 'use of undeclared identifier'

        touch $out
      '';
    };

  meta = llvm_meta // {
    description = "Standalone command line tools for C++ development";
+1 −1
Original line number Diff line number Diff line
@@ -40,7 +40,7 @@ buildcpluspath() {
# don't want to infect user-specified toolchain and headers with our stuff.
extendcpath=true

for arg in "$@"; do
for arg in "$@" $CLANGD_FLAGS; do
  if [[ "${arg}" == \-\-query\-driver* ]]; then
    extendcpath=false
  fi