All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mattijs Korpershoek <mkorpershoek@redhat.com>
To: Simon Glass <sjg@chromium.org>,
	Mattijs Korpershoek <mkorpershoek@kernel.org>
Cc: U-Boot Mailing List <u-boot@lists.denx.de>,
	Tom Rini <trini@konsulko.com>,
	U-Boot Custodians <u-boot-custodians@lists.denx.de>
Subject: Re: Maintenance of Python tools
Date: Wed, 23 Apr 2025 14:43:20 +0200	[thread overview]
Message-ID: <874iyff3pz.fsf@kernel.org> (raw)
In-Reply-To: <CAFLszTiThPOHm+D=Zts9QP0mm4dYHE=kAqS1LAnfF1MLmc=QEg@mail.gmail.com>

Hi Simon,

On mer., avril 23, 2025 at 06:28, Simon Glass <sjg@chromium.org> wrote:

> Hi Mattijs,
>
> On Wed, 23 Apr 2025 at 01:07, Mattijs Korpershoek
> <mkorpershoek@kernel.org> wrote:
>>
>> Hi Simon,
>>
>> On mar., avril 22, 2025 at 17:39, Simon Glass <sjg@chromium.org> wrote:
>>
>> > Hi,
>> >
>> > Tom has indicated that he would like Patman to move out of his tree. I
>> > suggested on another thread[1] that I maintain it in my 'sjg' tree, so
>> > here is a new thread to discuss this.
>> >
>> > I have already done this for the qemu/efi/coreboot scripts as Tom has
>> > NAK'ed patches for those.
>> >
>> > For the other tools there is going to be quite a bit of churn, as I
>> > would like to resolve most of the many Python warnings.
>> >
>> > Given the shared source between the tools, it would be easier for me
>> > to do the same for buildman, binman and qconfig. I am thinking that I
>> > might try a move to allow Gitlab pull-requests for reviews on these as
>> > well as the mailing list, if that is useful.
>> >
>> > For tools which need to sync back to Tom's tree (i.e. not patman), I
>> > or Tom could do a pull request every now and then, omitting any
>> > changes that relate to pylint.
>> >
>> > Please let me know your thoughts. The timing is good as I am going to
>> > be sending out a new Patman feature in the next few weeks and it is a
>> > few thousand more lines of code.
>>
>> I have a preference for binman staying in the U-Boot upstream (Tom's)
>> tree. AFAIK, binman is used by the CI and is also very useful for composing
>> "complex" bootloader images (For example for the TI k3 architecture).
>> I don't know a good replacement of binman and I'm afraid that people
>> will go back to ad-hoc scripts if binman gets removed from the tree :(
>>
>> On the other hand, patman is a workflow tool that's not (I think) that
>> specific to U-Boot and is (to me) replaceable by b4.
>>
>> I understand that code sharing makes it more difficult to only move
>> buildman out of upstream, but in a perfect world, I'd like binman to
>> stay in upstream.
>
> Just to clarify this, I'm not suggesting removing binman. It's just
> the maintenance and development of new features that I'm referring to.
> We would still sync source back to Tom's tree.

Got it, thanks.

Wouldn't moving the binman development out of upstream reduce
contributions/visibility?
Or do you think that proposing alternative development process (like
Gitlab merge requests) would attract more folks?

What would be the policy for development/syncing back to upstream?

What happens if binman evolves in a way that's not aligned/practical for
upstream (Tom's tree) ?

Would that mean that we would have to maintain a fork in upstream?

>
> Regards,
> SImon
>
>> > [1] https://lore.kernel.org/u-boot/CAFLszTg7-yM0L8uuJyN7Jpsb1LbvOYO76cUWj+H719mfc97mMQ@mail.gmail.com/


  reply	other threads:[~2025-04-23 14:26 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-22 23:39 Maintenance of Python tools Simon Glass
2025-04-23  7:07 ` Mattijs Korpershoek
2025-04-23 12:28   ` Simon Glass
2025-04-23 12:43     ` Mattijs Korpershoek [this message]
2025-04-23 13:05       ` Simon Glass
2025-04-23 13:36         ` Heinrich Schuchardt
2025-04-23 14:08 ` Tom Rini
2025-04-24 18:34   ` Simon Glass
2025-04-24 18:55     ` Tom Rini
2025-04-25  0:16       ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=874iyff3pz.fsf@kernel.org \
    --to=mkorpershoek@redhat.com \
    --cc=mkorpershoek@kernel.org \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.com \
    --cc=u-boot-custodians@lists.denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.