[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v6 12/13] autoconf: xen: enable explicit preference option for xenstored preference
On Wed, 2014-07-02 at 22:09 +0200, Luis R. Rodriguez wrote: > On Wed, Jul 02, 2014 at 02:54:31PM +0100, Ian Campbell wrote: > > On Thu, 2014-06-12 at 18:18 -0700, Luis R. Rodriguez wrote: > > > Since the xenstored preference > > > is explicit now and since we require configure substitutions for it we > > > make use of the AX_XEN_EXPAND_CONFIG() helpers as otherwise substitution > > > for SBINDIR is not propagated from the top level configuration. > > > > I'm afraid I don't understand what is going on here. Why is SBINDIR not > > propagated? > > Its saying that we need AX_XEN_EXPAND_CONFIG() in order to do substitutions > for variables that include SBINDIR in this case for the xenstore full path. Specifically you mean that the autoconf/m4 code now uses these variables in addition to them appearing in Paths.mk? Strictly speaking you don't need the associated AC_SUBST's but they are harmless. > > > > Since we are now parsing an entry within Paths.mk.in on tools we let > > > the move the parsing of the file to be the tool's configure. > > > > I can't parse this. > > I meant to clarify that since we now have to do a substitution on tools > we move the calls to AX_XEN_EXPAND_CONFIG() there. This patch should have > also removed the AX_XEN_EXPAND_CONFIG() from the top level configure.ac. I think moving it would break stubdoms under ./configure --disable-tools? I think I would suggest keeping AX_XEN_EXPAND_CONFIG and the generation of config/Paths.mk in the top level configure but call AX_XEN_EXPAND_CONFIG again in tools/configure.ac without the generation of config/Paths.mk (I think the AC_SUBST's are benign for this second call) > After testing this though and removing AX_XEN_EXPAND_CONFIG() from > configure.ac > I do see though that substitution doesn't work, it only works if we leave > AX_XEN_EXPAND_CONFIG() on both places. I should have read this next para before commenting above! > This is the issue I mentioned that > I ran into and you had proposed we could try to figure out a fix if > this was happening. Well it is. Any ideas ? Or are you OK to live with > calling AX_XEN_EXPAND_CONFIG() twice, once on configure.ac and another > on tools/configure.ac ? Yes, so long as the generation of config/Paths.mk remains in the toplevel configure only. Ian. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |