[Accel-config] Re: [PATCH] accel-config: Remove non-interface headers from packaging

On 3/26/2021 4:25 PM, ramesh.thomas(a)intel.com wrote:
> From: Ramesh Thomas <ramesh.thomas(a)intel.com>
>
> Non interface files were getting included into interface files causing
> compilation errors. Other non-interface files were getting included into
> packaging.
>
> Signed-off-by: Ramesh Thomas <ramesh.thomas(a)intel.com>
> Reported-by: Ziye Yang <ziye.yang(a)intel.com>


Reviewed-by: Dave Jiang <dave.jiang(a)intel.com>


> ---
>   accfg/lib/Makefile.am   | 2 +-
>   accfg/libaccel_config.h | 1 -
>   2 files changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/accfg/lib/Makefile.am b/accfg/lib/Makefile.am
> index 2560314..64610e1 100644
> --- a/accfg/lib/Makefile.am
> +++ b/accfg/lib/Makefile.am
> @@ -3,7 +3,7 @@ include $(top_srcdir)/Makefile.am.in
>   %.pc: %.pc.in Makefile
>   	$(SED_PROCESS)
>   
> -pkginclude_HEADERS = ../libaccel_config.h ../accfg.h
> +pkginclude_HEADERS = ../libaccel_config.h
>   lib_LTLIBRARIES = libaccel-config.la
>   
>   DISTCLEANFILES = config.h
> diff --git a/accfg/libaccel_config.h b/accfg/libaccel_config.h
> index b7b64c4..94aef80 100644
> --- a/accfg/libaccel_config.h
> +++ b/accfg/libaccel_config.h
> @@ -11,7 +11,6 @@
>   #include <errno.h>
>   #include <limits.h>
>   #include <uuid/uuid.h>
> -#include <util/filter.h>
>   
>   #ifdef __cplusplus
>   extern "C" {