5
94%
master: 94%

Ran 25 Jan 2016 01:39AM UTC

Files 42

Run time 2s

Badge

Embed ▾

Committed 25 Jan 2016 01:39AM UTC coverage: 82.654%. Remained the same

Job # USE_CLANG_COMPLETER=true YCMD_PYTHON_VERSION=2.7 COVERAGE=true

Build Type

push

travis-ci

Committed by

homu

Commit Message

Auto merge of #318 - micbou:extra-cmake-args, r=Valloric

[READY] Properly split EXTRA_CMAKE_ARGS environment variable in build.py script

While I was playing with `EXTRA_CMAKE_ARGS` environment variable to answer issue Valloric/YouCompleteMe#1925, I found out it was not possible to pass CMake arguments with spaces in it because the variable is split on spaces. For example:
```sh
EXTRA_CMAKE_ARGS="-DCMAKE_VERBOSE_MAKEFILE=ON -DCMAKE_CXX_FLAGS='-march=native -O3'"
```
gives
```python
['-DCMAKE_VERBOSE_MAKEFILE=ON', "-DCMAKE_CXX_FLAGS='-march=native", "-O3'"]
```
which obviously fail.

This is fixed by using the [`shlex` module and its split feature](https://docs.python.org/2/library/shlex.html). With this, example above results in:
```python
['-DCMAKE_VERBOSE_MAKEFILE=ON', '-DCMAKE_CXX_FLAGS=-march=native -O3']
```
which make us happy.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/318)
<!-- Reviewable:end -->

Run Details

2597 of 3142 relevant lines covered (82.65%)

0.83 hits per line

Valloric / ycmd / 1200 / 5
94%
master: 94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1200.5 (USE_CLANG_COMPLETER=true YCMD_PYTHON_VERSION=2.7 COVERAGE=true)

Valloric / ycmd / 1200 / 5 94% master: 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1200.5 (USE_CLANG_COMPLETER=true YCMD_PYTHON_VERSION=2.7 COVERAGE=true)

Valloric / ycmd / 1200 / 5
94%
master: 94%

README BADGES
x