[CRIU] [PATCH 02/11] nmk: Filter out already seen targets from collect-deps

Dmitry Safonov 0x7f454c46 at gmail.com
Wed Jul 18 18:44:07 MSK 2018


2018-07-18 16:42 GMT+01:00 Dmitry Safonov <0x7f454c46 at gmail.com>:
> 2018-07-18 16:41 GMT+01:00 Dmitry Safonov <0x7f454c46 at gmail.com>:
>> 2018-07-18 16:28 GMT+01:00 Cyrill Gorcunov <gorcunov at gmail.com>:
>>> On Wed, Jul 18, 2018 at 04:16:19PM +0100, Dmitry Safonov wrote:
>>> ...
>>>>
>>>> Ok, but then do we need those lines at all in collect-deps:
>>>> :         ifeq ($(builtin-target),$(1))
>>>> :                deps-y += $(obj-y:.o=.d)
>>>> :        endif
>>>> :        ifeq ($(lib-target),$(1))
>>>> :                deps-y += $(lib-y:.o=.d)
>>>> :        endif
>>>>
>>>> If your change will add them to deps-y for both cases.
>>>
>>> Yes we do, since ifeq and ifneq are different cases. Maybe
>>> this commit might be a way better, agreed. If you prefer
>>> I could clean it up later, the whole series, and resend.
>>> Presumably in a couple of weeks.
>>
>> Well, I think those are minor nits.. But the code there already
>> is a bit tricky and hard to follow, so I would prefer not leaving
>> something which is not needed.
>>
>> Should be easy to fix with small rebase or adding a patch on
>> the top - whatever you prefer.
>
> Probably, with a rebase, because than it looks possible to drop
> 6/7 patches, huh?

Err, not 6&7, but 8 patch is than not needed, sorry mislooked the
number.

-- 
             Dmitry


More information about the CRIU mailing list