<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/tools/patman/func_test.py, branch v2025.01</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/tools/patman/func_test.py?h=v2025.01</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/tools/patman/func_test.py?h=v2025.01'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2024-07-03T06:36:32Z</updated>
<entry>
<title>patman: Add a tag for when a patch gets added to a series</title>
<updated>2024-07-03T06:36:32Z</updated>
<author>
<name>Sean Anderson</name>
<email>seanga2@gmail.com</email>
</author>
<published>2024-04-19T02:36:32Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=18de1afd4895bb143eb906e52b93db1b1aac6dfc'/>
<id>urn:sha1:18de1afd4895bb143eb906e52b93db1b1aac6dfc</id>
<content type='text'>
When a patch is added to a series after the initial version, there are no
changes to note except that it is new. This is typically done to suppress
the "(no changes in vN)" message. It's also nice to add a change to the
cover letter so reviewers know there is an additional patch. Add a tag to
automate this process a bit.

There are two nits with the current approach:

- It favors '-' as a bullet point, but some people may prefer '*' (or
  something else)
- Tags (e.g. 'patman: ' in 'patman: foo bar') are not stripped. They are
  probably just noise in most series, but they may be useful for treewide
  series to distinguish 'gpio: frobnicate' from 'reset: frobnicate', so
  I've left them in.

Suggestions for the above appreciated.

Suggested-by: Douglas Anderson &lt;dianders@chromium.org&gt;
Signed-off-by: Sean Anderson &lt;seanga2@gmail.com&gt;
Reviewed-by: Douglas Anderson &lt;dianders@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Add Commit-cc as an alias for Patch-cc</title>
<updated>2024-07-03T06:36:32Z</updated>
<author>
<name>Sean Anderson</name>
<email>seanga2@gmail.com</email>
</author>
<published>2024-04-19T02:36:31Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=b4f73931ed1dda09872ebe6ac47cf2fd6d690704'/>
<id>urn:sha1:b4f73931ed1dda09872ebe6ac47cf2fd6d690704</id>
<content type='text'>
Most tags referring to commits (or patches) are named Commit-something. The
exception is Patch-cc. Add a Commit-cc alias so we can use whichever one is
convenient.

Signed-off-by: Sean Anderson &lt;seanga2@gmail.com&gt;
</content>
</entry>
<entry>
<title>patman: Fix tests if add_maintainers is set to False</title>
<updated>2024-07-03T06:36:32Z</updated>
<author>
<name>Sean Anderson</name>
<email>seanga2@gmail.com</email>
</author>
<published>2024-04-19T02:36:30Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=eba80858039c403cb3e2ef166c7f1418838d4ec2'/>
<id>urn:sha1:eba80858039c403cb3e2ef166c7f1418838d4ec2</id>
<content type='text'>
If add_maintainers is set to False in the user's ~/.patman config, it will
cause the custom_get_maintainer_script to fail since that test expects
maintainers to be added. Set add_maintainer to True in the .patman config
to prevent this.

Fixes: 8c042fb7f9f ("patman: add '--get-maintainer-script' argument")
Signed-off-by: Sean Anderson &lt;seanga2@gmail.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: fix class TestFunctional</title>
<updated>2023-04-28T17:49:00Z</updated>
<author>
<name>Heinrich Schuchardt</name>
<email>heinrich.schuchardt@canonical.com</email>
</author>
<published>2023-04-20T18:07:29Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=c25be4f3ace634f132e92ae651ed9fbd2aa44171'/>
<id>urn:sha1:c25be4f3ace634f132e92ae651ed9fbd2aa44171</id>
<content type='text'>
Variable orig_dir cannot be used in the finally block if it has not be
assigned outside of the try block.

tools/patman/func_test.py:523:21:
E0601: Using variable 'orig_dir' before assignment
(used-before-assignment)

tools/patman/func_test.py:691:21:
E0601: Using variable 'orig_dir' before assignment
(used-before-assignment)

