Skip to content
This repository has been archived by the owner on Dec 1, 2024. It is now read-only.

Include prebuilt binary for alpine #174

Closed
gogoout opened this issue Jul 15, 2021 · 5 comments
Closed

Include prebuilt binary for alpine #174

gogoout opened this issue Jul 15, 2021 · 5 comments
Labels
enhancement New feature or request

Comments

@gogoout
Copy link

gogoout commented Jul 15, 2021

Hi all, Alpine is commonly used in container environment.
Currently seems alpine seems doesn't have a prebuilt binary. Not sure if this is a low hanging fruit.

@vweevers vweevers added the enhancement New feature or request label Jul 15, 2021
@vweevers
Copy link
Member

Assuming that RocksDB compiles against musl (no reason to believe it doesn't), this should be easy. We do the same for leveldown. It's a matter of adding -i alpine to:

"prebuild-linux": "prebuildify-cross -i centos7-devtoolset7 -t 8.14.0 --napi --strip",

If no one is opposed to increasing npm package size. I'm fine with that.

vweevers added a commit that referenced this issue Aug 1, 2021
@reuzel
Copy link

reuzel commented Aug 7, 2021

Great work!

Is there any ETA for the next release in which this binary will be prepackaged?

@vweevers
Copy link
Member

vweevers commented Aug 7, 2021

When I have a free moment, within a week.

@gogoout
Copy link
Author

gogoout commented Aug 21, 2021

Hi, any updates on releasing?

@vweevers
Copy link
Member

5.1.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants