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

Mason #462

Closed
wants to merge 14 commits into from
Closed

Mason #462

wants to merge 14 commits into from

Conversation

flippmoke
Copy link
Member

Mason working in node mapnik. /cc @springmeyer

@springmeyer
Copy link
Member

Taking a look at this. Builds are failing on both OS X and Linux.

Linux is failing (https://travis-ci.org/mapnik/node-mapnik/jobs/67223520#L2368) with:

cp: cannot stat `/home/travis/build/mapnik/node-mapnik/mason_packages/.link/share/gdal': No such file or directory

OS X is failing (https://travis-ci.org/mapnik/node-mapnik/jobs/67223524#L2149) with:

cp: the -H, -L, and -P options may not be specified with the -r option.

I can replicate on OS X. If I recall correctly @flippmoke could not replicate on OS X which likely means he has some custom coreutils installed and is not using the default OS X /bin/cp.

@springmeyer
Copy link
Member

Going to punt on this for now and continue using mapnik-packaging for the v3.3.1 release. Even when the above minor copy errors are fixed, there will be more work to do. For instance $ORIGIN is not being properly set for the libmapnik linking such that shapeindex cannot find libmapnik.so on linux. And trying to fix this (mapbox/mason@c6a6d2a) broke oddly + travis OSX is down: https://www.traviscistatus.com/incidents/h5bxh21wb8mr. So, going with AWS + mapnik-packaging and will return to this in a few weeks.

@flippmoke flippmoke closed this Mar 31, 2017
@springmeyer
Copy link
Member

Done in #738

@springmeyer springmeyer deleted the mason branch March 31, 2017 23:41
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

Successfully merging this pull request may close these issues.

2 participants