This change is fine, but it also needs the packageconfig change to match.
The thread from October has the full patch (title was "[PATCH v2]
spec/pkgconfig: Allow builds to override headers"). This also needs a
change to libbsd to account for this alteration, IIRC.

Kinsey

On Wed, Apr 12, 2023 at 5:20 PM <chr...@rtems.org> wrote:

> From: Chris Johns <chr...@rtems.org>
>
> - Export the BSP include path as a system include path so the order
>   of options on an application compiler command line does not matter.
>
> Closes #4896
> ---
>  spec/build/bsps/makecustom.yml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/spec/build/bsps/makecustom.yml
> b/spec/build/bsps/makecustom.yml
> index 139629b597..c00240595e 100644
> --- a/spec/build/bsps/makecustom.yml
> +++ b/spec/build/bsps/makecustom.yml
> @@ -2,7 +2,7 @@ SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause
>  build-type: config-file
>  content: |
>    include $$(RTEMS_ROOT)/make/custom/default.cfg
> -  CPU_DEFINES = -I$$(exec_prefix)/$$(RTEMS_BSP)/lib/include
> +  CPU_DEFINES = -isystem $$(exec_prefix)/$$(RTEMS_BSP)/lib/include
>    CPU_CFLAGS = ${ABI_FLAGS}
>    CFLAGS_OPTIMIZE_V = ${OPTIMIZATION_FLAGS}
>    LDFLAGS = -B$$(exec_prefix)/$$(RTEMS_BSP)/lib ${PKGCONFIG_LDFLAGS}
> --
> 2.37.1
>
> _______________________________________________
> devel mailing list
> devel@rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to