Created attachment 671698 [details]
This package does not build due to it using the bashisms(?) pushd/popd. Was found by symlinking /bin/sh to app-shells/dash (/bin/dash).
Created attachment 672085 [details, diff]
The usage of both pushd and popd in the Makefile is wholly superfluous. Please try the attached patch by placing it under /etc/portage/patches/app-admin/openrc-settingsd/.
Created attachment 672088 [details, diff]
Disregard the previous patch. Here's another that implements the equivalent logic. The difference is that cd is used to change directories, rather than pushd. There is no need to subsequently switch the directory back because, for all affected instances, the shell code is contained by parentheses and is therefore executed within a subshell. Additionally, as chdir(2) could theoretically fail, use && to verify that the operation succeeded.
Applied patch, and it still errors out with pushd and popd.
Created attachment 672091 [details]
build log after patch
(In reply to ppdplayer from comment #4)
> Applied patch, and it still errors out with pushd and popd.
Thanks for testing. In that case, it should have an effect if you invoke "eautoreconf" just after "default" in the src_prepare function. I'll attach another patch that saves the trouble of editing the ebuild.
Created attachment 672109 [details, diff]
Revised to also patch the generated Makefile.in included by the tarball.
(In reply to Kerin Millar from comment #7)
> Created attachment 672109 [details, diff] [details, diff]
> Revised to also patch the generated Makefile.in included by the tarball.
The patch worked. I'd imagine that's getting thrown in the repository?
The bug has been closed via the following commit(s):
Author: Sam James <firstname.lastname@example.org>
AuthorDate: 2021-04-15 01:24:59 +0000
Commit: Sam James <email@example.com>
CommitDate: 2021-04-15 01:31:06 +0000
app-admin/openrc-settingsd: fix build with non-Bash
Thanks-to: Kerin Millar
Signed-off-by: Sam James <firstname.lastname@example.org>
.../openrc-settingsd-1.0.1-remove-bashisms.patch | 81 ++++++++++++++++++++++
.../openrc-settingsd-1.0.1-r1.ebuild | 4 ++
2 files changed, 85 insertions(+)