[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] build: sync Kconfig with Linux v4.17

>>> On 21.06.18 at 04:10, <dougtrav@xxxxxxxxxx> wrote:
> From: Doug Goldstein <cardoe@xxxxxxxxxx>
> From: Doug Goldstein <cardoe@xxxxxxxxxx>
> Import the following files and directories from the Linux v4.17 tag
> (commit id 29dcea88779c856c7dc92040a0c01233263101d4):
> - scripts/kconfig/ -> xen/tools/kconfig/
> - scripts/Makefile.host -> xen/tools/kconfig/Makefile.host
> - Documentation/kbuild/kconfig{,-language}.txt ->
>   docs/misc/kconfig{-language}.txt
> - include/linux/kconfig.h -> xen/include/xen/kconfig.h
> Pulled in parts of scripts/Makefile.lib into
> xen/tools/kconfig/Makefile.kconfig
> Side effect of this change is that flex and bison are required to build
> Kconfig. Linux has switched from shipping the pre-generated files to
> always generating them with flex and bison.
> Signed-off-by: Doug Goldstein <cardoe@xxxxxxxxxx>

What I'm missing (and what I think is a requirement for this to go in) is
the "why" part: What do we gain from doing the update? Besides that
I think that in general it would be better to pull in Linux commits one
by one, rather than combining everything into a giant patch. That
might even already address part of my question, as it could highlight
bug fixes and/or enhancements which are of interest to us.

> So this makes flex & bison required to build Kconfig which means to
> build the hypervisor now these are required. These are already
> documented in the README as required but that was previously just for
> the toolstack. Since these are GNU utilities I would ask that non-Linux
> folks (e.g. Roger)

Well, I'm curious to see how well this fares on old distros, but so far I
had no issues building Linux 4.17.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.