Fixes: fd70986a62af ("patman: Add a test that uses gitpython")
Fixes: be051c0c7741 ("patman: Detect missing upstream in CountCommitsToBranch")
Signed-off-by: Heinrich Schuchardt &lt;heinrich.schuchardt@canonical.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Run get_maintainer.pl in parallel</title>
<updated>2023-03-08T21:15:14Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2023-03-08T18:52:54Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=27409e35d5fa56f1b51b6e0704081133e3881058'/>
<id>urn:sha1:27409e35d5fa56f1b51b6e0704081133e3881058</id>
<content type='text'>
This script can take ages on some series. Try to limit the time by
using threads. If a few stubborn patches remain, show progress so the
user has some idea what is going on.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Douglas Anderson &lt;dianders@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Move library functions into a library directory</title>
<updated>2023-03-08T19:40:49Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2023-02-24T01:18:04Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=4583c00236efd4ee768ff874f92526c229891a05'/>
<id>urn:sha1:4583c00236efd4ee768ff874f92526c229891a05</id>
<content type='text'>
The patman directory has a number of modules which are used by other tools
in U-Boot. This makes it hard to package the tools using pypi since the
common files must be copied along with the tool that uses them.

To address this, move these files into a new u_boot_pylib library. This
can be packaged separately and listed as a dependency of each tool.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: add '--get-maintainer-script' argument</title>
<updated>2023-01-06T02:21:57Z</updated>
<author>
<name>Maxim Cournoyer</name>
<email>maxim.cournoyer@gmail.com</email>
</author>
<published>2022-12-20T05:28:46Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=8c042fb7f9f475367804b26a892fd522ad8fcfcc'/>
<id>urn:sha1:8c042fb7f9f475367804b26a892fd522ad8fcfcc</id>
<content type='text'>
This makes it possible to configure a project to use some other
location or script than the default scripts/get_maintainer.pl one used
in the U-Boot and Linux projects. It can be configured via a .patman
configuration file and accepts arguments, as documented.

Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Signed-off-by: Maxim Cournoyer &lt;maxim.cournoyer@savoirfairelinux.com&gt;
</content>
</entry>
<entry>
<title>patman: locate test data files via __file__ and pathlib</title>
<updated>2023-01-06T02:21:57Z</updated>
<author>
<name>Maxim Cournoyer</name>
<email>maxim.cournoyer@gmail.com</email>
</author>
<published>2022-12-19T22:32:39Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1c77598402890ef148c5e114bb5a3a475305ced8'/>
<id>urn:sha1:1c77598402890ef148c5e114bb5a3a475305ced8</id>
<content type='text'>
Previously it would rely on the executing script location, which could
break for example when running the tests via 'pytest'.

Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Signed-off-by: Maxim Cournoyer &lt;maxim.cournoyer@savoirfairelinux.com&gt;
</content>
</entry>
<entry>
<title>patman: Correct pylint errors</title>
<updated>2022-03-02T15:28:12Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2022-02-11T20:23:18Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=32cc6ae273128510cffc536fc72f260181ef1744'/>
<id>urn:sha1:32cc6ae273128510cffc536fc72f260181ef1744</id>
<content type='text'>
Fix pylint errors that can be fixed and mask those that seem to be
incorrect.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
<entry>
<title>patman: Convert camel case in terminal.py</title>
<updated>2022-02-09T19:30:13Z</updated>
<author>
<name>Simon Glass</name>
<email>sjg@chromium.org</email>
</author>
<published>2022-01-29T21:14:18Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=098b10fb34140f4ac37cfab9c9afade9135710a8'/>
<id>urn:sha1:098b10fb34140f4ac37cfab9c9afade9135710a8</id>
<content type='text'>
Convert this file to snake case and update all files which use it.

Signed-off-by: Simon Glass &lt;sjg@chromium.org&gt;
</content>
</entry>
</feed>
