-
Notifications
You must be signed in to change notification settings - Fork 0
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
Implementing #1
Comments
Here's my plan: Write a build process using ideas from progrium/buildstep#109, especially the unincluded patches mentioned in progrium/buildstep#109 (comment), specifically yabawock/buildstep@4c84186. Compilation will be a script, and it will be called by a wrapping script that creates a temporary non-privileged user to do the build (and, later, to do the run as well). Might also take another look at progrium/buildstep#98, integrate some of that. Also, since it's possible that an app may want to run as root (as discussed in plushu/enter-sandbox#5), maybe some mechanism to skip the user drop (using a mechanism like the one to keep the git directory in plushu-git-apps) should be included. Scripts for building are going to be included in this plugin's lib/ directory, which will be mounted in the image as a volume at |
Because of the let's-not-break-enter-sandbox-right-now fuss of plushu/enter-sandbox#5, I think I'll actually table the user-creation stuff for now and just start with a straight build-and-run-as-root build script, maybe factoring it to be a little closer to the structure that will be needed to create such a framing user. Then I'll tackle the framing user stuff later, with another issue for the architectural concerns and decisions involved, including the relevant stuff for possibly allowing root build/runs. |
Differences from buildstep(based on yabawock's script)
|
Seeing as how this is implemented now (except for the non-root stuff), I'm going to close this issue and open another one for the root thing. |
continued from plushu/plushu-buildstep#5
The text was updated successfully, but these errors were encountered: