Re: [RFCv2 00/13] Kbuild: factor parser rules

Arnaud Lacombe
2011-05-24T14:18:01Z

Hi,

2011/5/24 Michal Marek <mmarek@suse.cz>:
> On 23.5.2011 10:10, Arnaud Lacombe wrote:
>>
>> Hi,
>>
>> [ Original RFC and motivation can be found at:
>> http://marc.info/?l=linux-kbuild&m=130456101131801&w=2 ]
>>
>> I tried to re-think the order in this serie to address Michal comments.
>>
>> Main changes since v1 are:
>>  - include scripts/dtc/' parser in the scope of the patchset
>>  - do not rename any parser source
>>  - make lexer file name consistent, ie. name it %.lex.c, not lex.%.c
>>  - rebase on top of v2.6.39
>>
>> A few notes:
>>  - the parser should not include the lexer; however, this has the
>> advantage to
>>    avoid having to deals with the parser's header.
>>  - the tuple ( "migrate parser to implicit rules", regen parser) should
>> really
>>    be one commit, to enforce bisect'ability
>>
>> Comments welcome!
>>  - Arnaud
>
> Hi,
>
> thanks for the series, but I will probably not have time to review it before
> this merge window closes. So it will have to wait for .41 most likely.
>
fine with me, I am not entirely happy with that serie anyway, so I'll
certainly do another iteration after -rc1 (to include whatever has
changed in the mean time).

- Arnaud