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

Ditch more (hopefully unused) parts of busybox #57

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DemiMarie
Copy link

No description provided.

@marmarek
Copy link
Member

By skimming over the diff, I'm pretty sure it will break stuff. You disabled "echo" and "printf", both used in the init script. There may be more, I haven't checked further. Anyway, how much space (-> stubdom RAM) does it save? If negligible, it isn't really worth the effort (and it seems there would be several iterations...).

@DemiMarie
Copy link
Author

By skimming over the diff, I'm pretty sure it will break stuff. You disabled "echo" and "printf", both used in the init script.

I was under the assumption (possibly incorrect) that those are shell built-ins.

There may be more, I haven't checked further. Anyway, how much space (-> stubdom RAM) does it save? If negligible, it isn't really worth the effort (and it seems there would be several iterations...).

Fair point, and I have not measured yet. There are some commands that are clearly unneeded, such as anything related to cryptography, but still.

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.

2 participants