From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] mk: fix dependencies when modifying config Date: Wed, 26 Mar 2014 23:14:41 +0100 Message-ID: <1944547.AP9M4ybhh5@xps13> References: <1395228982-24858-1-git-send-email-thomas.monjalon@6wind.com> <5332FB65.3000708@6wind.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Olivier MATZ Return-path: In-Reply-To: <5332FB65.3000708-pdR9zngts4EAvxtiuMwx3w@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2014-03-26 17:08, Olivier MATZ: > 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 > > Looks good to me. Just see one comment below. > > > ..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. Applied with the typo fix. Thanks for the review -- Thomas