New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[READY] Add -B to INCLUDE_FLAGS #1197
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for explaining what the flag does.
Reviewed 2 of 2 files at r1.
Reviewable status: 1 of 2 LGTMs obtained
Codecov Report
@@ Coverage Diff @@
## master #1197 +/- ##
=======================================
Coverage 97.28% 97.28%
=======================================
Files 95 95
Lines 7109 7109
=======================================
Hits 6916 6916
Misses 193 193 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 2 files at r1.
Reviewable status: complete! 2 of 2 LGTMs obtained
@zzbot r+ |
📌 Commit 7a83a14 has been approved by |
[READY] Add -B to INCLUDE_FLAGS This flag tells the compiler frontend (`gcc`/`clang`) where to look for the subprograms (`cpp`, `cc`, `cc1plus`, `as`, `ld` etc.). This flag is needed when bootstrapping a toolchain, so it is unlikely to be found in a custom extra conf, but it can appear in users' compilation databases. While it is not really an include flag, it behaves just like one. Fixes ycm-core/YouCompleteMe#3333 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/1197) <!-- Reviewable:end -->
💔 Test failed - status-travis |
@zzbot retry |
[READY] Add -B to INCLUDE_FLAGS This flag tells the compiler frontend (`gcc`/`clang`) where to look for the subprograms (`cpp`, `cc`, `cc1plus`, `as`, `ld` etc.). This flag is needed when bootstrapping a toolchain, so it is unlikely to be found in a custom extra conf, but it can appear in users' compilation databases. While it is not really an include flag, it behaves just like one. Fixes ycm-core/YouCompleteMe#3333 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/1197) <!-- Reviewable:end -->
💔 Test failed - status-travis |
☀️ Test successful - status-appveyor, status-travis |
[READY] Update ycmd Include the following changes: - PR ycm-core/ycmd#1180: trigger semantic completion when instructed by server; - PR ycm-core/ycmd#1184: simplify LSP completer API for starting server; - PR ycm-core/ycmd#1187: improve Clangd completer initialization; - PR ycm-core/ycmd#1191: ease Clangd completer initialization; - PR ycm-core/ycmd#1193: fix system header search paths on macOS for Objective-C++; - PR ycm-core/ycmd#1194: update Jedi to 0.13.3 and Parso to 0.3.4; - PR ycm-core/ycmd#1195: ignore identifiers returned by TSServer in JavaScript; - PR ycm-core/ycmd#1196: update TSServer to 3.3.3333; - PR ycm-core/ycmd#1197: support -B flag in C-family languages. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/youcompleteme/3339) <!-- Reviewable:end -->
This flag tells the compiler frontend (
gcc
/clang
) where to look for the subprograms (cpp
,cc
,cc1plus
,as
,ld
etc.). This flag is needed when bootstrapping a toolchain, so it is unlikely to be found in a custom extra conf, but it can appear in users' compilation databases. While it is not really an include flag, it behaves just like one.Fixes ycm-core/YouCompleteMe#3333
This change is