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

sys-apps/busybox: Disable standalone shell mode with USE=make-symlinks #22038

Closed
wants to merge 1 commit into from

Conversation

jsmolic
Copy link
Member

@jsmolic jsmolic commented Aug 19, 2021

  • Busybox has by default enabled CONFIG_FEATURE_PREFER_APPLETS
    and CONFIG_FEATURE_SH_STANDALONE which means it will
    bypass all PATH lookups and use its builtin applets.
    This is problematic as we sometimes might want to use the
    original program instead of Busybox applets, so let's
    disable those two options when building with USE=make-symlinks.
    This also enables users to disable a builtin by deleting the
    corresponding symlink.

Closes: https://bugs.gentoo.org/729184

* Busybox has by default enabled CONFIG_FEATURE_PREFER_APPLETS
  and CONFIG_FEATURE_SH_STANDALONE which means it will
  bypass all PATH lookups and use its builtin applets.
  This is problematic as we sometimes might want to use the
  original program instead of Busybox applets, so let's
  disable those two options when building with USE=make-symlinks.
  This also enables users to disable a builtin by deleting the
  corresponding symlink.

Closes: https://bugs.gentoo.org/729184
Signed-off-by: Jakov Smolic <jakov.smolic@sartura.hr>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @jsmolic
Areas affected: ebuilds
Packages affected: sys-apps/busybox

sys-apps/busybox: @gentoo/embedded

Linked bugs

Bugs linked: 729184


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Aug 19, 2021
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-08-19 17:34 UTC
Newest commit scanned: 979e05f
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/931c63b3e9/output.html

@jsmolic jsmolic deleted the busybox branch August 27, 2021 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet
3 participants