Francois Gouget : testbot: Add some section headers in PatchUtils.

[Alexandre Julliard]

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

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Fri Aug 10 20:09:19 2018 +0200

testbot: Add some section headers in PatchUtils.

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

---

 testbot/lib/WineTestBot/PatchUtils.pm | 74 +++++++++++++++++++----------------
 1 file changed, 41 insertions(+), 33 deletions(-)

diff --git a/testbot/lib/WineTestBot/PatchUtils.pm b/testbot/lib/WineTestBot/PatchUtils.pm
index be5ce14..d399ced 100644
--- a/testbot/lib/WineTestBot/PatchUtils.pm
+++ b/testbot/lib/WineTestBot/PatchUtils.pm
@@ -36,39 +36,9 @@ our @EXPORT = qw(GetPatchImpact UpdateWineData);
 use WineTestBot::Config;
 
 
-# These paths are too generic to be proof that this is a Wine patch.
-my $AmbiguousPathsRe = join('|',
-  'Makefile\.in$',
-  # aclocal.m4 gets special treatment
-  # configure gets special treatment
-  # configure.ac gets special treatment
-  'include/Makefile\.in$',
-  'include/config\.h\.in$',
-  'po/',
-  'tools/Makefile.in',
-  'tools/config.guess',
-  'tools/config.sub',
-  'tools/install-sh',
-  'tools/makedep.c',
-);
-
-# Patches to these paths don't impact the Wine build. So ignore them.
-my $IgnoredPathsRe = join('|',
-  '\.mailmap$',
-  'ANNOUNCE$',
-  'AUTHORS$',
-  'COPYING\.LIB$',
-  'LICENSE\$',
-  'LICENSE\.OLD$',
-  'MAINTAINERS$',
-  'README$',
-  'VERSION$',
-  'documentation/',
-  'tools/c2man\.pl$',
-  'tools/winapi/',
-  'tools/winemaker/',
-);
-
+#
+# Source repository maintenance
+#
 
 =pod
 =over 12
@@ -140,6 +110,44 @@ sub _LoadWineFiles()
   }
 }
 
+
+#
+# Wine patch analysis
+#
+
+# These paths are too generic to be proof that this is a Wine patch.
+my $AmbiguousPathsRe = join('|',
+  'Makefile\.in$',
+  # aclocal.m4 gets special treatment
+  # configure gets special treatment
+  # configure.ac gets special treatment
+  'include/Makefile\.in$',
+  'include/config\.h\.in$',
+  'po/',
+  'tools/Makefile.in',
+  'tools/config.guess',
+  'tools/config.sub',
+  'tools/install-sh',
+  'tools/makedep.c',
+);
+
+# Patches to these paths don't impact the Wine build. So ignore them.
+my $IgnoredPathsRe = join('|',
+  '\.mailmap$',
+  'ANNOUNCE$',
+  'AUTHORS$',
+  'COPYING\.LIB$',
+  'LICENSE\$',
+  'LICENSE\.OLD$',
+  'MAINTAINERS$',
+  'README$',
+  'VERSION$',
+  'documentation/',
+  'tools/c2man\.pl$',
+  'tools/winapi/',
+  'tools/winemaker/',
+);
+
 sub _HandleFile($$$)
 {
   my ($Impacts, $FilePath, $Change) = @_;