All of lore.kernel.org
 help / color / mirror / Atom feed
From: Juergen Gross <juergen.gross@ts.fujitsu.com>
To: Jan Beulich <JBeulich@novell.com>
Cc: Keir Fraser <keir.xen@gmail.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: xen-unstable: build fails
Date: Thu, 17 Mar 2011 11:29:04 +0100	[thread overview]
Message-ID: <4D81E270.9000100@ts.fujitsu.com> (raw)
In-Reply-To: <4D81ECC1020000780003705D@vpn.id2.novell.com>

On 03/17/11 11:13, Jan Beulich wrote:
>>>> On 17.03.11 at 09:59, Juergen Gross<juergen.gross@ts.fujitsu.com>  wrote:
>> The problem is still the relative XEN_ROOT specification.
>> Or the missing .config under XEN_ROOT, so creating this file solves my
>> problem. :-)
>
> I'd favor the latter, e.g. by adding something like
>
> cfg-create := $(shell touch $(XEN_ROOT)/.config)
>
> to ./Config.mk, or putting the include in a conditional instead
> of making use of -include.

As short time measure this seems to be okay.

On the long run I'd prefer to use absolute paths only. Otherwise you've
always a risk to use something from outside the intended build environment.

XEN_ROOT = $(abspath ../..)

seems not too hard to use instead of

XEN_ROOT = ../..

And I'm still favoring usage of conditional setting of XEN_ROOT in included
Makefiles other than config/* to avoid hard to find problems.


Juergen

-- 
Juergen Gross                 Principal Developer Operating Systems
TSP ES&S SWE OS6                       Telephone: +49 (0) 89 3222 2967
Fujitsu Technology Solutions              e-mail: juergen.gross@ts.fujitsu.com
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html

  reply	other threads:[~2011-03-17 10:29 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-16 13:50 xen-unstable: build fails Juergen Gross
2011-03-16 14:45 ` Juergen Gross
2011-03-16 16:50 ` Keir Fraser
2011-03-17  5:52   ` Juergen Gross
2011-03-17  8:11     ` Keir Fraser
2011-03-17  8:59       ` Juergen Gross
2011-03-17 10:13         ` Jan Beulich
2011-03-17 10:29           ` Juergen Gross [this message]
2011-03-17 10:48             ` Jan Beulich
2011-03-17 10:55               ` Keir Fraser
2011-03-17 10:36         ` Keir Fraser
  -- strict thread matches above, loose matches on Subject: below --
2010-05-05  5:46 xen-unstable " Juergen Gross
2010-05-05  6:31 ` Shriram R
2010-05-05  9:00 ` Keir Fraser

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=4D81E270.9000100@ts.fujitsu.com \
    --to=juergen.gross@ts.fujitsu.com \
    --cc=JBeulich@novell.com \
    --cc=keir.xen@gmail.com \
    --cc=xen-devel@lists.xensource.com \
    /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.