[Piglit] [PATCH 00/24] Change Test to use only a list for 1st argument
Brian Paul
brianp at vmware.com
Mon Jan 5 15:21:10 PST 2015
On 01/05/2015 02:50 PM, Dylan Baker wrote:
> Currently Test takes a command argument, which can either be a list of
> strings or a string. String by itself causes problems on cygwin (as
> reported by Brian Paul), so it's worth fixing that. This also simplifies
> the signature by not allowing Test to take string arguments.
>
> Brian sent patches the fix the problem, but the it's still possible to
> use either one. This series removes all uses of string arguments and
> replaces them with lists.
>
> I also took advantage of this test to make use of os.path and grouptools
> for working with paths and groups instead of using string formatting and
> concatenation for the same work. This accounts for most of the extra
> LOC, since in many cases this is more verbose and made single lines so
> long they were unreadable.
>
> I have verified that all.py, quick.py, sanity.py, and xts.py still work.
> I have touch tested igt, and cl.py, but I haven't run them to ensure
> that they run to completion.
>
> This series assumes my series from last week that fixes bugs in
> grouptools and xts.
>
> This series is available at my github:
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dcbaker_piglit&d=AwIGaQ&c=Sqcl0Ez6M0X8aeM67LKIiDJAXVeAw-YihVMNtXt-uEs&r=T0t4QG7chq2ZwJo6wilkFznRSFy-8uDKartPGbomVj8&m=xSdCiH8Uaj4V9OH4Lr0OayoTUaSS7FeW_ZRZnd_ztXY&s=Cj4geWx04I--Lnt_bfUOugAyd2HGI80bL54ooPOJcK4&e= submit/command-list-only
>
> Dylan Baker (24):
> framework: fix bug in Test class
> all.py: remove unnecessary ; on line ends
> all.py: replace tabs with four spaces
> all.py: change add_plain and add_concurrent to use list
> all.py: change add_single_param_test_set() to use a list for args
> all.py: change add_fbo_depthstencil_tests.py to use list for args
> all.py: change add_depthstencil_render_miplevel_tests to use list for
> args
> all.py: change add_msaa_visual_plain_tests to use list for args
> all.py: change add_fbo_formats_tests to use list for args
> all.py: change add_msaa_formats_tests to use list for args
> all.py: change add_fbo* to use list for args
> all.py: change add_getactiveuniform_count to use list for args
> all.py: change texwrap_target to return list
> all.py: change add_fbo_* to use a list for args
> all.py: change add_color_buffer_float_test to use list for args
> all.py: Fix string arguments that are hard to fix with a script
> no upstream: add tool to fix all.py
> all.py: replace simple string arguments with lists
> all.py: use os.path.join for all path joining
> all.py: Use grouptools.join for joining group strings
> all.py: Replace some string concatenation with str.format()
> sanity.py: Make Test() command argument a list instead of string
> sanity.py: use grouptools.join instead of hardcoded '/' for groups
> Test: don't accept strings for command arguments.
>
> fix_all.py | 79 +
> framework/test/base.py | 14 +-
> framework/tests/base_tests.py | 38 +-
> framework/tests/gtest_tests.py | 2 +-
> framework/tests/piglit_test_tests.py | 22 +-
> tests/all.py | 3996 +++++++++++++++++-----------------
> tests/sanity.py | 5 +-
> 7 files changed, 2171 insertions(+), 1985 deletions(-)
> create mode 100755 fix_all.py
>
Thanks for doing this, Dylan. It should help prevent problems on cygwin
going forward.
I didn't study all the patches in detail, but they look good to me. I
don't have time to test right now either, but if there's any cygwin
breakage I'm sure it can be fixed up.
Acked-by: Brian Paul <brianp at vmware.com>
More information about the Piglit
mailing list