u-boot.git/tools/patman, branch v2024.01Unnamed repository; edit this file 'description' to name the repository.http://cgit.235523.xyz/u-boot.git/atom/tools/patman?h=v2024.012023-11-15T03:04:01Zpatman: Correct Python 3.6 behaviour2023-11-15T03:04:01ZSimon Glasssjg@chromium.org2023-11-04T16:25:24Zurn:sha1:abf7004321c05333afb56ae230095f9bee55e924
The importlib_resources import is not actually used. Fix this so that
patman can run on Python 3.6 to some extent, once
'pip3 install importlib-resources' has been run.
Signed-off-by: Simon Glass <sjg@chromium.org>
patman: Avoid using func_test at top level2023-11-15T03:04:01ZSimon Glasssjg@chromium.org2023-11-04T16:25:23Zurn:sha1:dfdf621ff285da4e8e5995e05d570ebe7eb2e318
Import this only when it is needed, since it is not present when
installed via 'pip install'.
Signed-off-by: Simon Glass <sjg@chromium.org>
Fixes: https://source.denx.de/u-boot/u-boot/-/issues/26
patman: Correct easy pylint warnings in __main__2023-11-15T03:04:00ZSimon Glasssjg@chromium.org2023-11-04T16:25:22Zurn:sha1:21229c921bf62a5480e887d71ba13ac482282870
Tidy up the code a little to reduce the number of pylint warnings.
Signed-off-by: Simon Glass <sjg@chromium.org>
patman: Move the main program into a function2023-11-15T03:04:00ZSimon Glasssjg@chromium.org2023-11-04T16:25:21Zurn:sha1:e296a3c73d22bf95f38318a8dac2dc0177a3b145
Add a new run_patman() function to hold the main logic.
Signed-off-by: Simon Glass <sjg@chromium.org>
patman: Split out arg parsing into its own file2023-11-15T03:04:00ZSimon Glasssjg@chromium.org2023-11-04T16:25:20Zurn:sha1:18f8830ab95328431a61960b30bfa1755515e8b9
Move this code into a separate cmdline module, as is done with the
other tools.
Use the same HAS_TESTS check as buildman
Signed-off-by: Simon Glass <sjg@chromium.org>
buildman: Support upstream branch name containing /2023-11-03T02:38:01ZSimon Glasssjg@chromium.org2023-10-30T17:22:30Zurn:sha1:a44cb1f2402a7f20f68e245bd37d6bdfe34e70f2
Buildman assumes that branch names do not have a slash in them, since
slash is used to delimit remotes, etc. This means that a branch called
'WIP/tryme' in remote dm ends up being 'tryme'.
Adjust the logic a little, to try to accommodate this.
For now, no tests are added for this behaviour.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
patman: Add a 'keep_change_id' setting2023-11-02T15:38:54ZMaxim Cournoyermaxim.cournoyer@gmail.com2023-10-13T03:06:24Zurn:sha1:a13af89e10391189b87ad32616ba688d6d7c5878
A Change-Id can be useful for traceability purposes, and some projects
may wish to have them preserved. This change makes it configurable
via a new 'keep_change_id' setting.
Signed-off-by: Maxim Cournoyer <maxim.cournoyer@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
patman: Add a little documentation on the checkpatch tests2023-10-24T20:34:45ZSimon Glasssjg@chromium.org2023-10-13T16:28:33Zurn:sha1:2f34b0331a60239b7708f125ad621e4e62f51803
These texts lack comments. Add some so that it is clearer what is going
on.
Signed-off-by: Simon Glass <sjg@chromium.org>
checkpatch.pl: Make common.h check boarder2023-10-24T20:34:45ZTom Rinitrini@konsulko.com2023-10-13T16:28:32Zurn:sha1:a8384f8da1d6f9cab9fe941cc42d60f1a4acc4ea
At this point in time we should not add common.h to any new files, so
make checkpatch.pl complain.
Signed-off-by: Tom Rini <trini@konsulko.com>
Signed-off-by: Simon Glass <sjg@chromium.org>
patman: Respect include directive on Git config lookup2023-09-23T15:00:37ZFei Shaofshao@chromium.org2023-09-08T14:15:59Zurn:sha1:ce8cb76c7cf5cadf9d4b6293df8fbede74535d93
People may put their user name and email in a local config file and
reference it by the include.* directives, however `git config --global`
doesn't look up the included configs by default.
Enable the --includes option explicitly to support such use cases.
Signed-off-by: Fei Shao <fshao@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>