Re: [Xen-devel] [PATCH] build: save generated config in /boot

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Mon, 2015-12-21 at 08:35 -0600, Doug Goldstein wrote:
> On 12/21/15 6:11 AM, Jan Beulich wrote:
> > > > > On 18.12.15 at 22:35, <cardoe@xxxxxxxxxx> wrote:
> > > Since we now support changing Xen options with Kconfig, we should
> > > save
> > > the configuration that was used to build up Xen. This will save it in
> > > /boot alongside the installed xen.gz and call it
> > > xen-$(FULLVERSION).config
> > > 
> > > Suggested-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > > Signed-off-by: Doug Goldstein <cardoe@xxxxxxxxxx>
> > > ---
> > > Âxen/Makefile | 1 +
> > > Â1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/xen/Makefile b/xen/Makefile
> > > index 9023863..460b977 100644
> > > --- a/xen/Makefile
> > > +++ b/xen/Makefile
> > > @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX)
> > > Â ln -f -s $(T)-$(XEN_FULLVERSION)$(Z)
> > > $(D)$(BOOT_DIR)/$(T)$(Z)
> > > Â [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR)
> > > Â $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-
> > > $(XEN_FULLVERSION)
> > > + $(INSTALL_DATA) $(KCONFIG_CONFIG) $(D)$(BOOT_DIR)/$(T)-
> > > $(XEN_FULLVERSION).config
> > 
> > Was it really suggested to put this into /boot? It has no business
> > being there...
> > 
> > Jan
> > 
> 
> Yes. By multiple people. Ian Campbell was the first person to suggest it
> in that location.

I don't really care where it goes so long as it is somewhere which is (or
can become) well known.

IOW: I don't care to join in the argument about the location.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel