diff options
| author | Ulf Magnusson <ulfalizer@gmail.com> | 2018-08-23 10:45:42 +0200 |
|---|---|---|
| committer | Ulf Magnusson <ulfalizer@gmail.com> | 2018-08-23 11:42:49 +0200 |
| commit | 3aca83eaf20f5e278b6d260cb255e6a9385590db (patch) | |
| tree | 895b9683a32f5e90be4c38c3bf1a4f0cf512fafd /kconfiglib.py | |
| parent | 9b7751103e7985c9184b41d89d8454249ed36875 (diff) | |
Fix recursive 'source' error reporting
The recursive 'source' detection was still fine, but the error reporting
had broken due a missed variable renaming.
The test suite didn't catch it, because a different type of KconfigError
was raised instead, due to a separate error in the test suite (need to
include tests/Krecursive{1,2}, since paths are relative to $srctree).
Fix the variable name and tighten up the tests to check that the
KconfigError message is the one we except. Tighten up the dependency
loop detection tests in the same way too.
Diffstat (limited to 'kconfiglib.py')
| -rw-r--r-- | kconfiglib.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kconfiglib.py b/kconfiglib.py index 3b4e8d7..a16a215 100644 --- a/kconfiglib.py +++ b/kconfiglib.py @@ -1551,7 +1551,7 @@ class Kconfig(object): "\n{}:{}: Recursive 'source' of '{}' detected. Check that " "environment variables are set correctly.\n" "Backtrace:\n{}" - .format(self._filename, self._linenr, filename, + .format(self._filename, self._linenr, rel_filename, "\n".join("{}:{}".format(name, linenr) for name, linenr, _ in reversed(self._filestack)))) |
