[webkit-dev] Commits in branches triggering build in the bots now

Carlos Alberto Lopez Perez clopez at igalia.com
Thu May 4 15:55:58 PDT 2017

On 05/05/17 00:24, Lucas Forschler wrote:
> Hi Carlos!
> I recently updated the svn post commit hook to use the newest version of svn_buildbot.py from https://github.com/buildbot/buildbot-contrib/blob/master/master/contrib/svn_buildbot.py <https://github.com/buildbot/buildbot-contrib/blob/master/master/contrib/svn_buildbot.py>
> I am going to look through the diff now to see if there are any obvious changes which would cause this.
> Thanks for bringing it to our attention!
> Lucas

Its ok that the svn_buildbot script sends changes about any branches.

But the buildbot config should only trigger branches for changes on
trunk due to the change_filter": "trunk_filter" below:



Maybe the issue is that the new svn_buildbot scripts fails to identify the branch name
(for WebKitGTK+ we use a non-standard branch name: "releases") and sends the changes with
branch "None"?

Maybe the patch below fixes the issue?

--- a/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg
+++ b/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg
@@ -951,7 +951,7 @@ class PlatformSpecificScheduler(AnyBranchScheduler):
     def filter(self, change):
         return wkbuild.should_build(self.platform, change.files)
-trunk_filter = ChangeFilter(branch=["trunk", None])
+trunk_filter = ChangeFilter(branch="trunk")
 def loadBuilderConfig(c):
     # FIXME: These file handles are leaked.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 883 bytes
Desc: OpenPGP digital signature
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20170505/edb176f4/attachment.bin>

More information about the webkit-dev mailing list