[U-Boot] [PATCH v3] patman: add distutils based installer

Simon Glass sjg at chromium.org
Thu Jul 23 04:10:58 CEST 2015


Hi Chris,

On 22 July 2015 at 03:21, Chris Packham <judge.packham at gmail.com> wrote:
> 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 at chromium.org>
> Signed-off-by: Chris Packham <judge.packham at gmail.com>
>
> ---
> This gives us something that can be distributed separately as well as
> in-tree. The import trick allows the python module "patman" to be
> distributed and enables in-tree use without moving things around. An
> alternative would be to move the files into a sub directory
> (unfortunately a directory called "patman" would clash with the existing
> symlink).
>
> Thanks,
> Chris
>
> Changes in v3:
> - Add installation instructions to tools/patman/README
> - Drop RFC, Add sign-off
> - Collect tested tag from Simon
>
> Changes in v2:
> - Install as "patman" package
> - Allow running in-tree or out-of-tree
>
>  tools/patman/README      | 11 +++++++++++
>  tools/patman/__init__.py |  3 +++
>  tools/patman/patman.py   | 20 ++++++++++++--------
>  tools/patman/setup.py    | 13 +++++++++++++
>  4 files changed, 39 insertions(+), 8 deletions(-)
>  create mode 100644 tools/patman/__init__.py
>  create mode 100644 tools/patman/setup.py

Acked-by: Simon Glass <sjg at chromium.org>

BTW in the README it should be U-Boot with a capital b. I can fix that
up when I apply it if you like. For me I needed 'sudo' on the python
command. Is that expected?

Regards
Simon


More information about the U-Boot mailing list