https://bugs.winehq.org/show_bug.cgi?id=45025 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #1 from François Gouget <fgouget(a)codeweavers.com> --- This is done. This is the result of a long patch series that reworked the Build.pl script to rely more on the Wine makefiles, and on the move of the patch parser to the new PatchUtils.pm module so that Build.pl can know which tests to rebuild. The list below has the most important patches (some may be missing). commit 6fbd57b4f50c5a39e394dfbf0f97662e8a15a226 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Jun 13 10:23:42 2018 +0200 testbot: Create a single Job per patch even for multiple modules. So far a build task could only provide the 32 and 64 bit test executables for a single dll or program. Also, because of the lack of proper dependency support between steps it was impossible to have multiple build steps. So the TestBot used to create one job per module. However the patch status site can only keep track of one job per patch. This means some failures could be missed if they did not happen in the one job tracked by the patch status site. Now that Build.pl rebuilds all the impacted test executables the Testbot systematically creates a single job and a single build task and retrieves all the test executables in one go. This also has the advantage of reducing the number of VM reverts. Another side-effect of this change is that if the patch modifies the tests of multiple modules and the compilation of one them fails, no test will be run. Before only the tests impacted by the failed compilation would have been skipped. However a compilation error is grounds for resubmitting the patch, at which time the tests will be rerun anyway. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit fb84879ce70a0ad379bc779a5c9c1c1483443874 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Jun 13 10:23:28 2018 +0200 testbot: Remove the unneeded Build.pl parameters. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit 15ab1a7995221e0b93bdf9c3fc4dccb8bf461109 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Tue Jun 12 11:52:13 2018 +0200 testbot/build: Always rebuild all the impacted test executables. This means Build.pl no longer needs the PatchType and BaseName parameters. But still keep support for them for compatibility with WineRunBuild.pl. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit 6521df329b7b38d55f3d84ad7783097ee97ef382 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Thu May 31 10:06:10 2018 +0200 testbot: Fix handling of test unit additions/removals in patch series. The TestBot used to only analyse the last patch in the series to determine whether it impacts the tests or not. However patches to resource files and helper dlls require having an up-to-date list of the test units, and that list may be impacted by previous patches in the series. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit f9501a1f91a5e2a5da2649d1e74ca678c950d375 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed May 30 08:05:35 2018 +0200 testbot/build: Fix handling of directory addition / removal. When a directory is added or removed we must re-run make_makefiles. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit aa2a4355b05cdcbbc5f0a1184859f0eeaa6c83c1 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed May 30 08:05:15 2018 +0200 testbot/build: Always rebuild the native tools. It does not take long and it is actually hard to really know when to rebuild them if taking into account their libs/ and include/ dependencies. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit 4bf681b11dfafa2dd7f9f769e957d0fa5b6b43ec Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Tue May 29 00:42:49 2018 +0200 testbot: Fix handling of test unit deletion + helper dll patch. Take into account deleted test units when getting the full test unit list. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit 6641ea6dea6368169786b3018525abb9f23775fa Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Tue May 29 00:42:30 2018 +0200 testbot/build: Simplify Build.pl by reusing the patch analyser. Also better identify patches that require rebuilding the native tools. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit 603027bebc8d118f767e5cf83eb17246258bf9e9 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon May 28 15:09:15 2018 +0200 testbot/build: Rely on Wine's makefiles and simplify Build.pl. Assume the dependencies in Wine's makefiles work and let them decide whether to re-run configure, update include files or rebuild some dlls before the test executable, etc. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> commit 569b5f5b47fdfc0d37f9e8139d3ad4e31be60412 Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon May 28 15:06:14 2018 +0200 testbot/build: Add support for test helper dlls and make the patch parser reusable. Don't mistake the helper dlls C source files for test units. Also correctly handle patches that add or remove (i.e. rename) a resource file or helper dll. And move the patch parser to its own module so it can be reused, including by processes that don't use / have access to the database. Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.