Re: [PATCH] scripts: Fallback to KBUILD_OUTPUT if O is not specified

Path	csiph.com!goblin2!goblin.stu.neva.ru!aioe.org!gothmog.csi.it!bofh.it!news.nic.it!robomod
From	Arnaldo Carvalho de Melo <acme@redhat.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] scripts: Fallback to KBUILD_OUTPUT if O is not specified
Date	Fri, 08 Apr 2016 17:50:02 +0200
Message-ID	<rlGPw-649-5@gated-at.bofh.it> (permalink)
References	<rlAK6-1xG-23@gated-at.bofh.it> <rlG39-5o2-39@gated-at.bofh.it> <rlGFR-60n-31@gated-at.bofh.it>
MIME-Version	1.0
Content-Type	text/plain; charset=us-ascii
Content-Disposition	inline
X-URL	http://acmel.wordpress.com
User-Agent	Mutt/1.5.20 (2009-12-10)
X-Scanned-By	MIMEDefang 2.68 on 10.5.11.23
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	77
Organization	linux.* mail to news gateway
X-Original-Cc	Thierry Reding <thierry.reding@gmail.com>, linux-kernel@vger.kernel.org, Jiri Olsa <jolsa@kernel.org>, Arnaldo Carvalho de Melo <acme@kernel.org>
X-Original-Date	Fri, 8 Apr 2016 12:48:46 -0300
X-Original-Message-ID	<20160408154846.GG5327@redhat.com>
X-Original-References	<1460106849-6435-1-git-send-email-thierry.reding@gmail.com> <20160408145847.GE5327@redhat.com> <5707D022.90002@huawei.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1374321

Show key headers only | View raw

Em Fri, Apr 08, 2016 at 11:37:06PM +0800, Wangnan (F) escreveu:
> 
> 
> On 2016/4/8 22:58, Arnaldo Carvalho de Melo wrote:
> >Em Fri, Apr 08, 2016 at 11:14:09AM +0200, Thierry Reding escreveu:
> >>From: Thierry Reding <treding@nvidia.com>
> >>
> >>The KBUILD_OUTPUT environment can be used to set the build directory as
> >>an alternative to passing the O variable on the command-line. This works
> >>with regular kernel builds, so it's surprising if it doesn't work within
> >>the tools/scripts directory.
> >>
> >>Allow KBUILD_OUTPUT if O is not specified on the command-line to avoid
> >>any surprises.
> >I think this is fine, Jiri, Wang?
> 
> The idea is fine, but we should be very careful when touching Makefile

Yeah, I haven't tested it, Thierry, would you try the 'make -C
tools/perf build-test' with this set up?

> to avoid breaking build-test like this:
> 
>  # export KBUILD_OUTPUT=/tmp/xxxxx
>  #  ./tests/perf-targz-src-pkg .
>    PERF_VERSION = 4.5.g337b7b
>  ls: cannot access perf-*.tar.gz: No such file or directory
>  tar: -C: Cannot open: No such file or directory
>  tar: Error is not recoverable: exiting now
>  make: *** /tmp/tmp.q4egWrFd3L/perf*/tools/perf: No such file or
> directory.  Stop
> 
> Moreover, in the above case, the output directory should be
> 
>  /tmp/xxxxx
> 
> or
> 
>  /tmp/xxxxx/tools/perf ?
> 
> Since kernel build output things to /tmp/xxxxx, I think we'd better
> output to /tmp/xxxxx/tools/perf.
> 
> Thank you.
> 
> >- Arnaldo
> >>Signed-off-by: Thierry Reding <treding@nvidia.com>
> >>---
> >>Hi Arnaldo,
> >>
> >>I'm not sure if you're the right person to send this to. There was noone
> >>reported by scripts/get_maintainer.pl, but it looks like you ended up
> >>applying most of the patches to this file lately, which kinda makes you
> >>the de facto maintainer =).
> >>
> >>Thierry
> >>
> >>  tools/scripts/Makefile.include | 4 ++++
> >>  1 file changed, 4 insertions(+)
> >>
> >>diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include
> >>index 8abbef164b4e..5467da41dc72 100644
> >>--- a/tools/scripts/Makefile.include
> >>+++ b/tools/scripts/Makefile.include
> >>@@ -8,6 +8,10 @@ ifeq ($(objtree),)
> >>  	objtree := $(O)
> >>  endif
> >>  endif
> >>+else
> >>+ifneq ($(KBUILD_OUTPUT),)
> >>+	OUTPUT := $(KBUILD_OUTPUT)/$(if $(subdir),$(subdir)/)
> >>+endif
> >>  endif
> >>  # check that the output directory actually exists
> >>-- 
> >>2.8.0
>

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] scripts: Fallback to KBUILD_OUTPUT if O is not specified Thierry Reding <thierry.reding@gmail.com> - 2016-04-08 11:20 +0200
  Re: [PATCH] scripts: Fallback to KBUILD_OUTPUT if O is not specified Arnaldo Carvalho de Melo <acme@redhat.com> - 2016-04-08 17:00 +0200
    Re: [PATCH] scripts: Fallback to KBUILD_OUTPUT if O is not specified "Wangnan (F)" <wangnan0@huawei.com> - 2016-04-08 17:40 +0200
      Re: [PATCH] scripts: Fallback to KBUILD_OUTPUT if O is not specified Arnaldo Carvalho de Melo <acme@redhat.com> - 2016-04-08 17:50 +0200

csiph-web