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

Add small bubblewrap like program as a build step #339

Closed
wants to merge 16 commits into from

Conversation

MaxHearnden
Copy link
Contributor

This allow the user to perform a rootless build without bubblewrap on machines with user namespaces enabled

@fosslinux
Copy link
Owner

I understand the idea/implementation in principle. Would you mind outlining any particular, specific usecases for this?

@MaxHearnden
Copy link
Contributor Author

It's to reduce the binary seed required for an unprivileged build to just hex0+kaem-optional and can be used with the following commands

cp sysa/stage0-posix/src/* .
cp sysa/after_wrap.kaem after.kaem

# This stage could be integrated into after_wrap.kaem
echo "CHROOT=True
CHROOT_WRAP=True" >> sysa/bootstrap.cfg

./bootstrap-seeds/POSIX/x86/kaem-optional-seed

@oriansj
Copy link

oriansj commented Nov 29, 2023

I like it but 2 minor details:

  1. the syscalls should added to M2libc such that the code becomes more standard C.
  2. the wrap.c program belongs in mescc-tools-extra which live-bootstrap does already use and will allow other bootstrapping projects to leverage your work.

great work

MaxHearnden added a commit to MaxHearnden/mescc-tools-extra that referenced this pull request Nov 30, 2023
MaxHearnden added a commit to MaxHearnden/mescc-tools-extra that referenced this pull request Dec 12, 2023
@fosslinux
Copy link
Owner

If this could be

  1. rebased on master
  2. updated to reflect that wrap.c is in mescc-tools-extra

I think I'm happy to include this in principle.

@Googulator
Copy link
Collaborator

#436 will bring in wrap.c into live-bootstrap, although currently unused

@MaxHearnden
Copy link
Contributor Author

Replaced by #479

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