Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.debian.kernel > #62717 > unrolled thread
| Started by | riku.voipio@linaro.org |
|---|---|
| First post | 2018-12-13 09:30 +0100 |
| Last post | 2018-12-21 09:10 +0100 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.debian.kernel
[PATCH] deb-pkg: generate correct build dependencies riku.voipio@linaro.org - 2018-12-13 09:30 +0100
Re: [PATCH] deb-pkg: generate correct build dependencies Masahiro Yamada <yamada.masahiro@socionext.com> - 2018-12-13 12:30 +0100
Re: [PATCH] deb-pkg: generate correct build dependencies Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> - 2018-12-21 09:10 +0100
| From | riku.voipio@linaro.org |
|---|---|
| Date | 2018-12-13 09:30 +0100 |
| Subject | [PATCH] deb-pkg: generate correct build dependencies |
| Message-ID | <x4uxz-8rA-5@gated-at.bofh.it> |
From: Riku Voipio <riku.voipio@linaro.org> bison/flex is now needed always for building since genksyms is always generated. Some build dependencies depend on kernel configuration, enable them as needed: - libelf-dev when UNWINDER_ORC is set - libssl-dev for SYSTEM_TRUSTED_KEYRING Since the libssl-dev is needed for extract_cert binary, denote with :native to install the libssl-dev for the build machines architecture, rather than for the architecture of the kernel being built. Signed-off-by: Riku Voipio <riku.voipio@linaro.org> --- scripts/package/mkdebian | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian index edcad61fe3cd..c858abf4569e 100755 --- a/scripts/package/mkdebian +++ b/scripts/package/mkdebian @@ -134,6 +134,8 @@ fi mkdir -p debian/ echo $debarch > debian/arch +extra_build_depends=", $(if_enabled_echo UNWINDER_ORC libelf-dev)" +extra_build_depends="$extra_build_depends, $(if_enabled_echo SYSTEM_TRUSTED_KEYRING libssl-dev:native)" # Generate a simple changelog template cat <<EOF > debian/changelog @@ -170,7 +172,7 @@ Source: $sourcename Section: kernel Priority: optional Maintainer: $maintainer -Build-Depends: bc, kmod, cpio +Build-Depends: bc, kmod, cpio, bison, flex $extra_build_depends Homepage: http://www.kernel.org/ Package: $packagename -- 2.19.2
[toc] | [next] | [standalone]
| From | Masahiro Yamada <yamada.masahiro@socionext.com> |
|---|---|
| Date | 2018-12-13 12:30 +0100 |
| Message-ID | <x4xlL-1LY-1@gated-at.bofh.it> |
| In reply to | #62717 |
On Thu, Dec 13, 2018 at 5:07 PM <riku.voipio@linaro.org> wrote: > > From: Riku Voipio <riku.voipio@linaro.org> The code looks good to me. Just a comment about the commit log. > bison/flex is now needed always for building since genksyms is always > generated. I do not think this is true. genksyms is built when CONFIG_MODVERSIONS=y (See scripts/Makefile) Rather, bison/flex is always needed for kconfig. > Some build dependencies depend on kernel configuration, > enable them as needed: > > - libelf-dev when UNWINDER_ORC is set > - libssl-dev for SYSTEM_TRUSTED_KEYRING > > Since the libssl-dev is needed for extract_cert binary, denote with > :native to install the libssl-dev for the build machines architecture, > rather than for the architecture of the kernel being built. > > Signed-off-by: Riku Voipio <riku.voipio@linaro.org> > --- > scripts/package/mkdebian | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian > index edcad61fe3cd..c858abf4569e 100755 > --- a/scripts/package/mkdebian > +++ b/scripts/package/mkdebian > @@ -134,6 +134,8 @@ fi > > mkdir -p debian/ > echo $debarch > debian/arch > +extra_build_depends=", $(if_enabled_echo UNWINDER_ORC libelf-dev)" > +extra_build_depends="$extra_build_depends, $(if_enabled_echo SYSTEM_TRUSTED_KEYRING libssl-dev:native)" > > # Generate a simple changelog template > cat <<EOF > debian/changelog > @@ -170,7 +172,7 @@ Source: $sourcename > Section: kernel > Priority: optional > Maintainer: $maintainer > -Build-Depends: bc, kmod, cpio > +Build-Depends: bc, kmod, cpio, bison, flex $extra_build_depends > Homepage: http://www.kernel.org/ > > Package: $packagename > -- > 2.19.2 > -- Best Regards Masahiro Yamada
[toc] | [prev] | [next] | [standalone]
| From | Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> |
|---|---|
| Date | 2018-12-21 09:10 +0100 |
| Message-ID | <x7o2B-8qe-7@gated-at.bofh.it> |
| In reply to | #62717 |
On Thu, Dec 13, 2018 at 10:07:11AM +0200, riku.voipio@linaro.org wrote: > From: Riku Voipio <riku.voipio@linaro.org> > > bison/flex is now needed always for building since genksyms is always > generated. Some build dependencies depend on kernel configuration, > enable them as needed: > > - libelf-dev when UNWINDER_ORC is set > - libssl-dev for SYSTEM_TRUSTED_KEYRING > > Since the libssl-dev is needed for extract_cert binary, denote with > :native to install the libssl-dev for the build machines architecture, > rather than for the architecture of the kernel being built. > > Signed-off-by: Riku Voipio <riku.voipio@linaro.org> Tested-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Regards, Mani > --- > scripts/package/mkdebian | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian > index edcad61fe3cd..c858abf4569e 100755 > --- a/scripts/package/mkdebian > +++ b/scripts/package/mkdebian > @@ -134,6 +134,8 @@ fi > > mkdir -p debian/ > echo $debarch > debian/arch > +extra_build_depends=", $(if_enabled_echo UNWINDER_ORC libelf-dev)" > +extra_build_depends="$extra_build_depends, $(if_enabled_echo SYSTEM_TRUSTED_KEYRING libssl-dev:native)" > > # Generate a simple changelog template > cat <<EOF > debian/changelog > @@ -170,7 +172,7 @@ Source: $sourcename > Section: kernel > Priority: optional > Maintainer: $maintainer > -Build-Depends: bc, kmod, cpio > +Build-Depends: bc, kmod, cpio, bison, flex $extra_build_depends > Homepage: http://www.kernel.org/ > > Package: $packagename > -- > 2.19.2 >
[toc] | [prev] | [standalone]
Back to top | Article view | linux.debian.kernel
csiph-web