All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Olaf Hering <olaf@aepfle.de>
Cc: Keir Fraser <keir@xen.org>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Tim Deegan <tim@xen.org>, Ian Jackson <ian.jackson@eu.citrix.com>,
	xen-devel@lists.xen.org, Jan Beulich <jbeulich@suse.com>,
	Samuel Thibault <samuel.thibault@ens-lyon.org>
Subject: Re: [PATCH 02/13] Config.mk: replace dependency to genpath with actual target
Date: Thu, 18 Sep 2014 18:44:46 +0100	[thread overview]
Message-ID: <1411062286.1920.17.camel@citrix.com> (raw)
In-Reply-To: <20140918080932.GA5049@aepfle.de>

On Thu, 2014-09-18 at 10:09 +0200, Olaf Hering wrote:
> On Wed, Sep 17, Ian Campbell wrote:
> 
> > On Wed, 2014-09-17 at 12:02 +0200, Olaf Hering wrote:
> > > A question related to the change below:
> > > 
> > > How are the make variables supposed to be changed? Right now this can be
> > > done either with rerunning configure (untested by me) and by calling
> > > something like 'make SBINDIR=/x/y XEN_RUN_DIR=/a/b' (used as testcase).
> > > 
> > > I think we should arrange the code such that a new configure run is
> > > required to change their value.
> > 
> > I think we've been moving (and should be moving) in the direction of
> > configuring these things via configure, so that would be ok by me. (I'm
> > assuming these don't affect the hypervisor build/install, which isn't
> > configure driven)
> 
> Unfortunately, m4/paths.m4 shows that configure will change just prefix
> and libdir. Other parts of config/Paths.mk.in are kind of hardcoded.
> I will see how to implement a knob for at least CONFIG_DIR and INITD_DIR.

I think there is an existing autoconf name for CONFIG_DIR (sysconfdir?)
which I suppose we aren't properly using.

Not sure how INITD_DIR is dealt with in autofoo world, I suppose the
default is $sysconfdir/init.d? Does it ever get set to anything else?

Ian.

  reply	other threads:[~2014-09-18 17:44 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-28  9:05 [PATCH 00/13 v2] tools changes to honor --prefix= Olaf Hering
2014-07-28  9:05 ` [PATCH 01/13] Config.mk: move directory list into BUILD_MAKE_VARS Olaf Hering
2014-07-28  9:05 ` [PATCH 02/13] Config.mk: replace dependency to genpath with actual target Olaf Hering
2014-08-26 20:00   ` Ian Campbell
2014-08-27  8:36     ` Olaf Hering
2014-08-28 17:34       ` Ian Campbell
2014-08-29  6:33         ` Olaf Hering
2014-09-03 10:28           ` Ian Campbell
2014-09-03 10:43             ` Olaf Hering
2014-09-17 10:02   ` Olaf Hering
2014-09-17 16:53     ` Ian Campbell
2014-09-18  8:09       ` Olaf Hering
2014-09-18 17:44         ` Ian Campbell [this message]
2014-09-19  6:55           ` Olaf Hering
2014-07-28  9:05 ` [PATCH 03/13] Config.mk: add new macro buildmakevars2header Olaf Hering
2014-08-26 20:02   ` Ian Campbell
2014-07-28  9:05 ` [PATCH 04/13] tools/libxl: use buildmakevars2header to create _paths.h Olaf Hering
2014-07-28  9:05 ` [PATCH 05/13] tools/libxc: provide variable paths to libxc Olaf Hering
2014-07-28  9:05 ` [PATCH 06/13] tools/libxc: use XEN_RUN_DIR for SUSPEND_LOCK_FILE Olaf Hering
2014-07-28  9:05 ` [PATCH 07/13] tools/pygrub: store kernels in /var/run/xen/pygrub Olaf Hering
2014-08-26 20:04   ` Ian Campbell
2014-07-28  9:05 ` [PATCH 08/13] tools/hotplug: use XEN_SCRIPT_DIR instead of hardcoded path Olaf Hering
2014-08-26 20:07   ` Ian Campbell
2014-08-27  8:42     ` Olaf Hering
2014-07-28  9:05 ` [PATCH 09/13] tools/hotplug: create XEN_RUN_DIR at runtime Olaf Hering
2014-07-28  9:05 ` [PATCH 10/13] tools/hotplug: create XEN_LOCK_DIR " Olaf Hering
2014-07-28  9:05 ` [PATCH 11/13] tools/hotplug: use XEN_LOCK_DIR instead of hardcoded path Olaf Hering
2014-08-26 20:08   ` Ian Campbell
2014-07-28  9:05 ` [PATCH 12/13] tools/examples: remove obsolete install targets Olaf Hering
2014-07-28  9:05 ` [PATCH 13/13] remove obsolete SUBSYS_DIR variable Olaf Hering
2014-08-26 20:08   ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1411062286.1920.17.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jbeulich@suse.com \
    --cc=keir@xen.org \
    --cc=olaf@aepfle.de \
    --cc=samuel.thibault@ens-lyon.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.