The reftest mach command redefines all of the arguments  exposed by the reftest harness . This is bad for a few reasons: * options need to be defined twice * makes command line for running locally different from command line when running in production * encourages multiple entry points into the test harness and separation of features (i.e features that only work when using the mach command). Instead of re-defining arguments, the mach command should just *use* the harness' argument parser directly. This is possible by using the 'parser' argument to the @Command decorator in mach. This was already done for mochitest, so bug 1155338 can be used as a rough guideline. A pre-requisite to this is getting the reftest harness to use argparse instead of optparse.  https://dxr.mozilla.org/mozilla-central/source/layout/tools/reftest/mach_commands.py#317  https://dxr.mozilla.org/mozilla-central/source/layout/tools/reftest/runreftest.py#655
You need to log in before you can comment on or make changes to this bug.