indutny / indutny/node-netroute
Remove sys/sysctrl.h include
Open
- Dominant language
- C++
- Stars
- 47
- Forks
- 9
- PR merge metrics
- No merged PRs in 30d
Description
Currently can't node-gyp build node-netroute on Alpine linux. This include doesn't seem POSIX. In other words, this doesn't work
```bash
npm install netroute
```
See https://github.com/libressl-portable/portable/issues/96
Could we just remove the include?
Contributor guide
No contributing guide indexed for this repository
Research direction
Inspect the C++ addon source for the sys/sysctrl.h include, then reproduce the failure with npm install netroute on Alpine Linux. Remove the non-POSIX include if it is unused and confirm that node-gyp builds successfully without it.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, node.js
- Domain
- build-system, networking, operating-systems
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100