From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 5/6] xen: dont create dependency on cmdline library Date: Mon, 24 Mar 2014 22:01:05 +0100 Message-ID: <4807409.VkCA9iV8rJ@xps13> References: <20140307181335.024904493@vyatta.com> <20140307181430.240987691@vyatta.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org, Stephen Hemminger To: Stephen Hemminger Return-path: In-Reply-To: <20140307181430.240987691-ZtmgI6mnKB3QT0dZR+AlfA@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" 07/03/2014 10:13, Stephen Hemminger: > The driver should not introduce an unnecessary dependency on the cmdline > code. We don't build that code since it is not used in our product. Your product is not the concern but I agree this dependency is unnecessary. I'd just remove the comment about your product. > +/* defined in but that has conflicts with rte_ethdev.h */ > +extern struct ether_addr *ether_aton(const char *); It seems we should fix this conflict first. I think the problem is about libc redefinitions in rte_ether.h. Patches are welcome. -- Thomas