Artem Pavlenko

Results 235 comments of Artem Pavlenko

@claytondus - I like [prebuildify](https://github.com/prebuild/prebuildify)+[node-gyp-build](https://github.com/prebuild/node-gyp-build) but agree the size of a package can be an issue. ### possible approach (?) "optionalDependencies" with "postinstall" fallback script to install `arch/os` specific `mapnik`...

@claytondus This is all very much [WIP] and needs more work. Could you try and let me know if this solves package size issue on Lambda. `npm install @mapnik/[email protected]` On...

@claytondus thanks for testing! Could you run ``` npm install @mapnik/[email protected] --foreground-scripts --loglevel=verbose ``` and post output either here or as a gist if too large. Also, could you try...

@claytondus ok, thanks for feedback. I'll try to replicate this issue locally.

@claytondus - I see the issue -> "overrides" strips `@scope` (e.g `@mapnik`) which in turn breaks node_modules directory logic in `preinstall` script. ``` ... MODULE_PATH:/Users/CZD05/src/rms-api/mapnik-split-test/node_modules/mapnik/prebuilds/darwin-arm64 ... ``` `preinstall` is expecting...

[@mapnik/[email protected]](https://github.com/mapnik/node-mapnik/pkgs/npm/mapnik/281433688) @claytondus should work, tested with - ``` { "name": "mapnik-split-test", "version": "1.0.0", "main": "index.js", "author": "", "license": "ISC", "description": "", "dependencies": { "@mapbox/spritezero": "^8.0.3" }, "overrides": { "@mapbox/spritezero": {...

@matkoniecz - I agree https://nodejs.org/dist/latest/docs/api/n-api.html#n_api_node_api_version_matrix is not easy to navigate. Based on `npm test` passing following "stable" node versions should work with `node-mapnik v4.6.9` ```bash v18.20.7 v20.19.0 v22.14.0 v23.9.0 ```...

@hummeltech thanks! https://github.com/mapnik/mapnik/commit/f91f522e712387cf135f57c0151e2303b9f7babc 054c2a18a982b67e3b1da40e9104c6e72a41ef6f explicit versions + remove macos-13 * macos-14 * ubuntu- ~~24.04~~ 22.04 * windows-2022

@mathisloge `std::format` 👍 As long as logging lib is minimal and can be statically linked (from looking briefly looks like `spdlog` can be used as header only) without significantly increasing...

@LennardSchaap This looks related to https://github.com/mapnik/mapnik/issues/4375 Could you try defining `BOOST_PHOENIX_STL_TUPLE_H_` e.g. ```bash ./scons/scons.py CUSTOM_DEFINES="-DBOOST_PHOENIX_STL_TUPLE_H_" configure ```