[PATCH] kconfig: tests: fix recursive inclusion unit test

[PATCH] kconfig: tests: fix recursive inclusion unit test

[Masahiro Yamada]

Adding SPDX license identifier is pretty safe; however, here is one
exception.

Since commit ec8f24b7faaf ("treewide: Add SPDX license identifier -
Makefile/Kconfig"), "make testconfig" would not pass.

When Kconfig detects a circular file inclusion, it displays error
messages with a file name and a line number prefixed to each line.

The unit test checks if Kconfig emits the error messages correctly
(this also checks the line number correctness).

Now that the test input has the SPDX license identifier at the very top,
the line numbers in the expected stderr should be incremented by 1.

Fixes: ec8f24b7faaf ("treewide: Add SPDX license identifier - Makefile/Kconfig")
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 scripts/kconfig/tests/err_recursive_inc/expected_stderr | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/kconfig/tests/err_recursive_inc/expected_stderr b/scripts/kconfig/tests/err_recursive_inc/expected_stderr
index 6b582eee2176..b070a31fdfeb 100644
--- a/scripts/kconfig/tests/err_recursive_inc/expected_stderr
+++ b/scripts/kconfig/tests/err_recursive_inc/expected_stderr
@@ -1,6 +1,6 @@
 Recursive inclusion detected.
 Inclusion path:
   current file : Kconfig.inc1
-  included from: Kconfig.inc3:1
-  included from: Kconfig.inc2:3
-  included from: Kconfig.inc1:4
+  included from: Kconfig.inc3:2
+  included from: Kconfig.inc2:4
+  included from: Kconfig.inc1:5
-- 
2.17.1

Re: [PATCH] kconfig: tests: fix recursive inclusion unit test

[Masahiro Yamada]

On Sat, May 25, 2019 at 8:49 PM Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
>
> Adding SPDX license identifier is pretty safe; however, here is one
> exception.
>
> Since commit ec8f24b7faaf ("treewide: Add SPDX license identifier -
> Makefile/Kconfig"), "make testconfig" would not pass.
>
> When Kconfig detects a circular file inclusion, it displays error
> messages with a file name and a line number prefixed to each line.
>
> The unit test checks if Kconfig emits the error messages correctly
> (this also checks the line number correctness).
>
> Now that the test input has the SPDX license identifier at the very top,
> the line numbers in the expected stderr should be incremented by 1.
>
> Fixes: ec8f24b7faaf ("treewide: Add SPDX license identifier - Makefile/Kconfig")
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---

Applied to linux-kbuild/fixes.


>
>  scripts/kconfig/tests/err_recursive_inc/expected_stderr | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/scripts/kconfig/tests/err_recursive_inc/expected_stderr b/scripts/kconfig/tests/err_recursive_inc/expected_stderr
> index 6b582eee2176..b070a31fdfeb 100644
> --- a/scripts/kconfig/tests/err_recursive_inc/expected_stderr
> +++ b/scripts/kconfig/tests/err_recursive_inc/expected_stderr
> @@ -1,6 +1,6 @@
>  Recursive inclusion detected.
>  Inclusion path:
>    current file : Kconfig.inc1
> -  included from: Kconfig.inc3:1
> -  included from: Kconfig.inc2:3
> -  included from: Kconfig.inc1:4
> +  included from: Kconfig.inc3:2
> +  included from: Kconfig.inc2:4
> +  included from: Kconfig.inc1:5
> --
> 2.17.1
>


-- 
Best Regards
Masahiro Yamada