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

bootstrap: Run {initialize-empty,remake}-image tasks in the container #393

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

qookei
Copy link
Member

@qookei qookei commented Oct 31, 2024

Also make host-limine not a containerless tool and build it using host-llvm-toolchain.

The make-image task is still containerless because it actually mounts the disk image (either via loopback or guestfs), but ideally it shouldn't be, because it invokes 'strip' from cross-binutils, which just happens to work outside the container.

Depends on qookei/image_create#2.

qookei and others added 2 commits November 1, 2024 00:41
Also make host-limine not a containerless tool and build it using
host-llvm-toolchain.

The make-image task is still containerless because it actually mounts
the disk image (either via loopback or guestfs), but ideally it
shouldn't be, because it invokes 'strip' from cross-binutils, which
just happens to work outside the container.
Copy link
Member

@no92 no92 left a comment

Choose a reason for hiding this comment

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

LGTM

@qookei qookei merged commit d501a83 into managarm:master Nov 1, 2024
3 checks passed
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.

3 participants