From: Mattijs Korpershoek <mkorpershoek@kernel.org>
To: Simon Glass <sjg@chromium.org>,
U-Boot Mailing List <u-boot@lists.denx.de>
Cc: 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 09:07:11 +0200 [thread overview]
Message-ID: <87a587fja8.fsf@kernel.org> (raw)
In-Reply-To: <CAFLszTjnXQu3bwWt3Do7S_Vu0UO5si4HRUeW9bv0981WbUn_7Q@mail.gmail.com>
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.
Thanks,
Mattijs
>
> Regards,
> Simon
>
> [1] https://lore.kernel.org/u-boot/CAFLszTg7-yM0L8uuJyN7Jpsb1LbvOYO76cUWj+H719mfc97mMQ@mail.gmail.com/
next prev parent reply other threads:[~2025-04-23 7:07 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 [this message]
2025-04-23 12:28 ` Simon Glass
2025-04-23 12:43 ` Mattijs Korpershoek
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=87a587fja8.fsf@kernel.org \
--to=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.