diff mbox series

minikconf: print error entirely on stderr

Message ID 20240906073328.492089-1-pbonzini@redhat.com
State New
Headers show
Series minikconf: print error entirely on stderr | expand

Commit Message

Paolo Bonzini Sept. 6, 2024, 7:33 a.m. UTC
While debugging an invalid configuration, I noticed that the clauses debug
ends up on stderr but the header ("The following clauses were found..."
ends up on stdout.  This makes the contents of meson-logs/meson-log.txt
a bit confusing.

Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 scripts/minikconf.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Philippe Mathieu-Daudé Sept. 6, 2024, 7:52 a.m. UTC | #1
On 6/9/24 09:33, Paolo Bonzini wrote:
> While debugging an invalid configuration, I noticed that the clauses debug
> ends up on stderr but the header ("The following clauses were found..."
> ends up on stdout.  This makes the contents of meson-logs/meson-log.txt
> a bit confusing.
> 
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   scripts/minikconf.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
diff mbox series

Patch

diff --git a/scripts/minikconf.py b/scripts/minikconf.py
index bcd91015d34..6f7f43b2918 100644
--- a/scripts/minikconf.py
+++ b/scripts/minikconf.py
@@ -112,7 +112,7 @@  def has_value(self):
         def set_value(self, val, clause):
             self.clauses_for_var.append(clause)
             if self.has_value() and self.value != val:
-                print("The following clauses were found for " + self.name)
+                print("The following clauses were found for " + self.name, file=sys.stderr)
                 for i in self.clauses_for_var:
                     print("    " + str(i), file=sys.stderr)
                 raise KconfigDataError('contradiction between clauses when setting %s' % self)