Adar Dembo has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15296 )

Change subject: [ycm] Support recent versions of YouCompleteMe
......................................................................

[ycm] Support recent versions of YouCompleteMe

The Vim plugin YouCompleteMe now looks for a Settings() method in the
.ycm_extra_conf.py file instead of FlagsForFile(). According to some bug
reports I saw, they claim to be backwards-compatible, but that wasn't my
experience using --clang-completer. This patch just adds a Settings()
call that invokes FlagsForFile(), and leaves FlagsForFile() in place for
backcompat with older versions of YCM. This works for me on the latest
master version of YCM.

Change-Id: I21cc5e2cf05f91c1a6ed44cfbacc5b325438f5b7
Reviewed-on: http://gerrit.cloudera.org:8080/15296
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
---
M .ycm_extra_conf.py
1 file changed, 6 insertions(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Adar Dembo: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/15296
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I21cc5e2cf05f91c1a6ed44cfbacc5b325438f5b7
Gerrit-Change-Number: 15296
Gerrit-PatchSet: 2
Gerrit-Owner: Mike Percy <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>

Reply via email to