On Mon, 23 Jul 2018 10:50:53 +0200, Andrei Borzenkov wrote:
On Mon, Jul 23, 2018 at 11:41 AM, Michal Kubecek
wrote: On Monday, 23 July 2018 10:34 Andreas Schwab wrote:
On Jul 23 2018, Michal Kubecek
wrote: I don't. Checked ~20 minutes ago. And you still didn't explain what "all the goo" is supposed to mean.
Everything that's glued behind it.
Once again: I took a Tumbleweed system which didn't have libelf-devel, bison, flex, not even make and gcc installed. All of these are needed to build the kernel these days. Then I ran "zypper install kernel-source". And the only packages zypper decided to install were
bc kernel-devel kernel-macros kernel-source
Except kernel-macros and kernel-devel, the only "goo" it added was bc which was only added because your patch already got into Kernel:stable. In particular, it did _not_ install kernel-default-devel (or any other kernel-${flavor}-devel) and neither it did install any of the other packages needed to build the kernel.
"goo" is pulled in by kernel-default-devel (and even there make and gcc are just recommended, not required).
Right. So recap: - kernel-source.rpm itself has no hard dependency on toolchains; it has a Requires to kernel-devel.rpm at most, and kernel-devel has also no toolchain dependency, either. - But the toolchans can be grabbed via kernel-default-devel.rpm Supplements(kernel-default:kernel-devel). Even these are provided as recommends. - kernel-source.rpm contains kernel/time/timeconst.bc but no include/generated/timeconst.h, because include/generated/* go to kernel-*-devel package. (And the output of timeconst.h depends on HZ, so it's kconfig-dependent, hence not suitable for kernel-source.) - The lack of bc becomes a problem only when you build kernel from the tree of kernel-source.rpm due to timeconst.bc. - Since include/generated/timeconst.h is already included in kernel-default-devel, the KMP build doesn't need bc. bc is needed only when you build the kernel manually from scratch with kernel-source.rpm tree. So, we need a dependency on bc to some package. And, since it's an *optional* requirement *only* for kernel-source scratch build, the easiest option would be to "Recommends:bc" to kernel-source, supposedly. BTW, this also leads to another question: why flex and bison aren't needed. I thought they are also converted at compile time. Takashi -- To unsubscribe, e-mail: opensuse-kernel+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-kernel+owner@opensuse.org