[bug#60540] teams: Add scope for Embedded / Bootstrap

[bug#60540] teams: Add scope for Embedded / Bootstrap

[Vagrant Cascadian]

[-- Attachment #1.1: Type: text/plain, Size: 206 bytes --]

There are some packages that are obviously related to embedded
(e.g. u-boot from bootloaders.scm, arm-trusted-firmware from
firmware.scm) and bootstrapping (mes.scm).

Patch attached.

live well,
  vagrant

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0003-teams-Add-scope-for-Embedded-Bootstrap-team.patch --]
[-- Type: text/x-diff, Size: 939 bytes --]

From 6c8076261cf1a93134e3435124d4a7e9dcbbba1c Mon Sep 17 00:00:00 2001
From: Vagrant Cascadian <vagrant@debian.org>
Date: Tue, 3 Jan 2023 21:41:34 -0800
Subject: [PATCH 3/4] teams: Add scope for Embedded / Bootstrap team.

* etc/teams.scm.in (embedded-bootstrap): Add bootloaders, firmware and mes to
  scope.
---
 etc/teams.scm.in | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/etc/teams.scm.in b/etc/teams.scm.in
index b3a1c72c28..921c101aa7 100644
--- a/etc/teams.scm.in
+++ b/etc/teams.scm.in
@@ -244,7 +244,10 @@ asdf-build-system."
 
 (define-team embedded-bootstrap
   (team 'embedded-bootstrap
-        #:name "Embedded / Bootstrap"))
+        #:name "Embedded / Bootstrap"
+        #:scope (list "gnu/packages/bootloaders.scm"
+                      "gnu/packages/firmware.scm"
+                      "gnu/packages/mes.scm")))
 
 (define-team rust
   (team 'rust
-- 
2.30.2


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#60540] teams: Add scope for Embedded / Bootstrap

[Mathieu Othacehe]

Hello Vagrant,

>  (define-team embedded-bootstrap
>    (team 'embedded-bootstrap
> -        #:name "Embedded / Bootstrap"))
> +        #:name "Embedded / Bootstrap"
> +        #:scope (list "gnu/packages/bootloaders.scm"
> +                      "gnu/packages/firmware.scm"
> +                      "gnu/packages/mes.scm")))

Seems fine.

I think we could have two distinct teams here: embedded and
bootstrap. There are quite a few people, like me, following closely the
embedded part, but not so closely the bootstrap part.

WDYT?

Thanks,

Mathieu

[bug#60540] teams: Add scope for Embedded / Bootstrap

[Vagrant Cascadian]

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

On 2023-01-08, Mathieu Othacehe wrote:
>>  (define-team embedded-bootstrap
>>    (team 'embedded-bootstrap
>> -        #:name "Embedded / Bootstrap"))
>> +        #:name "Embedded / Bootstrap"
>> +        #:scope (list "gnu/packages/bootloaders.scm"
>> +                      "gnu/packages/firmware.scm"
>> +                      "gnu/packages/mes.scm")))
>
> Seems fine.
>
> I think we could have two distinct teams here: embedded and
> bootstrap. There are quite a few people, like me, following closely the
> embedded part, but not so closely the bootstrap part.

Yeah, I think they should be split so started a thread on guix-devel:

  https://lists.gnu.org/archive/html/guix-devel/2023-01/msg00048.html

But I also wanted to start by keeping this patch simple. Although it
could really use a description too ...

live well,
  vagrant

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#60540] teams: Add scope for Embedded / Bootstrap

[Vagrant Cascadian]

[-- Attachment #1.1: Type: text/plain, Size: 836 bytes --]

On 2023-01-08, Mathieu Othacehe wrote:
>>  (define-team embedded-bootstrap
>>    (team 'embedded-bootstrap
>> -        #:name "Embedded / Bootstrap"))
>> +        #:name "Embedded / Bootstrap"
>> +        #:scope (list "gnu/packages/bootloaders.scm"
>> +                      "gnu/packages/firmware.scm"
>> +                      "gnu/packages/mes.scm")))
>
> Seems fine.
>
> I think we could have two distinct teams here: embedded and
> bootstrap. There are quite a few people, like me, following closely the
> embedded part, but not so closely the bootstrap part.

It makes total sense to me, so...

New patch attached which also splits the teams.

They could still probably significantly benefit from
descriptions... though I haven't heard much of anything weighed in on
that from the discussion on guix-devel.

live well,
  vagrant

[-- Attachment #1.2: 0001-teams-Split-Embedded-and-Bootstrap-into-separate-tea.patch --]
[-- Type: text/x-diff, Size: 1893 bytes --]

From 0ae1c6951bcfd3c1a83e8a86adcbc6afae122c01 Mon Sep 17 00:00:00 2001
From: Vagrant Cascadian <vagrant@debian.org>
Date: Wed, 18 Jan 2023 11:17:36 -0800
Subject: [PATCH] teams: Split Embedded and Bootstrap into separate teams.

* etc/teams.scm.in (embedded-bootstrap): Replace by two separate teams.
  (embedded): New variable.
  (bootstrap): New variable.
  Adjust membership to use new teams.
---
 etc/teams.scm.in | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/etc/teams.scm.in b/etc/teams.scm.in
index 77c0127bb2..8d21f63d9d 100644
--- a/etc/teams.scm.in
+++ b/etc/teams.scm.in
@@ -242,9 +242,16 @@ (define-team go
                       "guix/scripts/import/go.scm"
                       "tests/go.scm")))
 
-(define-team embedded-bootstrap
-  (team 'embedded-bootstrap
-        #:name "Embedded / Bootstrap"))
+(define-team bootstrap
+  (team 'bootstrap
+        #:name "Bootstrap"
+        #:scope (list "gnu/packages/mes.scm")))
+
+(define-team embedded
+  (team 'embedded
+        #:name "Embedded"
+        #:scope (list "gnu/packages/bootloaders.scm"
+                      "gnu/packages/firmware.scm")))
 
 (define-team rust
   (team 'rust
@@ -438,7 +445,7 @@ (define-member (person "Jonathan Brielmaier"
 
 (define-member (person "Ludovic Courtès"
                        "ludo@gnu.org")
-  core home embedded-bootstrap mentors)
+  core home embedded bootstrap mentors)
 
 (define-member (person "Andreas Enge"
                        "andreas@enge.fr")
@@ -458,7 +465,7 @@ (define-member (person "Leo Famulari"
 
 (define-member (person "Efraim Flashner"
                        "efraim@flashner.co.il")
-  embedded-bootstrap julia rust science)
+  embedded bootstrap julia rust science)
 
 (define-member (person "jgart"
                        "jgart@dismail.de")
-- 
2.39.0


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

[bug#60540] teams: Add scope for Embedded / Bootstrap

[Josselin Poiret via Guix-patches via]

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

Hi everyone,

Vagrant Cascadian <vagrant@debian.org> writes:

> New patch attached which also splits the teams.
>
> They could still probably significantly benefit from
> descriptions... though I haven't heard much of anything weighed in on
> that from the discussion on guix-devel.
>
> live well,
>   vagrant
> From 0ae1c6951bcfd3c1a83e8a86adcbc6afae122c01 Mon Sep 17 00:00:00 2001
> From: Vagrant Cascadian <vagrant@debian.org>
> Date: Wed, 18 Jan 2023 11:17:36 -0800
> Subject: [PATCH] teams: Split Embedded and Bootstrap into separate teams.
>
> * etc/teams.scm.in (embedded-bootstrap): Replace by two separate teams.
>   (embedded): New variable.
>   (bootstrap): New variable.
>   Adjust membership to use new teams.

Any news about this patch?

Best,
-- 
Josselin Poiret

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 682 bytes --]

[bug#60540] teams: Add scope for Embedded / Bootstrap

[Vagrant Cascadian]

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

On 2023-05-15, Josselin Poiret wrote:
> Vagrant Cascadian <vagrant@debian.org> writes:
>
>> New patch attached which also splits the teams.
>>
>> They could still probably significantly benefit from
>> descriptions... though I haven't heard much of anything weighed in on
>> that from the discussion on guix-devel.
>>
>> live well,
>>   vagrant
>> From 0ae1c6951bcfd3c1a83e8a86adcbc6afae122c01 Mon Sep 17 00:00:00 2001
>> From: Vagrant Cascadian <vagrant@debian.org>
>> Date: Wed, 18 Jan 2023 11:17:36 -0800
>> Subject: [PATCH] teams: Split Embedded and Bootstrap into separate teams.
>>
>> * etc/teams.scm.in (embedded-bootstrap): Replace by two separate teams.
>>   (embedded): New variable.
>>   (bootstrap): New variable.
>>   Adjust membership to use new teams.
>
> Any news about this patch?

Maybe it just wasn't controversial enough and I should just push it
already. :)

live well,
  vagrant

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

bug#60540: teams: Add scope for Embedded / Bootstrap

[Vagrant Cascadian]

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

On 2023-05-15, Vagrant Cascadian wrote:
> On 2023-05-15, Josselin Poiret wrote:
>> Vagrant Cascadian <vagrant@debian.org> writes:
>>
>>> New patch attached which also splits the teams.
>>>
>>> They could still probably significantly benefit from
>>> descriptions... though I haven't heard much of anything weighed in on
>>> that from the discussion on guix-devel.
>>>
>>> live well,
>>>   vagrant
>>> From 0ae1c6951bcfd3c1a83e8a86adcbc6afae122c01 Mon Sep 17 00:00:00 2001
>>> From: Vagrant Cascadian <vagrant@debian.org>
>>> Date: Wed, 18 Jan 2023 11:17:36 -0800
>>> Subject: [PATCH] teams: Split Embedded and Bootstrap into separate teams.
>>>
>>> * etc/teams.scm.in (embedded-bootstrap): Replace by two separate teams.
>>>   (embedded): New variable.
>>>   (bootstrap): New variable.
>>>   Adjust membership to use new teams.
>>
>> Any news about this patch?
>
> Maybe it just wasn't controversial enough and I should just push it
> already. :)

Pushed as: 7cf606a5a0515b9a124fb58a06e3daa24a4a4e69

live well,
  vagrant

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]