Skip to content

Conversation

@WebReflection
Copy link
Contributor

There's an override error with latest alpine and various versions of glibc (including 2.35): https://github.com/sgerrand/alpine-pkg-glibc/issues/185

This MR proposes a workaround (tested on fly.io - it works) so that it's possible to have bun usable with minimal linux dependencies.

There's an override error with latest alpine and various versions of glibc (including 2.35): https://github.com/sgerrand/alpine-pkg-glibc/issues/185

This MR proposes a workaround (tested on fly.io - it works) so that it's possible to have bun usable with minimal linux dependencies.
@Jarred-Sumner
Copy link
Collaborator

@Electroid what if we have an alpine image version with this? Maybe like oven-sh/bun-lite or oven-sh/bun:lite

@moderation
Copy link

My experience with Envoy Proxy suggests trying to force a glibc dependent into an Alpine image isn't a good fit. A better option might be https://github.com/wolfi-dev/ which is designed to be secure and minimal and has native support for glibc

@WebReflection
Copy link
Contributor Author

I have zero opinions around this topic but current file is part of this repo and cannot be built ... is this going to be fixed regardless of your future plan to simplify a minimalistic Docker image? 🤔 ... and about wolfi, which image would be minimalistic enough? I am actually pretty happy with the 100MB alpine total image size (where AFAIK 90MB are just for bun) but if it goes like 120MB and makes it as easy as this to have bun running please go ahead 👍

### GET ###
FROM wolfi:thingy as get

# prepare environment
WORKDIR /tmp
RUN apk --no-cache add unzip

# get bun
ADD https://github.com/oven-sh/bun/releases/latest/download/bun-linux-x64.zip bun-linux-x64.zip
RUN unzip bun-linux-x64.zip

FROM wolfi:thingy

# install bun
COPY --from=get /tmp/bun-linux-x64/bun /usr/local/bin

Copy link
Contributor

@Electroid Electroid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix, and for testing. We'll consider the slim/wolfi image separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants