Created attachment 682297 [details, diff]
Make configure.ac dash compatible.
Aegisub's configure script fails at the "checking for version" configure
step, because the line '. $srcdir/build/version.sh "$srcdir"' in
'configure.ac' doesn't work in dash.
I've attached a patch that'll make it work by replacing this:
. $srcdir/build/version.sh "$srcdir"
set -- "$srcdir"
set -- $old
Which saves the old configure arguments before modifying them with 'set --' and then restores them after sourcing 'version.sh'.
Created attachment 682309 [details]
build.log + emerge --info.txt
Can confirm, attaching own logs/info for completeness.
Thanks for the patch, can you make a PR please
The given patch could, in principle, damage the original positional parameter list. There is actually no need to manipulate the positional parameter list at all. I shall attach a cleaner patch momentarily.
Created attachment 682318 [details, diff]
Make configure.ac dash compatible (improved)
Revised patch that works by simply allowing for the sourced file to leverage the fact that the needed variable is already set. The hunk that applies to version.sh could be rendered neater still by instead removing the srcdir="$1" line. However, I had it retain the ability to collect the value from $1 for the reason stated in the commit message.
Thanks, can you make a github PR?
(In reply to Nick Sarnie from comment #5)
> Thanks, can you make a github PR?
I cannot in this instance. I disapprove of GitHub increasingly becoming the bar for open-source participation and cancelled my account some time ago. That said, if anyone else would like to run with it then they are more than welcome to. Alternatively, I could try emailing it to someone involved with the project.
Okay, do you give me your signed-off-by to push the patch to ::gentoo? I know it's in the patch yourself but I want to double check given your last comment about github. Note that Gentoo only uses github for mirroring and PRs and ::gentoo is not hosted by github.
Yes, absolutely. Anyone is free to commit/submit the patch wherever/however they please. It's just that I won't personally interact with anyone in a manner that requires me to register at GitHub and to effectively use it as a communication medium.
The bug has been closed via the following commit(s):
Author: Nick Sarnie <firstname.lastname@example.org>
AuthorDate: 2021-01-11 00:22:15 +0000
Commit: Nick Sarnie <email@example.com>
CommitDate: 2021-01-11 00:23:43 +0000
media-video/aegisub: Fix build with dash
Package-Manager: Portage-3.0.12, Repoman-3.0.2
Signed-off-by: Nick Sarnie <firstname.lastname@example.org>
.../aegisub/aegisub-3.2.2_p20160518-r101.ebuild | 158 +++++++++++++++++++++
...g-positional-parameters-to-source-builtin.patch | 47 ++++++
2 files changed, 205 insertions(+)
Apparently, the original patch was submitted as a PR against an up and coming fork: See https://github.com/wangqr/Aegisub/pull/89.