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

Support for Alpine Linux #43

Closed
accnops opened this issue May 12, 2020 · 4 comments
Closed

Support for Alpine Linux #43

accnops opened this issue May 12, 2020 · 4 comments
Labels
feature-request A feature should be added or improved. p2 This is a standard priority issue

Comments

@accnops
Copy link

accnops commented May 12, 2020

Is your feature request related to a problem? Please describe.
Currently, there is no Alpine Linux support.

When using following example:

File index.js:

require('aws-crt');

File Dockerfile:

FROM node:12.16.2-alpine
RUN apk add gcompat
RUN npm install aws-crt@1.1.3
COPY index.js .
CMD ["node", "index.js"]

Command:
docker build -t aws-crt-test . && docker run aws-crt-test

It seems like aws-crt is "hijacking" the process, and printing:

Signal received: 32, errno: 0
Signal received: 1530847424, errno: 32668
Signal received: 0, errno: 0
Signal received: 1530847424, errno: 32668
Signal received: 0, errno: 0
Signal received: 1530847424, errno: 32668
Signal received: 0, errno: 0
Signal received: 1530847424, errno: 32668
Signal received: 0, errno: 0
Signal received: 1530847424, errno: 32668
Signal received: 0, errno: 0
Signal received: 1530847424, errno: 32668
Signal received: 0, errno: 0
...

And then exiting.

Describe the solution you'd like
Apparently, it's because there's not yet an alpine target where it's linked against libmusl. That would be great, since I can then use this library on Alpine.

@accnops accnops added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels May 12, 2020
@jmklix
Copy link
Member

jmklix commented Apr 13, 2021

Update: This is a feature that we are still interested in adding but we have not gotten to it yet. We will update in the future if our priorities change.

@jmklix jmklix removed the needs-triage This issue or PR still needs to be triaged. label Apr 13, 2021
@cazzer
Copy link

cazzer commented Sep 21, 2021

@jmklix any update on your roadmap? From my experience, Alpine is a pretty common base image for Node containers so I'd imagine this would be a common blocker for other teams, as it is for mine.

Adding a reference to what base images do work would be a massive help - or any other workarounds you could suggest!

@jmklix jmklix added p2 This is a standard priority issue needs-review labels Nov 7, 2022
@bretambrose
Copy link
Contributor

Initial alpine support has been added to the CRT package and incorporated into the SDK as of the https://github.com/aws/aws-iot-device-sdk-js-v2/releases/tag/v1.16.0 release

@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A feature should be added or improved. p2 This is a standard priority issue
Projects
None yet
Development

No branches or pull requests

4 participants