docs: Makefile: use the new script to check for bad ABI references

[PATCH] docs: Makefile: use the new script to check for bad ABI references

Posted by Mauro Carvalho Chehab 10 months, 1 week ago

The get_abi.pl script was replaced by get_abi.py. Update it at docs
makefile.

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202502110736.ZGWaWsep-lkp@intel.com/
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
---
 Documentation/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/Makefile b/Documentation/Makefile
index 52c6c5a3efa9..63094646df28 100644
--- a/Documentation/Makefile
+++ b/Documentation/Makefile
@@ -12,7 +12,7 @@ endif
 
 # Check for broken ABI files
 ifeq ($(CONFIG_WARN_ABI_ERRORS),y)
-$(shell $(srctree)/scripts/get_abi.pl validate --dir $(srctree)/Documentation/ABI)
+$(shell $(srctree)/scripts/get_abi.py --dir $(srctree)/Documentation/ABI validate)
 endif
 
 # You can set these variables from the command line.
-- 
2.48.1

Re: [PATCH] docs: Makefile: use the new script to check for bad ABI references

Posted by Jonathan Corbet 10 months ago

Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:

> The get_abi.pl script was replaced by get_abi.py. Update it at docs
> makefile.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202502110736.ZGWaWsep-lkp@intel.com/
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> ---
>  Documentation/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index 52c6c5a3efa9..63094646df28 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -12,7 +12,7 @@ endif
>  
>  # Check for broken ABI files
>  ifeq ($(CONFIG_WARN_ABI_ERRORS),y)
> -$(shell $(srctree)/scripts/get_abi.pl validate --dir $(srctree)/Documentation/ABI)
> +$(shell $(srctree)/scripts/get_abi.py --dir $(srctree)/Documentation/ABI validate)
>  endif

Applied, thanks.

jon

Re: [PATCH] docs: Makefile: use the new script to check for bad ABI references

Posted by Bagas Sanjaya 10 months, 1 week ago

On Tue, Feb 11, 2025 at 06:44:06AM +0100, Mauro Carvalho Chehab wrote:
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index 52c6c5a3efa9..63094646df28 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -12,7 +12,7 @@ endif
>  
>  # Check for broken ABI files
>  ifeq ($(CONFIG_WARN_ABI_ERRORS),y)
> -$(shell $(srctree)/scripts/get_abi.pl validate --dir $(srctree)/Documentation/ABI)
> +$(shell $(srctree)/scripts/get_abi.py --dir $(srctree)/Documentation/ABI validate)
>  endif
>  
>  # You can set these variables from the command line.

The warning squashed away, thanks!

Tested-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara