Compile error

[piechade]

• SerialPort Version: 6.1.0
• NodeJS Version: 9.5.0
• Operating System and Hardware Platform: Alpine Linux Docker

Summary of Problem

CXX(target) Release/obj.target/serialport/src/serialport_linux.o
../src/serialport_linux.cpp: In function 'int linuxSetCustomBaudRate(int, unsigned int)':
../src/serialport_linux.cpp:10:18: error: 'TCGETS2' was not declared in this scope
if(ioctl(fd, TCGETS2, &t)) {
^~~~~~~
../src/serialport_linux.cpp:18:18: error: 'TCSETS2' was not declared in this scope
if(ioctl(fd, TCSETS2, &t)) {
^~~~~~~
../src/serialport_linux.cpp: In function 'int linuxGetSystemBaudRate(int, int*)':
../src/serialport_linux.cpp:29:16: error: 'TCGETS2' was not declared in this scope
if(ioctl(fd, TCGETS2, &t)) {
^~~~~~~
make: *** [serialport.target.mk:98: Release/obj.target/serialport/src/serialport_linux.o] Error 1

Steps and Code to Reproduce the Issue

npm install

[reconbot]

Cc @Fumon

[Fumon]

Sorry for the delay, notification got filtered. Looks like an easy fix.

Seems to be an issue with musl libc which Alpine Linux is based on. Found a reference and fix in a similar serial based project here which appeared to resolve the issue.

I have a branch for you to try sticking in your package.json to see if it compiles https://github.com/Fumon/node-serialport/tree/fix_musl

If I don't hear from you @piechade, I'll spin up an Alpine VM to test this myself.

[piechade]

@Fumon i will test it later this day. I need to look how to use GitHub instant of npm.
I had earlier time and used this command to test it:
npm install git+https://github.com/Fumon/node-serialport.git
Unfortunately the same issue.

[Fumon]

Ah, you need
npm install git+https://github.com/Fumon/node-serialport.git#fix_musl
to get the correct branch.

I just built a docker image using:

FROM gliderlabs/alpine:3.4
RUN apk add --no-cache build-base linux-headers nodejs git python\
  && npm install git+https://github.com/Fumon/node-serialport.git#fix_musl
ENTRYPOINT /bin/sh

Which built successfully.

[piechade]

Sorry about that, i forgot the branch. For me ist also everything working.