[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Remarks about commit 2c82d4ad10de8
From: |
Mathieu Othacehe |
Subject: |
Remarks about commit 2c82d4ad10de8 |
Date: |
Mon, 16 Dec 2019 16:13:27 +0100 |
User-agent: |
mu4e 1.2.0; emacs 26.3 |
Hello Brett,
I have a few remarks on the aforementioned commit.
> + (list (string-append "-DCMAKE_CXX_FLAGS='-isystem "
> + (assoc-ref %build-inputs "gcc")
> + "/include/c++'"))))
Why is this needed? The following snippet in clang-from-llvm isn't enough?
--8<---------------cut here---------------start------------->8---
;; Make clang look for libstdc++ in the right
;; location.
(("LibStdCXXIncludePathCandidates\\[\\] = \\{")
(string-append
"LibStdCXXIncludePathCandidates[] = { \"" gcc "/include/c++\","))
--8<---------------cut here---------------end--------------->8---
> + (inputs
> `(("clang" ,clang)
> - ("llvm" ,llvm)))
> + ("ncurses" ,ncurses)))
> + (native-inputs
> + `(("rapidjson" ,rapidjson)
Rapidjson is an input (even if ccls is not linked against because its a
header-only library).
> + ("gcc" ,gcc)))
This shouldn't be needed as it is an implicit input.
Thanks,
Mathieu
- Remarks about commit 2c82d4ad10de8,
Mathieu Othacehe <=