Continued from bug 727832 comment 6, bash_command_not_found_re does not handle the case where sh is busybox: > $ ash -c 'foo' > ash: foo: not found lib/portage/package/ebuild/doebuild.py: bash_command_not_found_re = re.compile( lib/portage/package/ebuild/doebuild.py- r"(.*): line (\d*): (.*): command not found$"
If sh is busybox, then command-not-found detection not working properly is the least of our worries. IIRC busybox sh isn't POSIX-compliant.
While at it: % dash -c foo dash: 1: foo: not found
Another one... $ mksh -c 'foo' mksh: foo: inaccessible or not found
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/proj/portage.git/commit/?id=a2409ba65d38d2eb8cb3923733d9062e3b4da1ad commit a2409ba65d38d2eb8cb3923733d9062e3b4da1ad Author: Eli Schwartz <eschwartz93@gmail.com> AuthorDate: 2023-12-20 05:56:53 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-12-20 14:04:12 +0000 ebuild: command not found QA: detect in dash as well Bug: https://bugs.gentoo.org/822033 Signed-off-by: Eli Schwartz <eschwartz93@gmail.com> Signed-off-by: Sam James <sam@gentoo.org> lib/portage/package/ebuild/doebuild.py | 7 +++++++ 1 file changed, 7 insertions(+)
I'll add a depends-on for portage-3.0.59, but we shouldn't close this on release given busybox/mksh isn't covered. Not that I really think busybox is a suitable /bin/sh.
I'll also note that I kinda have zero interest in mksh on a personal level, and while I do like busybox as a general thing I have *negative* interest in it as a general purpose /bin/sh and would strongly prefer refusing to support it. Someone other than me should add support for mksh though. :D
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0a1f19cdd7a598070b7eb08b3954e677aa4868ad commit 0a1f19cdd7a598070b7eb08b3954e677aa4868ad Author: Sam James <sam@gentoo.org> AuthorDate: 2023-12-27 21:27:55 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-12-27 21:28:01 +0000 sys-apps/portage: add 3.0.59 Closes: https://bugs.gentoo.org/587088 Closes: https://bugs.gentoo.org/822033 Closes: https://bugs.gentoo.org/915494 Closes: https://bugs.gentoo.org/916135 Closes: https://bugs.gentoo.org/917120 Closes: https://bugs.gentoo.org/919862 Closes: https://bugs.gentoo.org/920095 Closes: https://bugs.gentoo.org/920258 Closes: https://bugs.gentoo.org/920537 Closes: https://bugs.gentoo.org/920654 Signed-off-by: Sam James <sam@gentoo.org> sys-apps/portage/Manifest | 1 + sys-apps/portage/portage-3.0.59.ebuild | 246 +++++++++++++++++++++++++++++++++ 2 files changed, 247 insertions(+)
