travis fails

[rmichaelis]

Has something changed ?

ERR! compile /home/travis/build/Geoportail-Luxembourg/geoportailv3/geoportailv3/static/js/search/searchdirective.js:528: ERROR - actual parameter 2 of ngeoCreateGeoJSONBloodhound does not match formal parameter
ERR! compile found : (null|ol.proj.Projection)
ERR! compile required: (function ((GeoJSONFeature|null)): boolean|undefined)
ERR! compile var bloodhound = ngeoCreateGeoJSONBloodhound(url, ol.proj.get('EPSG:3857'));
ERR! compile ^
ERR! compile
ERR! compile
ERR! compile 1 error(s), 0 warning(s), 97.3% typed
ERR! compile
ERR! closure-util Process exited with non-zero status, see log for more detail: 1

[petzlux]

I think @elemoine changed search in ngeo , maybe try with update-node-modules ?

[petzlux]

#817 ?

[rmichaelis]

Are node modules not automatically updated in Travis ?

[petzlux]

dont know about travis, did it work in your build?

[rmichaelis]

I don't know... I don't want to break my build :-)

[petzlux]

im trying with your branch as we speak ...

[petzlux]

Unfortunately, yes get the same problem :-(

./node_modules/openlayers/node_modules/.bin/closure-util build build.json .build/build.js
info closure-util Reading build config
info closure-util Getting Closure dependencies
info closure-util Compiling 431 sources
ERR! compile /home/wepa/geoportailv3/geoportailv3/static/js/search/searchdirective.js:531: ERROR - actual parameter 2 of ngeoCreateGeoJSONBloodhound does not match formal parameter
ERR! compile found   : (null|ol.proj.Projection)
ERR! compile required: (function ((GeoJSONFeature|null)): boolean|undefined)
ERR! compile   var bloodhound = ngeoCreateGeoJSONBloodhound(url, ol.proj.get('EPSG:3857'));
ERR! compile                                                     ^
ERR! compile
ERR! compile 1 error(s), 0 warning(s),
ERR! compile 97.3% typed
ERR! compile
ERR! closure-util Process exited with non-zero status, see log for more detail: 1
make: *** [.build/build.js] Error 1

[petzlux]

I think I found the issue , ngeo bloodhound function changed
https://github.com/camptocamp/ngeo/blob/master/src/services/creategeojsonbloodhound.js#L10

Will create pull request to change!

[petzlux]

@rmichaelis use #835 to fix problem.

[petzlux]

okay @rmichaelis , if you rebase from master , then problem should be gone at next build

[rmichaelis]

It works. Thanks.