[arch-projects] [devtools] [PATCH 3/7] arch-nspawn: make sure that makepkg.conf is always parsed as text
lukeshu at lukeshu.com
Mon Jan 15 19:29:56 UTC 2018
On Mon, 15 Jan 2018 12:11:29 -0500,
Eli Schwartz wrote:
> On 01/15/2018 11:57 AM, Luke Shumaker wrote:
> > From: Luke Shumaker <lukeshu at parabola.nu>
> > https://lists.parabola.nu/pipermail/dev/2017-June/005576.html
> > ---
> > arch-nspawn.in | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> > diff --git a/arch-nspawn.in b/arch-nspawn.in
> > index c55f498..3949ee1 100644
> > --- a/arch-nspawn.in
> > +++ b/arch-nspawn.in
> > @@ -107,7 +107,7 @@ fi
> > build_mount_args
> > copy_hostconf
> > -eval "$(grep '^CARCH=' "$working_dir/etc/makepkg.conf")"
> > +eval "$(grep -a '^CARCH=' "$working_dir/etc/makepkg.conf")"
> > [[ -z $nosetarch ]] || unset CARCH
> I'm curious, in what situation were you having grep parse makepkg.conf
> as not text?
Some unicode in a comment near the top was tripping it up (this was on
the box of a Spanish-speaking developer).
> I also wonder whether we should source makepkg.conf in a subshell and
> printf the relevant variables...
I have no problem with that.
~ Luke Shumaker
More information about the arch-projects