1200
94%

Ran 25 Jan 2016 01:39AM UTC

Jobs 1

Files 42

Run time 2s

Badge

Embed ▾

pending completion

Build # 1200

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

Jobs

ID	Job ID	Ran	Files	Coverage
5	1200.5 (USE_CLANG_COMPLETER=true YCMD_PYTHON_VERSION=2.7 COVERAGE=true)	25 Jan 2016 01:39AM UTC	0	82.65	Travis Job 1200.5

Source Files on build 1200

Detailed source file information is not available for this build.

Valloric / ycmd / 1200
94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1200

Valloric / ycmd / 1200 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1200

Valloric / ycmd / 1200
94%

README BADGES
x