mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Bug 116131: tools/perf and O= fails to build
@ 2016-04-10 21:36 TJ
  2016-04-11  7:40 ` Andy Shevchenko
  0 siblings, 1 reply; 4+ messages in thread
From: TJ @ 2016-04-10 21:36 UTC (permalink / raw)
  To: linux-kernel; +Cc: David Howells, Steven Rostedt

Trying to do an out-of-tree build using either absolute or relative 
paths for O= fails when trying to build tools/perf.

More details in ""Directory does not exist" when doing tools 
perf_install out-of-tree build"

https://bugzilla.kernel.org/show_bug.cgi?id=116131

git blame shows commits c883122a and bf35182f are responsible for the 
path calculation code.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Bug 116131: tools/perf and O= fails to build
  2016-04-10 21:36 Bug 116131: tools/perf and O= fails to build TJ
@ 2016-04-11  7:40 ` Andy Shevchenko
  2016-04-11  8:35   ` TJ
  0 siblings, 1 reply; 4+ messages in thread
From: Andy Shevchenko @ 2016-04-11  7:40 UTC (permalink / raw)
  To: TJ; +Cc: linux-kernel, David Howells, Steven Rostedt

On Mon, Apr 11, 2016 at 12:36 AM, TJ <linux@iam.tj> wrote:
> Trying to do an out-of-tree build using either absolute or relative paths
> for O= fails when trying to build tools/perf.
>
> More details in ""Directory does not exist" when doing tools perf_install
> out-of-tree build"
>
> https://bugzilla.kernel.org/show_bug.cgi?id=116131
>
> git blame shows commits c883122a and bf35182f are responsible for the path
> calculation code.
>

Create directory manually.
acme@ pointed me out to some commit where it's done on purpose.


-- 
With Best Regards,
Andy Shevchenko

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Bug 116131: tools/perf and O= fails to build
  2016-04-11  7:40 ` Andy Shevchenko
@ 2016-04-11  8:35   ` TJ
  2016-04-11  8:45     ` Andy Shevchenko
  0 siblings, 1 reply; 4+ messages in thread
From: TJ @ 2016-04-11  8:35 UTC (permalink / raw)
  To: Andy Shevchenko; +Cc: linux-kernel, David Howells, Steven Rostedt


On 11-04-2016 08:40, Andy Shevchenko wrote:
> On Mon, Apr 11, 2016 at 12:36 AM, TJ <linux@iam.tj> wrote:
>> Trying to do an out-of-tree build using either absolute or relative 
>> paths
>> for O= fails when trying to build tools/perf.
> 
> Create directory manually.
> acme@ pointed me out to some commit where it's done on purpose.

It's trying to use *the wrong* directory path entirely.

It looks to be caused by the sub-make calls that pass O=$(OUTPUT) 
because OUTPUT has been set to $(O)+subdir.

Then, in the sub-make tools/scripts/Makefile.include is called *again* 
and does the same $(O)+subdir resulting in basedir+subdir+subdir.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Bug 116131: tools/perf and O= fails to build
  2016-04-11  8:35   ` TJ
@ 2016-04-11  8:45     ` Andy Shevchenko
  0 siblings, 0 replies; 4+ messages in thread
From: Andy Shevchenko @ 2016-04-11  8:45 UTC (permalink / raw)
  To: TJ; +Cc: linux-kernel, David Howells, Steven Rostedt

On Mon, Apr 11, 2016 at 11:35 AM, TJ <linux@iam.tj> wrote:
>
> On 11-04-2016 08:40, Andy Shevchenko wrote:
>>
>> On Mon, Apr 11, 2016 at 12:36 AM, TJ <linux@iam.tj> wrote:
>>>
>>> Trying to do an out-of-tree build using either absolute or relative paths
>>> for O= fails when trying to build tools/perf.
>>
>>
>> Create directory manually.
>> acme@ pointed me out to some commit where it's done on purpose.
>
>
> It's trying to use *the wrong* directory path entirely.
>
> It looks to be caused by the sub-make calls that pass O=$(OUTPUT) because
> OUTPUT has been set to $(O)+subdir.
>
> Then, in the sub-make tools/scripts/Makefile.include is called *again* and
> does the same $(O)+subdir resulting in basedir+subdir+subdir.

Didn't remember if my case was the same, but as workaround I can
propose you to build in a dedicated worktree.

% git worktree add ../perf-test my_current_branch
% cd ../perf-test
% make (no O= provided) bla-bla-bla

-- 
With Best Regards,
Andy Shevchenko

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-04-11  8:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-10 21:36 Bug 116131: tools/perf and O= fails to build TJ
2016-04-11  7:40 ` Andy Shevchenko
2016-04-11  8:35   ` TJ
2016-04-11  8:45     ` Andy Shevchenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®