[dpdk-dev] [PATCH] mk: fix dependencies when modifying config

Olivier MATZ olivier.matz at 6wind.com
Wed Mar 26 17:08:05 CET 2014


Hi Thomas,

On 03/19/2014 12:36 PM, Thomas Monjalon wrote:
> If .config file is changed and .depdirs was built before,
> it must be updated.
> It is now done silently (-s) in checkconfig rule.
> Now .depdirs is not redone if .config is older.
>
> It has been tested by enabling PMD_PCAP after a default build.
> KVARGS dependency should appear in .depdirs.
>
> Signed-off-by: Thomas Monjalon <thomas.monjalon at 6wind.com>
> ---

Looks good to me. Just see one comment below.

>   # use a "for" in a shell to process dependencies: we don't want this
>   # task to be run in parallel.
>   ..PHONY: depdirs

It seems there is a typo (..PHONY instead of .PHONY), it could
be a good occasion to fix it. In my opinion, it can be done in the
same patch as this typo does not create a real issue.

Regards,
Olivier



More information about the dev mailing list