Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1383168 > unrolled thread

Re: [PATCH] kbuild: drop redundant "PHONY += FORCE"

Started byMichal Marek <mmarek@suse.cz>
First post2016-04-20 10:50 +0200
Last post2016-04-25 14:10 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH] kbuild: drop redundant "PHONY += FORCE" Michal Marek <mmarek@suse.cz> - 2016-04-20 10:50 +0200
    Re: [PATCH] kbuild: drop redundant "PHONY += FORCE" Masahiro Yamada <yamada.masahiro@socionext.com> - 2016-04-20 11:00 +0200
      Re: [PATCH] kbuild: drop redundant "PHONY += FORCE" Michal Marek <mmarek@suse.cz> - 2016-04-25 14:10 +0200

#1383168 — Re: [PATCH] kbuild: drop redundant "PHONY += FORCE"

FromMichal Marek <mmarek@suse.cz>
Date2016-04-20 10:50 +0200
SubjectRe: [PATCH] kbuild: drop redundant "PHONY += FORCE"
Message-ID<rpVZE-4XX-15@gated-at.bofh.it>
On Fri, Apr 08, 2016 at 11:16:10AM +0900, Masahiro Yamada wrote:
> "PHONY += FORCE" is already cared by scripts/Makefile.build,
> which these files are included from.

arch/ia64/Makefile is in fact included via the main Makefile. But this
one also has the PHONY += FORCE statement. Maybe we should initialize
PHONY with FORCE in Kbuild.include. Nevertheless, this patch is correct.

Michal

[toc] | [next] | [standalone]


#1383171

FromMasahiro Yamada <yamada.masahiro@socionext.com>
Date2016-04-20 11:00 +0200
Message-ID<rpW9k-51U-3@gated-at.bofh.it>
In reply to#1383168
Hi Michal,

2016-04-20 17:47 GMT+09:00 Michal Marek <mmarek@suse.cz>:
> On Fri, Apr 08, 2016 at 11:16:10AM +0900, Masahiro Yamada wrote:
>> "PHONY += FORCE" is already cared by scripts/Makefile.build,
>> which these files are included from.
>
> arch/ia64/Makefile is in fact included via the main Makefile. But this
> one also has the PHONY += FORCE statement. Maybe we should initialize
> PHONY with FORCE in Kbuild.include. Nevertheless, this patch is correct.
>
> Michal

Ah, right.

If it is not too late and you do not mind,
could you rephrase the git-log?

Like this?

----------------------->8------------------------------------
"PHONY += FORCE" is already cared by scripts/Makefile.build or the
top level Makefile, which these files are included from.
-----------------------8<------------------------------------



-- 
Best Regards
Masahiro Yamada

[toc] | [prev] | [next] | [standalone]


#1386319

FromMichal Marek <mmarek@suse.cz>
Date2016-04-25 14:10 +0200
Message-ID<rrNuW-5F5-5@gated-at.bofh.it>
In reply to#1383171
On 2016-04-20 10:56, Masahiro Yamada wrote:
> Hi Michal,
> 
> 2016-04-20 17:47 GMT+09:00 Michal Marek <mmarek@suse.cz>:
>> On Fri, Apr 08, 2016 at 11:16:10AM +0900, Masahiro Yamada wrote:
>>> "PHONY += FORCE" is already cared by scripts/Makefile.build,
>>> which these files are included from.
>>
>> arch/ia64/Makefile is in fact included via the main Makefile. But this
>> one also has the PHONY += FORCE statement. Maybe we should initialize
>> PHONY with FORCE in Kbuild.include. Nevertheless, this patch is correct.
>>
>> Michal
> 
> Ah, right.
> 
> If it is not too late and you do not mind,
> could you rephrase the git-log?

Hi Masahiro,

I'd rather not rewrite the git log. It was just a minor nitpick and
there is not so much going on in arch/ia64 to cause confusion.

Michal

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web