[pve-devel] [PATCH kernel v2] d/control: add some missing build dependencies
Fabian Grünbichler
f.gruenbichler at proxmox.com
Tue Apr 24 09:38:40 CEST 2018
On Tue, Apr 24, 2018 at 08:07:14AM +0200, Thomas Lamprecht wrote:
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> ---
>
> changes v1 -> v2:
> * omit libssl-dev for now
seems I was wrong there - libssl-dev is needed for some module signing
scripts (which are LGPL 2.1+), which we don't use, but some users might.
it is already explicitly disabled for the (GPL-2 only) perf binary,
where it would be problematic:
debian/rules:
128: ${MAKE} -C ${KERNEL_SRC}/tools/perf prefix=/usr HAVE_CPLUS_DEMANGLE=1 NO_LIBPYTHON=1 NO_LIBPERL=1 NO_LIBCRYPTO=1 PYTHON=python2.7
we probably want to add 'feature-libbfd=0' to prevent accidentally
linking with libbfd from binutils-dev (which is GPL-3+ only, and thus
not compatible with GPL-2 only perf).
I'll apply a combination of your v1 and v2 (with libssl-dev and the
comma-dot fix), and send the latter + a check with ldd as separate
patch.
> * replace wrong dot after 'xmlto' with comma
>
> debian/control.in | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/debian/control.in b/debian/control.in
> index fa58872..18e8381 100644
> --- a/debian/control.in
> +++ b/debian/control.in
> @@ -5,6 +5,11 @@ Maintainer: Proxmox Support Team <support at proxmox.com>
> Build-Depends: gcc-6 (>= 6.3.0-18+deb9u1),
> lintian,
> sed,
> + flex,
> + bison,
> + bc,
> + asciidoc,
> + xmlto,
> tar,
> Build-Conflicts: pve-headers- at KVNAME@,
> Vcs-Git: git://git.proxmox.com/git/pve-kernel
> --
> 2.14.2
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
More information about the pve-devel
mailing list