Francois Gouget : testbot/Submit: Allow submitting patches that touch zero or multiple test units.

Alexandre Julliard julliard at winehq.org
Wed Oct 2 16:49:22 CDT 2019


Module: tools
Branch: master
Commit: cee9428e9c32135ce491b038ee552da999819871
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=cee9428e9c32135ce491b038ee552da999819871

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Wed Oct  2 06:56:19 2019 +0200

testbot/Submit: Allow submitting patches that touch zero or multiple test units.

Patches submitted through the submit page no longer need to impact one
and only one test unit. This means it's no longer necessary to split
longer patches in order to test them.
With the Wine VMs it also no longer makes sense to ban patches that
don't touch the tests as one may need to make sure they compile and
don't cause regressions in existing tests.
However it is still necessary to specify command line arguments for
the test executable which means picking a single test unit to run.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/bin/WineRunWineTest.pl        |   1 +
 testbot/bin/build/Build.pl            |   6 ++
 testbot/bin/build/WineTest.pl         |  79 +++++++++++++-----
 testbot/lib/Build/Utils.pm            |  43 ++++++----
 testbot/lib/WineTestBot/PatchUtils.pm |   1 +
 testbot/web/Submit.pl                 | 153 +++++++++++++++++++++++-----------
 6 files changed, 199 insertions(+), 84 deletions(-)

Diff:   https://source.winehq.org/git/tools.git/?a=commitdiff;h=cee9428e9c32135ce491b038ee552da999819871



More information about the wine-cvs mailing list