2016-04-08 15:26 GMT+02:00 Jérémy Lal <kapo...@melix.org>: > 2016-04-08 15:10 GMT+02:00 Sebastiaan Couwenberg <sebas...@xs4all.nl>: > >> On 04/08/2016 02:20 PM, Jérémy Lal wrote: >> > 2016-04-08 14:04 GMT+02:00 Sebastiaan Couwenberg: >> >> The test target fails because it requires node-pre-gyp, as do the >> >> bin/mapnik-*.js executables. We'll need to patch those to not require >> >> node-pre-gyp too. >> >> >> >> Do you have any suggestions for that? >> > >> > Yes, i have: i'm packaging node-pre-gyp and rc modules, it seems pkg-tar >> > could stay a recommandation. >> > It will be way simpler than spending the time patching node-mapnik. >> >> Thanks! >> >> I've patched the mapnik-*.js executables to not require node-pre-gyp, >> that just leaves lib/mapnik.js which uses it to find mapnik_settings.js. >> >> Since its path differs between the build environment and after >> installation, I'm not sure if we can make it support both without >> node-pre-gyp. >> >> I think I'll leave the node-mapnik packaging as it is for now, add have >> another look when node-pre-gyp is available or if it becomes clear we >> can work around it. >> > > Ok. I'll be able to upload node-pre-gyp and node-rc later today. >
node-pre-gyp is now in the archive. There are a couple of fixes to do to node-mapnik, and i'll upload it later today ? Unless you want to do the work. Cheers, Jérémy
