[Buildroot] [PATCH 15/36] check-package: enable for toolchain/

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sat Mar 31 06:38:21 UTC 2018


Hello,

On Fri, 30 Mar 2018 23:22:40 -0300, Ricardo Martincoski wrote:
> The toolchain directory can benefit from this script to prevent common
> mistakes when submitting patches.
> 
> In order to accomplish this:
> Do not ignore anymore files from the toolchain/ directory.
> Accept toolchain-common.in as a valid Config.in name.
> Ignore this symbol:
>  - BR_LIBC: defined by the buildroot toolchain, used by gcc-final.mk.
> But do not ignore this symbol:
>  - GCONV_LIBS: it is used only inside the file that defines it.

You're not actually ignoring this variable in this patch, and you
instead fixed it in PATCH 14/36 to have a TOOLCHAIN_ prefix.

> Ignore toolchain/toolchain-external/pkg-toolchain-external.mk as it
> declares a package infra and not a package itself.
> Ignore toolchain/helpers.mk as it contains only helper functions.
> 
> Signed-off-by: Ricardo Martincoski <ricardo.martincoski at gmail.com>
> Cc: Romain Naour <romain.naour at gmail.com>
> ---
>  utils/check-package             | 5 ++++-
>  utils/checkpackagelib/lib_mk.py | 1 +
>  2 files changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/utils/check-package b/utils/check-package
> index a5f5dc44c0..35a4a70858 100755
> --- a/utils/check-package
> +++ b/utils/check-package
> @@ -44,7 +44,7 @@ def parse_args():
>      return parser.parse_args()
>  
>  
> -CONFIG_IN_FILENAME = re.compile("/Config\.\S*$")
> +CONFIG_IN_FILENAME = re.compile("/Config\.\S*$|/toolchain-common.in$")

Not directly related to your series, but I believe this
toolchain-common.in file is silly. It is just included by
toolchain/Config.in, and toolchain/Config.in is not that long. We
should bring back the toolchain-common.in contents in
toolchain/Config.in, and stop having this odd toolchain-common.in.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com



More information about the buildroot mailing list