Hi Tom, On Tue, 8 Apr 2025 at 16:13, Tom Rini <tr...@konsulko.com> wrote: > > On Mon, Apr 07, 2025 at 10:51:42PM +1200, Simon Glass wrote: > > > > > The gitutil module uses Patman's settings module, which is not allowed > > as it is supposed to be a separate package. This series ties up this > > dependency. > > > > This series depends on these two patches being applied: > > > > https://patchwork.ozlabs.org/project/uboot/patch/20250227192735.406389-1-...@chromium.org/ > > https://patchwork.ozlabs.org/project/uboot/patch/20250328130225.2607974-1-...@chromium.org/ > > > > > > Simon Glass (5): > > patman: Untangle settings from gitutil > > patman: Pass the alias dict into gitutil.build_email_list() > > patman: Pass the alias dict into gitutil.email_patches() > > patman: Pass aliases to Series.MakeCcFile() > > patman: Update Series.ShowActions() to pass alias > > > > tools/patman/control.py | 11 ++++++++--- > > tools/patman/func_test.py | 8 ++++---- > > tools/patman/series.py | 32 ++++++++++++++++++++++---------- > > tools/u_boot_pylib/gitutil.py | 34 ++++++++++++++-------------------- > > 4 files changed, 48 insertions(+), 37 deletions(-) > > I just want to reiterate my question / request to host these outside of > the U-Boot sources themselves so they can be managed (both in the sense > of U-Boot as user and as maintainer of sources) following normal Python > best practices? I'd be fine even with something under > https://source.denx.de/u-boot/<something>/patman, etc.
Yes I am working towards that. This series cleans up something missed in the last series. After this series there is really just 'patchstream' left, which is used by buildman and patman. I haven't quite come to terms with the idea of moving it to u_boot_pylib yet. What are the best practices you are referring to? As to location, I can keep it in my tree and just delete it from yours if you like. Regards, Simon
