Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Travis build is broken #2041

Closed
elemoine opened this issue May 5, 2014 · 5 comments
Closed

Travis build is broken #2041

elemoine opened this issue May 5, 2014 · 5 comments

Comments

@elemoine
Copy link
Member

elemoine commented May 5, 2014

The problem may be related to my latest merge: #2040. I'll have a look.

https://travis-ci.org/openlayers/ol3/builds/24439585

@elemoine
Copy link
Member Author

elemoine commented May 5, 2014

The error occurs when checking the kml-earthquakes example:

2014-05-05 09:47:04,419 check-examples: phantomjs bin/check-example.js build/hosted/HEAD/examples/kml-earthquakes.html?mode=advanced
JavaScript ERROR: INDEX_SIZE_ERR: DOM Exception 1: Index or size was negative, or greater than the allowed value.

TRACE:

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 233 (in function "Jl")

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 258 (in function "Wl")

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 294

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 299

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 350

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 159

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 5

-> file:///home/travis/build/openlayers/ol3/build/hosted/HEAD/build/ol.js: 55

2014-05-05 09:47:05,028 check-examples: clean

2014-05-05 09:47:05,028 pake: check-examples: Command '['phantomjs', 'bin/check-example.js', 'build/hosted/HEAD/examples/kml-earthquakes.html?mode=advanced']' returned non-zero exit status 1

The command "./build.py integration-test" exited with 1.

Problem is that I'm not able to reproduce locally.

@elemoine
Copy link
Member Author

elemoine commented May 5, 2014

I managed to reproduce the bug locally, but it does not always occur. It would occur at one run (build.py check-examples) but not at the next one. Annoying. I don't know how to fix this at this point.

@tschaub
Copy link
Member

tschaub commented May 5, 2014

Maybe the map viewport has a different size in the Phantom run and there is a drawImage call with a negative value that Webkit doesn't like.

@elemoine
Copy link
Member Author

elemoine commented May 5, 2014

Thanks Tim. I'll look at it again tomorrow.

@elemoine
Copy link
Member Author

elemoine commented May 6, 2014

This reveals an actual bug in our Canvas code. The kml-earthquakes example does not work in FireFox:

IndexSizeError: Index or size is negative or greater than the allowed amount

The problem is related to the symbol size as calculated in ol.style.Circle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants