- 08 Feb, 2017 1 commit
-
-
Simon Glass authored
It is convenient to be able to deal with checkpatch warnings in the same way as build warnings. Tools such as emacs and kate can quickly locate the source file and line automatically. To achieve this, adjust the format to match the C compiler, and output to stderr. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
- 09 Oct, 2016 7 commits
-
-
Paul Burton authored
In python 3.x StringIO is no longer a module, and the class can instead be found in the io module. Adjust the code in the doctest input to account for both. Signed-off-by:
Paul Burton <paul.burton@imgtec.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Paul Burton authored
In python 3.x the iteritems() method has been removed from dictionaries, and the items() method does effectively the same thing. On python 2.x using items() is a little less efficient since it involves copying data, but as speed isn't a concern in this code switch to using items() anyway for simplicity. Signed-off-by:
Paul Burton <paul.burton@imgtec.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Paul Burton authored
In python 3.x module names used in import statements are case sensitive, and the configparser module is named in all lower-case. Import it as such in order to avoid errors when running with python 3.x. Signed-off-by:
Paul Burton <paul.burton@imgtec.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Paul Burton authored
Syntax for exception handling is a little more strict in python 3.x. Convert all uses to a form accepted by both python 2.x & python 3.x. Signed-off-by:
Paul Burton <paul.burton@imgtec.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Paul Burton authored
In python 3.x, print must be used as a function call. Convert all print statements to the function call style, importing from __future__ where we print with no trailing newline or print to a file object. Signed-off-by:
Paul Burton <paul.burton@imgtec.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Paul Burton authored
In preparation for running on python 3.x, which will refuse to run scripts which mix tabs & spaces for indentation, replace 2 tab characters present in series.py with spaces. Signed-off-by:
Paul Burton <paul.burton@imgtec.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
Output which does not include a newline will not be displayed unless flushed. Add a flush to ensure that it becomes visible. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
- 19 Sep, 2016 3 commits
-
-
Simon Glass authored
When tools want to display information of varying levels of importance, it helps to provide the user with control over the verbosity of these messages. Progress messages work best if they are displayed and then removed from the display when no-longer relevant. Add a new tout library (terminal out) to handle these tasks. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
For tools which want to use input files and temporary output, it is useful to have the handling of these dealt with in one place. Add a new library which allows input files to be read, and output files to be written, all based on a common directory structure. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
It is more useful to have this method raise an error when something goes wrong. Make this the default and adjust the few callers that don't want to use it this way. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
- 11 Jul, 2016 5 commits
-
-
Bin Meng authored
The following python error: Traceback (most recent call last): File "./tools/patman/patman", line 144, in <module> series = patchstream.FixPatches(series, args) File "./tools/patman/patchstream.py", line 477, in FixPatches commit = series.commits[count] IndexError: list index out of range is seen when: - 'END' is missing in those tags - those tags are put in the last part in a commit message - the commit is not the last commit of the series Add testing logic to see if a new commit starts. Signed-off-by:
Bin Meng <bmeng.cn@gmail.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Bin Meng authored
'Series-changes' uses blank line to indicate its end. If that is missing, series internal state variable 'in_change' may be wrong. Correct its state. Signed-off-by:
Bin Meng <bmeng.cn@gmail.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Bin Meng authored
If 'END' is missing in a 'Cover-letter' section, and that section happens to show up at the very end of the commit message, and the commit is the last commit of the series, patman fails to generate cover letter for us. Handle this in CloseCommit of patchstream. Signed-off-by:
Bin Meng <bmeng.cn@gmail.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Bin Meng authored
'Cover-letter', 'Series-notes' and 'Commit-notes' tags require an 'END' to be put at the end of its section. If we forget to put an 'END' in those sections, and these sections are followed by another patman tag, patman generates incorrect patches. This adds codes to handle such scenario. Signed-off-by:
Bin Meng <bmeng.cn@gmail.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Bin Meng authored
Like other patman tags, use a new variable cover_match to indicate a match for 'Cover-letter'. Signed-off-by:
Bin Meng <bmeng.cn@gmail.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
- 18 Mar, 2016 1 commit
-
-
Simon Glass authored
At present if you try to use buildman with the branch 'test' it will complain that it is unsure whether you mean the branch or the directory. This is a feature of the 'git log' command that buildman uses. Fix it by resolving the ambiguity. Signed-off-by:
Simon Glass <sjg@chromium.org> Reviewed-by:
Joe Hershberger <joe.hershberger@ni.com>
-
- 14 Mar, 2016 2 commits
-
-
Simon Glass authored
It is convenient to install symlinks to buildman and patman in the search patch, such as /usr/local/bin. But when this is done, the -H option fails to work because it looks in the directory containing the symlink instead of its target. Fix this. Signed-off-by:
Simon Glass <sjg@chromium.org> Reviewed-by:
Joe Hershberger <joe.hershberger@ni.com>
-
Simon Glass authored
Fix this nit to keep the code consistent. Signed-off-by:
Simon Glass <sjg@chromium.org> Reviewed-by:
Joe Hershberger <joe.hershberger@ni.com>
-
- 25 Feb, 2016 1 commit
-
-
Albert ARIBAUD authored
A patman series with a 'Series-notes' section causes buildman to crash with: self.series.notes += self.section TypeError: cannot concatenate 'str' and 'list' objects Fix by initializing series.notes as a one-element array rather than a scalar. Signed-off-by:
Albert ARIBAUD <albert.u.boot@aribaud.net> Acked-by:
Simon Glass <sjg@chromium.org> Tested-by:
Simon Glass <sjg@chromium.org>
-
- 06 Feb, 2016 1 commit
-
-
Bin Meng authored
Correct spelling of "U-Boot" shall be used in all written text (documentation, comments in source files etc.). Signed-off-by:
Bin Meng <bmeng.cn@gmail.com> Reviewed-by:
Heiko Schocher <hs@denx.de> Reviewed-by:
Simon Glass <sjg@chromium.org> Reviewed-by:
Minkyu Kang <mk7.kang@samsung.com>
-
- 21 Jan, 2016 1 commit
-
-
Mateusz Kulikowski authored
Add option to create threaded series of patches. With it, it will be possible to create patch threads like this: [PATCH 0/10] Add support for time travel [PATCH 1/10] Add Flux Capacitor driver [PATCH 2/10] Add Mr. Fusion driver (...) Internally it will call git send-email with --thread option Signed-off-by:
Mateusz Kulikowski <mateusz.kulikowski@gmail.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
- 09 Sep, 2015 1 commit
-
-
Masahiro Yamada authored
This reverts commit 19b4a336. Since that commit, patman generates useless patches for file removal; "git format -D" prints only the header but not the diff when deleting files, and "git am" always refuses such patches. The following is the quotation from "man git-format-patch": -D, --irreversible-delete Omit the preimage for deletes, i.e. print only the header but not the diff between the preimage and /dev/null. The resulting patch is not meant to be applied with patch nor git apply; this is solely for people who want to just concentrate on reviewing the text after the change. In addition, the output obviously lack enough information to apply such a patch in reverse, even manually, hence the name of the option. Signed-off-by:
Masahiro Yamada <yamada.masahiro@socionext.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
- 31 Aug, 2015 1 commit
-
-
Simon Glass authored
This requires 'Series-cover_cc' at present which is incorrect. Fix it. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
- 26 Aug, 2015 1 commit
-
-
Masahiro Yamada authored
This allows Patman to generate smaller patches for file removal. Signed-off-by:
Masahiro Yamada <yamada.masahiro@socionext.com> Acked-by:
Simon Glass <sjg@chromium.org> Acked-by:
Heiko Schocher <hs@denx.de>
-
- 31 Jul, 2015 1 commit
-
-
Simon Glass authored
Commit 488d19cb (patman: add distutils based installer) has the side effect of making patman run twice with each invocation. Fix this by checking for 'main program' invocation in patman.py. This is good practice in any case. Signed-off-by:
Simon Glass <sjg@chromium.org> Reviewed-by:
Chris Packham <judge.packham@gmail.com>
-
- 28 Jul, 2015 1 commit
-
-
Chris Packham authored
To make it easier to use patman on other projects add a distutils style installer. Now patman can be installed with cd u-boot/tools/patman && python setup.py install There are also the usual distutils options for creating source/binary distributions of patman. Tested-by:
Simon Glass <sjg@chromium.org> Signed-off-by:
Chris Packham <judge.packham@gmail.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
- 23 Apr, 2015 1 commit
-
-
Wu, Josh authored
For the local project, we may specified format.subjectprefix setting. Then the patch will be formated as [Project_prefix][PATCH]. But patman will not check this setting. It will remove the format.subjectprefix. So This patch will let patman check this setting and add it as a project prefix. Signed-off-by:
Josh Wu <josh.wu@atmel.com> Acked-by:
Simon Glass <sjg@chromium.org> Tested-by:
Simon Glass <sjg@chromium.org>
-
- 18 Apr, 2015 1 commit
-
-
Wu, Josh authored
Make cover letter shows like 0/x, 00/xx and 000/xxx etc. Signed-off-by:
Josh Wu <josh.wu@atmel.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
- 15 Feb, 2015 2 commits
-
-
Simon Glass authored
We should read this file to obtain a set of aliases. This reduces the need to create them in the ~/.patman file. This feature did exist in some version of patman, and is mentioned in the help but it did not find its way upstream. Reported-by:
Graeme Russ <gruss@tss-engineering.com> Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Simon Glass authored
This causes an error when trying to build a local branch which has a local branch as its upstream. Signed-off-by:
Simon Glass <sjg@chromium.org> Reported-by:
Masahiro Yamada <yamada.m@jp.panasonic.com>
-
- 30 Jan, 2015 3 commits
-
-
Simon Glass authored
Add an explanation for how to set up git so that patman can find the alias file. Fix up the get_maintainers message too. Reported-by:
Scott Wood <scottwood@freescale.com> Signed-off-by:
Simon Glass <sjg@chromium.org>
-
Scott Wood authored
True commit lines start at column zero. Anything that is indented is part of the commit message instead. I noticed this by trying to run buildman with commit e3a4facd as master, which contained a reference to a Linux commit inside the commit message. ProcessLine saw that as a genuite commit line, and thus buildman tried to build it, and died with an exception because that SHA is not present in the U-Boot tree. Signed-off-by:
Scott Wood <scottwood@freescale.com> Acked-by:
Simon Glass <sjg@chromium.org>
-
Peter Tyser authored
When run with the --dry-run argument patman prints out information showing what it would do. This information currently doesn't line up with what patman/git send-email really do. Some basic examples: - If an email address is addressed via "Series-cc" and "Patch-cc" patman shows that email address would be CC-ed two times. - If an email address is addressed via "Series-to" and "Patch-cc" patman shows that email address would be sent TO and CC-ed. - If an email address is addressed from a combination of tag aliases, get_maintainer.pl output, "Series-cc", "Patch-cc", etc patman shows that the email address would be CC-ed multiple times. Patman currently does try to send duplicate emails like the --dry-run output shows, but "git send-email" intelligently removes duplicate addresses so this patch shouldn't change the non-dry-run functionality. Change patman's output and email addressing to line up with the "git send-email" logic. This trims down patman's dry-run output and prevents confusion about what patman will do when emails are actually sent. Signed-off-by:
Peter Tyser <ptyser@xes-inc.com> Acked-by:
Simon Glass <sjg@chromium.org> Tested-by:
Simon Glass <sjg@chromium.org>
-
- 15 Jan, 2015 2 commits
-
-
Simon Glass authored
Adjust the -b flag to permit a range expression as well as a branch. Signed-off-by:
Simon Glass <sjg@chromium.org> Suggested-by:
Daniel Schwierzeck <daniel.schwierzeck@gmail.com> Tested-by:
Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
-
Simon Glass authored
Buildman normally obtains the upstream commit by asking git. Provided that the branch was created with 'git checkout -b <branch> <some_upstream>' then this normally works. When there is no upstream, we can try to guess one, by looking up through the commits until we find a branch. Add a function to try this and print a warning if buildman ends up relying on it. Also update the documentation to match. Signed-off-by:
Simon Glass <sjg@chromium.org> Suggested-by:
Wolfgang Denk <wd@denx.de>
-
- 27 Nov, 2014 1 commit
-
-
Simon Glass authored
Even with the initial 8 characeters of the hash we will sometimes get a collision. Use the full hash. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
- 03 Nov, 2014 1 commit
-
-
Simon Glass authored
Even with the initial 8 characeters of the hash we will sometimes get a collision. Use the full hash. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
- 21 Sep, 2014 1 commit
-
-
Simon Glass authored
The get_maintainers script is a useful default, but sometimes is copies too many people, or takes a long time to run. Add an option to disable it and update the README. Signed-off-by:
Simon Glass <sjg@chromium.org>
-
- 09 Sep, 2014 1 commit
-
-
Vadim Bendebury authored
For an occasional user of patman some failures are not obvious: for instance when checkpatch reports warnings, the dry run still reports that the email would be sent. If it is not dry run, the warnings are shown on the screen, but it is not clear that the email was not sent. Add some code to report failure to send email explicitly. Tested by running the script on a patch with style violations, observed error messages in the script output. Signed-off-by:
Vadim Bendebury <vbendeb@chromium.org> Reviewed-by:
Doug Anderson <dianders@chromium.org> Acked-by:
Simon Glass <sjg@chromium.org>
-