Re: Adding binutils to the GNU Toolchain buildbot on sourceware

[Mark Wielaard <mark@klomp.org>]

[-- Attachment #1: Type: text/plain, Size: 1393 bytes --]

Hi Tom,

On Thu, Apr 28, 2022 at 10:47:52AM -0400, Thomas Fitzsimmons wrote:
> Mark Wielaard <mark@klomp.org> writes:
> > On Thu, Apr 28, 2022 at 01:23:44PM +0100, Luis Machado wrote:
> >> It would make sense to build-test gdb if any of the following changes:
> >
> > Thanks, updated the gdb_files list.  Also added fedora-x86_64,
> > debian-armhf, debian-arm64 and fedora-s390 builders.  See attached.
> >
> > Tom, would it be OK to add a debian-ppc64 builder for gdb? As you can
> > see in the master.cfg it only does a build of gdb and gdbserver for
> > now, no tests are run. So it just would make sure that things
> > build. We'll might add tests later, if we can figure out a sane subset
> > that should always be green (and won't take more than a couple of
> > minutes to run). BTW. I am also working on adding more builders for
> > binutils (gas, ld, binutils) but the current testsuite doesn't seem
> > clean on anything than latest fedora-x86_64.
> 
> OK, let's try it.  The debian-ppc64 builder has 8 GiB of free disk
> space.

OK, gdb-debian-ppc64 added (see attached).

BTW. You should be able to add builders to your own worker by
commiting the master.cfg changes to the builder.git repo.

You might want to experiment with an binutils builder.  But I am still
experimenting to see which part of the testsuite actually passes on
the different distro/architectures.

Cheers,

Mark

[-- Attachment #2: 0001-Add-gdb-debian-ppc64-builder.patch --]
[-- Type: text/plain, Size: 1949 bytes --]

From c3d73b2d46b2b9e4f4e2a49bca9e4b6841c08abb Mon Sep 17 00:00:00 2001
From: Mark Wielaard <mark@klomp.org>
Date: Thu, 28 Apr 2022 18:24:09 +0200
Subject: [PATCH] Add gdb-debian-ppc64 builder

---
 builder/master.cfg | 9 ++++++++-
 htdocs/index.html  | 2 ++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/builder/master.cfg b/builder/master.cfg
index 3bf1e2a..ca123b5 100644
--- a/builder/master.cfg
+++ b/builder/master.cfg
@@ -413,7 +413,8 @@ gdb_scheduler = schedulers.SingleBranchScheduler(
                       "gdb-fedora-x86_64",
                       "gdb-debian-armhf",
                       "gdb-debian-arm64",
-                      "gdb-fedora-s390x"])
+                      "gdb-fedora-s390x",
+                      "gdb-debian-ppc64"])
 c['schedulers'].append(gdb_scheduler)
 
 # A scheduler for everything binutils-gdb without filters
@@ -1233,6 +1234,12 @@ gdb_fedora_s390x_builder = util.BuilderConfig(
         factory=gdb_factory)
 c['builders'].append(gdb_fedora_s390x_builder)
 
+gdb_debian_ppc64_builder = util.BuilderConfig(
+	name="gdb-debian-ppc64",
+        workernames=["debian-ppc64"],
+        tags=["gdb", "debian", "ppc64"],
+        factory=gdb_factory)
+c['builders'].append(gdb_debian_ppc64_builder)
 
 # binutils-gdb build steps, factory and builders
 # just a native build
diff --git a/htdocs/index.html b/htdocs/index.html
index b98b00a..9cf76b7 100644
--- a/htdocs/index.html
+++ b/htdocs/index.html
@@ -158,6 +158,8 @@
               <img src="/buildbot/badges/gdb-debian-arm64.svg"></a></td>
 	  <td><a href="/buildbot/#builders/gdb-fedora-s390x">fedora-s390x<br>
               <img src="/buildbot/badges/gdb-fedora-s390x.svg"></a></td>
+	  <td><a href="/buildbot/#builders/gdb-debian-ppc64">debian-ppc64<br>
+              <img src="/buildbot/badges/gdb-debian-ppc64.svg"></a></td>
 	</tr>
 	<tr>
 	  <td><a href="/buildbot/#/builders?tags=libabigail">libabigail</a></td>
-- 
2.27.0