From 4516e8a49ffff2c8781cce2de9680ed0ce2aab7d Mon Sep 17 00:00:00 2001
From: Nirbheek Chauhan <nirbheek@centricular.com>
Date: Fri, 1 Jul 2016 14:43:51 +0530
Subject: [PATCH] Add repr() implementations for build targets and File

This aids debugging
---
 mesonbuild/build.py    | 25 +++++++++++++++++++++++++
 mesonbuild/mesonlib.py | 10 ++++++++++
 2 files changed, 35 insertions(+)

diff --git a/mesonbuild/build.py b/mesonbuild/build.py
index 3ca7bf2f3..d33f69219 100644
--- a/mesonbuild/build.py
+++ b/mesonbuild/build.py
@@ -239,6 +239,10 @@ class BuildTarget():
             raise InvalidArguments('Build target %s has no sources.' % name)
         self.validate_sources()
 
+    def __repr__(self):
+        repr_str = "<{0} {1}: {2}>"
+        return repr_str.format(self.__class__.__name__, self.get_id(), self.filename)
+
     def get_id(self):
         # This ID must also be a valid file name on all OSs.
         # It should also avoid shell metacharacters for obvious
@@ -623,6 +627,10 @@ class Generator():
         self.exe = exe
         self.process_kwargs(kwargs)
 
+    def __repr__(self):
+        repr_str = "<{0}: {1}>"
+        return repr_str.format(self.__class__.__name__, self.exe)
+
     def get_exe(self):
         return self.exe
 
@@ -953,6 +961,10 @@ class CustomTarget:
             mlog.log(mlog.bold('Warning:'), 'Unknown keyword arguments in target %s: %s' %
                      (self.name, ', '.join(unknowns)))
 
+    def __repr__(self):
+        repr_str = "<{0} {1}: {2}>"
+        return repr_str.format(self.__class__.__name__, self.get_id(), self.command)
+
     def get_id(self):
         return self.name + self.type_suffix()
 
@@ -1079,6 +1091,10 @@ class RunTarget:
         self.args = args
         self.subdir = subdir
 
+    def __repr__(self):
+        repr_str = "<{0} {1}: {2}>"
+        return repr_str.format(self.__class__.__name__, self.get_id(), self.command)
+
     def get_id(self):
         return self.name + self.type_suffix()
 
@@ -1129,6 +1145,12 @@ class ConfigureFile():
         self.targetname = targetname
         self.configuration_data = configuration_data
 
+    def __repr__(self):
+        repr_str = "<{0}: {1} -> {2}>"
+        src = os.path.join(self.subdir, self.sourcename)
+        dst = os.path.join(self.subdir, self.targetname)
+        return repr_str.format(self.__class__.__name__, src, dst)
+
     def get_configuration_data(self):
         return self.configuration_data
 
@@ -1146,6 +1168,9 @@ class ConfigurationData():
         super().__init__()
         self.values = {}
 
+    def __repr__(self):
+        return repr(self.values)
+
     def get(self, name):
         return self.values[name]
 
diff --git a/mesonbuild/mesonlib.py b/mesonbuild/mesonlib.py
index 837f78ac3..1672d95de 100644
--- a/mesonbuild/mesonlib.py
+++ b/mesonbuild/mesonlib.py
@@ -28,6 +28,16 @@ class File:
         self.subdir = subdir
         self.fname = fname
 
+    def __str__(self):
+        return os.path.join(self.subdir, self.fname)
+
+    def __repr__(self):
+        ret = '<File: {0}'
+        if not self.is_built:
+            ret += ' (not built)'
+        ret += '>'
+        return ret.format(os.path.join(self.subdir, self.fname))
+
     @staticmethod
     def from_source_file(source_root, subdir, fname):
         if not os.path.isfile(os.path.join(source_root, subdir, fname)):