GNU bug report logs - #60540
teams: Add scope for Embedded / Bootstrap

Previous Next

Package: guix-patches;

Reported by: Vagrant Cascadian <vagrant <at> debian.org>

Date: Wed, 4 Jan 2023 05:54:01 UTC

Severity: normal

Tags: patch

Done: Vagrant Cascadian <vagrant <at> debian.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 60540 in the body.
You can then email your comments to 60540 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#60540; Package guix-patches. (Wed, 04 Jan 2023 05:54:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Vagrant Cascadian <vagrant <at> debian.org>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Wed, 04 Jan 2023 05:54:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Vagrant Cascadian <vagrant <at> debian.org>
To: guix-patches <at> gnu.org
Subject: teams: Add scope for Embedded / Bootstrap
Date: Tue, 03 Jan 2023 21:53:25 -0800
[Message part 1 (text/plain, inline)]
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
[0003-teams-Add-scope-for-Embedded-Bootstrap-team.patch (text/x-diff, inline)]
From 6c8076261cf1a93134e3435124d4a7e9dcbbba1c Mon Sep 17 00:00:00 2001
From: Vagrant Cascadian <vagrant <at> 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

[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#60540; Package guix-patches. (Sun, 08 Jan 2023 13:44:02 GMT) Full text and rfc822 format available.

Message #8 received at 60540 <at> debbugs.gnu.org (full text, mbox):

From: Mathieu Othacehe <othacehe <at> gnu.org>
To: Vagrant Cascadian <vagrant <at> debian.org>
Cc: 60540 <at> debbugs.gnu.org
Subject: Re: bug#60540: teams: Add scope for Embedded / Bootstrap
Date: Sun, 08 Jan 2023 14:43:04 +0100
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




Information forwarded to guix-patches <at> gnu.org:
bug#60540; Package guix-patches. (Tue, 10 Jan 2023 23:22:02 GMT) Full text and rfc822 format available.

Message #11 received at 60540 <at> debbugs.gnu.org (full text, mbox):

From: Vagrant Cascadian <vagrant <at> debian.org>
To: Mathieu Othacehe <othacehe <at> gnu.org>
Cc: 60540 <at> debbugs.gnu.org
Subject: Re: bug#60540: teams: Add scope for Embedded / Bootstrap
Date: Tue, 10 Jan 2023 15:21:12 -0800
[Message part 1 (text/plain, inline)]
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
[signature.asc (application/pgp-signature, inline)]

Added tag(s) patch. Request was from Bruno Victal <mirai <at> makinata.eu> to control <at> debbugs.gnu.org. (Wed, 11 Jan 2023 17:38:01 GMT) Full text and rfc822 format available.

Information forwarded to guix-patches <at> gnu.org:
bug#60540; Package guix-patches. (Wed, 18 Jan 2023 19:31:02 GMT) Full text and rfc822 format available.

Message #16 received at 60540 <at> debbugs.gnu.org (full text, mbox):

From: Vagrant Cascadian <vagrant <at> debian.org>
To: Mathieu Othacehe <othacehe <at> gnu.org>
Cc: 60540 <at> debbugs.gnu.org
Subject: Re: bug#60540: teams: Add scope for Embedded / Bootstrap
Date: Wed, 18 Jan 2023 11:30:02 -0800
[Message part 1 (text/plain, inline)]
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
[0001-teams-Split-Embedded-and-Bootstrap-into-separate-tea.patch (text/x-diff, inline)]
From 0ae1c6951bcfd3c1a83e8a86adcbc6afae122c01 Mon Sep 17 00:00:00 2001
From: Vagrant Cascadian <vagrant <at> 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 <at> gnu.org")
-  core home embedded-bootstrap mentors)
+  core home embedded bootstrap mentors)
 
 (define-member (person "Andreas Enge"
                        "andreas <at> enge.fr")
@@ -458,7 +465,7 @@ (define-member (person "Leo Famulari"
 
 (define-member (person "Efraim Flashner"
                        "efraim <at> flashner.co.il")
-  embedded-bootstrap julia rust science)
+  embedded bootstrap julia rust science)
 
 (define-member (person "jgart"
                        "jgart <at> dismail.de")
-- 
2.39.0

[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#60540; Package guix-patches. (Mon, 15 May 2023 18:55:02 GMT) Full text and rfc822 format available.

Message #19 received at 60540 <at> debbugs.gnu.org (full text, mbox):

From: Josselin Poiret <dev <at> jpoiret.xyz>
To: Vagrant Cascadian <vagrant <at> debian.org>, Mathieu Othacehe <othacehe <at> gnu.org>
Cc: 60540 <at> debbugs.gnu.org
Subject: Re: [bug#60540] teams: Add scope for Embedded / Bootstrap
Date: Mon, 15 May 2023 20:54:11 +0200
[Message part 1 (text/plain, inline)]
Hi everyone,

Vagrant Cascadian <vagrant <at> 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 <at> 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
[signature.asc (application/pgp-signature, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#60540; Package guix-patches. (Mon, 15 May 2023 19:23:02 GMT) Full text and rfc822 format available.

Message #22 received at 60540 <at> debbugs.gnu.org (full text, mbox):

From: Vagrant Cascadian <vagrant <at> debian.org>
To: Josselin Poiret <dev <at> jpoiret.xyz>, Mathieu Othacehe <othacehe <at> gnu.org>
Cc: 60540 <at> debbugs.gnu.org
Subject: Re: [bug#60540] teams: Add scope for Embedded / Bootstrap
Date: Mon, 15 May 2023 12:21:52 -0700
[Message part 1 (text/plain, inline)]
On 2023-05-15, Josselin Poiret wrote:
> Vagrant Cascadian <vagrant <at> 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 <at> 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
[signature.asc (application/pgp-signature, inline)]

Reply sent to Vagrant Cascadian <vagrant <at> debian.org>:
You have taken responsibility. (Wed, 24 May 2023 22:56:01 GMT) Full text and rfc822 format available.

Notification sent to Vagrant Cascadian <vagrant <at> debian.org>:
bug acknowledged by developer. (Wed, 24 May 2023 22:56:01 GMT) Full text and rfc822 format available.

Message #27 received at 60540-done <at> debbugs.gnu.org (full text, mbox):

From: Vagrant Cascadian <vagrant <at> debian.org>
To: Josselin Poiret <dev <at> jpoiret.xyz>, Mathieu Othacehe <othacehe <at> gnu.org>
Cc: 60540-done <at> debbugs.gnu.org
Subject: Re: [bug#60540] teams: Add scope for Embedded / Bootstrap
Date: Wed, 24 May 2023 15:55:26 -0700
[Message part 1 (text/plain, inline)]
On 2023-05-15, Vagrant Cascadian wrote:
> On 2023-05-15, Josselin Poiret wrote:
>> Vagrant Cascadian <vagrant <at> 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 <at> 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
[signature.asc (application/pgp-signature, inline)]

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 22 Jun 2023 11:24:09 GMT) Full text and rfc822 format available.

This bug report was last modified 301 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.