[ptxdist] [PATCH] platforms: document precedence rules for TARGET_EXTRA_*FLAGS

[ptxdist] [PATCH] platforms: document precedence rules for TARGET_EXTRA_*FLAGS

[Roland Hieber]

scripts/wrapper/libwrapper.sh adds these global options before the rest
of its command line, so later options from the package's build system
will overwrite them. Document this fact in case anybody wonders why
these options cannot be used to overwrite the default passed by the
package.

Signed-off-by: Roland Hieber <rhi@pengutronix.de>
---
 platforms/toolchain_options.in | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/platforms/toolchain_options.in b/platforms/toolchain_options.in
index afae2a67cd11..0361bf6e7e8f 100644
--- a/platforms/toolchain_options.in
+++ b/platforms/toolchain_options.in
@@ -95,22 +95,38 @@ config TARGET_EXTRA_CPPFLAGS
 	help
 	  Extra options for the C preprocessor.
 
+	  Note: if conflicting preprocessor options are passed, the options
+	  specified by the package's build system take precedence over those
+	  specified here.
+
 config TARGET_EXTRA_CFLAGS
 	string
 	prompt "Extra CFLAGS (c)"
 	help
 	  Extra options for the C compiler.
 
+	  Note: if conflicting compiler options are passed, the options
+	  specified by the package's build system take precedence over those
+	  specified here.
+
 config TARGET_EXTRA_CXXFLAGS
 	string
 	prompt "Extra CXXFLAGS (c++)"
 	help
 	  Extra options for the C++ compiler.
 
+	  Note: if conflicting compiler options are passed, the options
+	  specified by the package's build system take precedence over those
+	  specified here.
+
 config TARGET_EXTRA_LDFLAGS
 	string
 	prompt "Extra LDFLAGS (ld)"
 	help
 	  Extra options for the linker.
 
+	  Note: if conflicting linker options are passed, the options
+	  specified by the package's build system take precedence over those
+	  specified here.
+
 endmenu
-- 
2.25.0


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Re: [ptxdist] platforms: document precedence rules for TARGET_EXTRA_*FLAGS

[Michael Olbrich]

Thanks, applied.

Michael

[sent from post-receive hook]

On Fri, 27 Mar 2020 10:52:09 +0100, rhi@pengutronix.de <rhi@pengutronix.de> wrote:
> scripts/wrapper/libwrapper.sh adds these global options before the rest
> of its command line, so later options from the package's build system
> will overwrite them. Document this fact in case anybody wonders why
> these options cannot be used to overwrite the default passed by the
> package.
> 
> Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> ---
>  platforms/toolchain_options.in | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/platforms/toolchain_options.in b/platforms/toolchain_options.in
> index afae2a67cd11..0361bf6e7e8f 100644
> --- a/platforms/toolchain_options.in
> +++ b/platforms/toolchain_options.in
> @@ -95,22 +95,38 @@ config TARGET_EXTRA_CPPFLAGS
>  	help
>  	  Extra options for the C preprocessor.
>  
> +	  Note: if conflicting preprocessor options are passed, the options
> +	  specified by the package's build system take precedence over those
> +	  specified here.
> +
>  config TARGET_EXTRA_CFLAGS
>  	string
>  	prompt "Extra CFLAGS (c)"
>  	help
>  	  Extra options for the C compiler.
>  
> +	  Note: if conflicting compiler options are passed, the options
> +	  specified by the package's build system take precedence over those
> +	  specified here.
> +
>  config TARGET_EXTRA_CXXFLAGS
>  	string
>  	prompt "Extra CXXFLAGS (c++)"
>  	help
>  	  Extra options for the C++ compiler.
>  
> +	  Note: if conflicting compiler options are passed, the options
> +	  specified by the package's build system take precedence over those
> +	  specified here.
> +
>  config TARGET_EXTRA_LDFLAGS
>  	string
>  	prompt "Extra LDFLAGS (ld)"
>  	help
>  	  Extra options for the linker.
>  
> +	  Note: if conflicting linker options are passed, the options
> +	  specified by the package's build system take precedence over those
> +	  specified here.
> +
>  endmenu
> 

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de