Bug 158301 – [GTK] Update YouCompleteMe script to make all include paths absolute

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[youenn fablet]

YCM does not make all include paths absolute.
At least on my setup, dependencies headers are not always found.

[youenn fablet]

Created attachment 280328 [details]
Patch

[Michael Catanzaro]

What is this script?

[youenn fablet]

(In reply to comment #2)
> What is this script?

It allows https://github.com/Valloric/YouCompleteMe working properly for WebKit source base.

[Michael Catanzaro]

Comment on attachment 280328 [details]
Patch

OK, rubber stamp, but please separately add a comment to the top of this script to indicate what it's for.

[youenn fablet]

(In reply to comment #4)
> Comment on attachment 280328 [details]
> Patch
> 
> OK, rubber stamp, but please separately add a comment to the top of this
> script to indicate what it's for.

OK, thanks, I'll clarify the changelog.

Martin, changseok, would you be able to check that the change is not disrupting your setup?

[youenn fablet]

Created attachment 280444 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 280444 [details]
Patch for landing

Clearing flags on attachment: 280444

Committed r201642: <http://trac.webkit.org/changeset/201642>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.