linux-kbuild.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Michal Marek <mmarek@suse.cz>
To: Riku Voipio <riku.voipio@linaro.org>,
	linux-kbuild <linux-kbuild@vger.kernel.org>
Subject: Re: [PATCH v3] package Makefile: fix perf-tar targets when outdir is set
Date: Tue, 24 Nov 2015 17:50:13 +0100	[thread overview]
Message-ID: <56549545.5060908@suse.cz> (raw)
In-Reply-To: <CAAqcGH=s3VL8y4QzycZR0ueMHmwJ4UcyUZMP4sjxSteb-X_itA@mail.gmail.com>

On 2015-11-16 11:09, Riku Voipio wrote:
> On 1 September 2015 at 17:14,  <riku.voipio@linaro.org> wrote:
>> From: Riku Voipio <riku.voipio@linaro.org>
>>
>> building with $srctree != $objtree, perf-tar-* targets fail
>> to read the MANIFEST file and add the PERF-VERSION-FILE needed
>> by out-of-tree builds. The build errors and an incorrect tar is created:
>>
>> $ make O=build-x86 perf-targz-src-pkg
>>   TAR
>> cat: ../tools/perf/MANIFEST: No such file or directory
>> tar: perf-4.1.0-rc8/PERF-VERSION-FILE: Cannot stat: No such file or
>> dir..
>> tar: Exiting with failure status due to previous errors
>>
>> Kbuild sets objtree to "." and srctree to ".." The command to output
>> MANIFEST becomes:
>>
>>    $(cd ..; echo $(cat ../tools/perf/MANIFEST))
>>
>> Without MANIFEST, the entire kernel source tree is added to the perf
>> source tarball. The *correct* fix is to keep the cd and remove srctree
>> from cat command line since MANIFEST has wildcards that fail to expand
>> working directory isn't srctree.
>>
>> Second, PERF-VERSION-FILE gets not added, because in-tree build path is
>> hardcoded to Makefile:
>>
>>    util/PERF-VERSION-GEN ../../$(perf-tar)/ 2>/dev/null)
>>
>> The PERF-VERSION-GEN needs to be run from tools/perf directory,
>> and the output directory needs to be changed from relative to
>> to absolute. This can be achieved using the $(CURDIR) variable.
>>
>> Also remove the error redirect to /dev/null which hid the error.
>>
>> Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
> 
> Ping?

Added to kbuild.git#misc now, sorry for the delay.

Michal


      reply	other threads:[~2015-11-24 16:50 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-01 14:14 [PATCH v3] package Makefile: fix perf-tar targets when outdir is set riku.voipio
2015-11-16 10:09 ` Riku Voipio
2015-11-24 16:50   ` Michal Marek [this message]

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=56549545.5060908@suse.cz \
    --to=mmarek@suse.cz \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=riku.voipio@linaro.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).