Skip to content

Add e2fsprogs-extra to alpine-base - Fixes #132 - #140

Open
torrancew wants to merge 1 commit into
earendil-works:mainfrom
torrancew:fix/resize2fs-in-alpine-base
Open

Add e2fsprogs-extra to alpine-base - Fixes #132#140
torrancew wants to merge 1 commit into
earendil-works:mainfrom
torrancew:fix/resize2fs-in-alpine-base

Conversation

@torrancew

Copy link
Copy Markdown

Alpine packages resize2fs in e2fsprogs-extra, going back at least as far as Alpine 3.16:

ref: https://pkgs.alpinelinux.org/contents?file=resize2fs&path=&name=&branch=v3.16

Some other parts of the repo already document this:

  • host/src/alpine/utils.rs
  • docs/custom-images.md

I left a single comment un-changed, as (best I could tell), it was in a distro-agnostic context:

  • host/src/vm/core.ts:1614

I left host/src/build/container.ts unchanged, as to the best of my understanding, the container there does not implicitly hit any resize paths. Happy to update this if I'm wrong, to maintain consistency, etc.

Contribution Agreement

Please ensure this PR follows the guidelines in CONTRIBUTING.md.

Note from @torrancew: This repo does not contain a CONTRIBUTING.md. I have read, understood, and applied the relevant sections from the earendil-works/pi CONTRIBUTING.md in its absence.

By submitting this pull request, I confirm the following:

I understand that the entity Earendil Inc. (incorporated in the state of Delaware in 2025) needs some rights from me in order to utilize my contributions in this PR. As a contributor I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Earendil Inc. can use, modify, copy, and redistribute my contributions, under Earendil Inc.'s choice of terms.

Alpine packages `resize2fs` in `e2fsprogs-extra`, going back at least as far as Alpine 3.16:

ref: https://pkgs.alpinelinux.org/contents?file=resize2fs&path=&name=&branch=v3.16

Some other parts of the repo already document this:

- `host/src/alpine/utils.rs`
- `docs/custom-images.md`

I left a single comment un-changed, as (best I could tell), it was in a distro-agnostic context:
- `host/src/vm/core.ts:1614`

I left `host/src/build/container.ts` unchanged, as to the best of my
understanding, the container there does not implicitly hit any resize
paths. Happy to update this if I'm wrong, to maintain consistency, etc.
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.

1 participant