[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+r1Zhho-mqtM6Z76pCPFOvxO3n0KR_2Q6fLXJvtBZv5YSU24Q@mail.gmail.com>
Date: Mon, 27 Nov 2017 16:57:58 -0700
From: Jim Davis <jim.epost@...il.com>
To: Henning Schild <henning.schild@...mens.com>
Cc: linux-kernel <linux-kernel@...r.kernel.org>,
Ben Hutchings <ben.hutchings@...ethink.co.uk>,
Masahiro Yamada <yamada.masahiro@...ionext.com>,
Michal Marek <michal.lkml@...kovi.net>,
linux-kbuild <linux-kbuild@...r.kernel.org>,
Konrad Schwarz <konrad.schwarz@...mens.com>
Subject: Re: [PATCH] builddeb: introduce variables for control-file customization
On Mon, Nov 27, 2017 at 9:13 AM, Henning Schild
<henning.schild@...mens.com> wrote:
> The debian packages coming out of "make *deb-pkg" lack some critical
> information in the control-files e.g. the "Depends:" field. If one
> tries to install a fresh system with such a "linux-image" debootstrap or
> multistrap might try to install the kernel before its deps and the
> package hooks will fail.
Has that shown up in practice? The builddeb script goes back some years now...
> + eval 'create_package "$kernel_headers_packagename" \
> + "$kernel_headers_dir"' "$KDEB_OPTS_IMAGE_HEADERS"
eval in a shell script with arbitrary input can lead to shenanigans like
make bindeb-pkg KDEB_OPTS_IMAGE_HEADERS="; echo All your base"
and other potentially nastier things. Probably not an issue for a
typical kernel developer sitting in front of his or her laptop, but if
I ran a big automated unattended build farm I might prefer a non-eval
alternative.
--
Jim
Powered by blists - more mailing lists