[dpdk-dev] [PATCH 1/4] net/cxgbe: use relative paths for including header files
Rahul Lakkireddy
rahul.lakkireddy at chelsio.com
Wed Dec 19 13:42:03 CET 2018
On Tuesday, December 12/18/18, 2018 at 23:53:38 +0530, Ferruh Yigit wrote:
> On 12/13/2018 3:02 PM, Rahul Lakkireddy wrote:
> > Explicitly state header file location using relative paths.
> >
> > Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy at chelsio.com>
> > ---
> > drivers/net/cxgbe/Makefile | 2 --
> > drivers/net/cxgbe/base/adapter.h | 4 ++--
> > drivers/net/cxgbe/base/common.h | 2 +-
> > drivers/net/cxgbe/clip_tbl.c | 2 +-
> > drivers/net/cxgbe/cxgbe.h | 4 ++--
> > drivers/net/cxgbe/cxgbe_ethdev.c | 2 +-
> > drivers/net/cxgbe/cxgbe_filter.c | 7 ++++---
> > drivers/net/cxgbe/cxgbe_filter.h | 2 +-
> > drivers/net/cxgbe/cxgbe_flow.c | 2 +-
> > drivers/net/cxgbe/cxgbe_main.c | 6 +++---
> > drivers/net/cxgbe/cxgbevf_ethdev.c | 2 +-
> > drivers/net/cxgbe/cxgbevf_main.c | 6 +++---
> > drivers/net/cxgbe/l2t.c | 3 ++-
> > drivers/net/cxgbe/l2t.h | 2 +-
> > drivers/net/cxgbe/mps_tcam.h | 2 +-
> > drivers/net/cxgbe/sge.c | 6 +++---
> > 16 files changed, 27 insertions(+), 27 deletions(-)
> >
> > diff --git a/drivers/net/cxgbe/Makefile b/drivers/net/cxgbe/Makefile
> > index 68466f13e..d809f4720 100644
> > --- a/drivers/net/cxgbe/Makefile
> > +++ b/drivers/net/cxgbe/Makefile
> > @@ -9,8 +9,6 @@ include $(RTE_SDK)/mk/rte.vars.mk
> > #
> > LIB = librte_pmd_cxgbe.a
> >
> > -CFLAGS += -I$(SRCDIR)/base/
> > -CFLAGS += -I$(SRCDIR)
> > CFLAGS += -O3
> > CFLAGS += $(WERROR_FLAGS)
>
> Hi Rahul,
>
> What issue do you observe if you don't use relative paths?
The Intel C++ compiler [1] is not able to locate the header files without
relative path in Windows. We get the following error for these header
files.
# base\common.h(9): error : cannot open source file "cxgbe_compat.h"
#include "cxgbe_compat.h"
^
We could reproduce the same error by removing the above two lines for
Linux. To keep it consistent across both OS, I'm using relative paths
for both.
[1] https://software.intel.com/en-us/parallel-studio-xe
Thanks,
Rahul
More information about the dev
mailing list