On 07/07/2014 04:20 PM, Dylan Baker wrote:
With this patch piglit.conf can live in a number of different places. It
can live in the current directory, then in XDG_CONFIG_HOME
($HOME/.config by default), and finally in the piglit source directory.

v2: - fix types (chadv)
     - change load order to make ./.piglit.conf load first (chadv)
     - Add framework tests

Signed-off-by: Dylan Baker <[email protected]>
---
  framework/programs/run.py         |  39 +++++++--
  framework/tests/programs_tests.py | 180 ++++++++++++++++++++++++++++++++++++++
  2 files changed, 212 insertions(+), 7 deletions(-)
  create mode 100644 framework/tests/programs_tests.py




+        # Try XDG_CONFIG_DIR, then try the local directory, finally try the
+        # root of the piglit dir relative to this file
+        for d in ['.',
+                  os.environ.get('XDG_CONFIG_HOME',
+                                 os.path.expandvars('$HOME/.config')),
+                  os.path.join(os.path.dirname(__file__), '..', '..')]:


The code searches in the correct order, but the comment describes a
different order. Fix the comment and
Reviewed-by: Chad Versace <[email protected]>

_______________________________________________
Piglit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to