Message ID | 20230524031601.19938-1-bagasdotme@gmail.com |
---|---|
State | Accepted |
Headers | show |
Series | toolchain-external-custom: Describe BR2_TOOLCHAIN_EXTERNAL_CUSTOM_PREFIX | expand |
On Wed, 24 May 2023 10:16:01 +0700 Bagas Sanjaya <bagasdotme@gmail.com> wrote: > "Toolchain prefix" option apparently doesn't have any help describing > it, which causes confusion when using external toolchain. Leaving this > option at default prefix name ("$(ARCH)-linux") when external toolchain > components are called with different prefix (e.g. > "$(ARCH)-unknown-linux-gnu") may cause build failure unless the prefix > symlink is already in place (e.g. when using Buildroot-generated > toolchain as external toolchain). > > Describe the option to clarify. > > Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com> Reviewed-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> Thanks a lot! Thomas
Bags, All, On 2023-05-24 10:16 +0700, Bagas Sanjaya spake thusly: > "Toolchain prefix" option apparently doesn't have any help describing > it, which causes confusion when using external toolchain. Leaving this > option at default prefix name ("$(ARCH)-linux") when external toolchain > components are called with different prefix (e.g. > "$(ARCH)-unknown-linux-gnu") may cause build failure unless the prefix > symlink is already in place (e.g. when using Buildroot-generated > toolchain as external toolchain). > > Describe the option to clarify. > > Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com> > --- > .../toolchain-external-custom/Config.in.options | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options > index 0fd8841e5b..ed62bad990 100644 > --- a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options > +++ b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options > @@ -24,6 +24,17 @@ config BR2_TOOLCHAIN_EXTERNAL_REL_BIN_PATH > config BR2_TOOLCHAIN_EXTERNAL_CUSTOM_PREFIX > string "Toolchain prefix" > default "$(ARCH)-linux" > + help > + Component name prefix used by your external toolchain. > + Typically, this is target tuple (e.g. "$(ARCH)-unknown-linux-gnu"). > + If unsure, look at "bin" directory of your toolchain and > + note the part of component name before the last dash (i.e. if > + you call the toolchain's gcc as "$(ARCH)-unknown-linux-gnu-gcc", > + the prefix is "$(ARCH)-unknown-linux-gnu"). > + > + Note that Buildroot toolchains have "$(ARCH)-linux" symlink alias > + to the full tuple-named components, so you can leave this option at > + default value if you use them as external toolchain. $ ./utils/docker-run make check-package toolchain/toolchain-external/toolchain-external-custom/Config.in.options:29: help text: <tab><2 spaces><62 chars> (http://nightly.buildroot.org/#writing-rules-config-in) toolchain/toolchain-external/toolchain-external-custom/Config.in.options:32: help text: <tab><2 spaces><62 chars> (http://nightly.buildroot.org/#writing-rules-config-in) toolchain/toolchain-external/toolchain-external-custom/Config.in.options:35: help text: <tab><2 spaces><62 chars> (http://nightly.buildroot.org/#writing-rules-config-in) toolchain/toolchain-external/toolchain-external-custom/Config.in.options:36: help text: <tab><2 spaces><62 chars> (http://nightly.buildroot.org/#writing-rules-config-in) Applied to master with the above fixed (which required that I rephrase it slightly to keep lines of a consistent length). Thanks. Regards, Yann E. MORIN. > choice > bool "External toolchain gcc version" > > base-commit: 603df32a0cd416caa2d2688008cb87f14ad68ef3 > -- > An old man doll... just what I always wanted! - Clara > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot
>>>>> "Bagas" == Bagas Sanjaya <bagasdotme@gmail.com> writes: > "Toolchain prefix" option apparently doesn't have any help describing > it, which causes confusion when using external toolchain. Leaving this > option at default prefix name ("$(ARCH)-linux") when external toolchain > components are called with different prefix (e.g. > "$(ARCH)-unknown-linux-gnu") may cause build failure unless the prefix > symlink is already in place (e.g. when using Buildroot-generated > toolchain as external toolchain). > Describe the option to clarify. > Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com> Committed to 2023.02.x, thanks.
diff --git a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options index 0fd8841e5b..ed62bad990 100644 --- a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options +++ b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options @@ -24,6 +24,17 @@ config BR2_TOOLCHAIN_EXTERNAL_REL_BIN_PATH config BR2_TOOLCHAIN_EXTERNAL_CUSTOM_PREFIX string "Toolchain prefix" default "$(ARCH)-linux" + help + Component name prefix used by your external toolchain. + Typically, this is target tuple (e.g. "$(ARCH)-unknown-linux-gnu"). + If unsure, look at "bin" directory of your toolchain and + note the part of component name before the last dash (i.e. if + you call the toolchain's gcc as "$(ARCH)-unknown-linux-gnu-gcc", + the prefix is "$(ARCH)-unknown-linux-gnu"). + + Note that Buildroot toolchains have "$(ARCH)-linux" symlink alias + to the full tuple-named components, so you can leave this option at + default value if you use them as external toolchain. choice bool "External toolchain gcc version"
"Toolchain prefix" option apparently doesn't have any help describing it, which causes confusion when using external toolchain. Leaving this option at default prefix name ("$(ARCH)-linux") when external toolchain components are called with different prefix (e.g. "$(ARCH)-unknown-linux-gnu") may cause build failure unless the prefix symlink is already in place (e.g. when using Buildroot-generated toolchain as external toolchain). Describe the option to clarify. Signed-off-by: Bagas Sanjaya <bagasdotme@gmail.com> --- .../toolchain-external-custom/Config.in.options | 11 +++++++++++ 1 file changed, 11 insertions(+) base-commit: 603df32a0cd416caa2d2688008cb87f14ad68ef3