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
docker-slim: init at 1.26.1 #76831
docker-slim: init at 1.26.1 #76831
Conversation
@GrahamcOfBorg build docker-slim |
You should probably add @filalex77 to the commit message. https://help.github.com/en/github/committing-changes-to-your-project/creating-a-commit-with-multiple-authors |
@kalbasit Is this also to be applied although this patch was neither co-authored nor based on the other PR? I have no problem about giving credit if applicable, but wouldn't this express facts that are not true? Edit: @filalex77 Take a look at the PR, I'll leave it up to you to decide. Trying to avoid an argument about this. |
@mbrgm I managed to build it with custom |
@marsam I like your version better than mine, given you don't apply any patches. You'd still need to wrap the executable on installation; otherwise, docker-slim will try to create its state directory besides the binary (which will be in nix store). I'm going to integrate all of our changes in this PR. This should then also clarify the authorship question :D. |
Co-authored-by: Mario Rodas <marsam@users.noreply.github.com> Co-authored-by: Oleksii Filonenko <brightone@protonmail.com>
Ok, this should now have everything important in it. Incorporated @marsam's changes, set him and @filalex77 as co-author. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
diff LGTM
executable shows usage, minified an Alpine container successfully
closing #75564
[4 built, 4 copied (392.0 MiB), 98.5 MiB DL]
https://github.com/NixOS/nixpkgs/pull/76831
1 package built:
docker-slim
@mbrgm thank you, sorry for the delay |
I have just seen this by browsing nixpkgs. The purpose of |
This PR is a replacement for #75564.
Besides fixing some issues, which occur due to building inside the nix build
environment, I also added a patch to fix path/casing-related issues until
slimtoolkit/slim#121 is merged and a new version
is released.
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @