Re: [PATCH v3 0/5] pylibfdt: Add installation support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]



On Wed, Apr 05, 2017 at 10:01:37AM -0600, Simon Glass wrote:
> This series adds support for installation of the Pylib module. It adjusts
> the setup.py file to provide this functionality and the Makefile rules
> to call it correctly.
> 
> It also adds a way to disable building the Python module. This is useful
> since some build systems want to use setup.py to do both the build and
> the install step. In this case the correct build commands would be:
> 
>    make NO_PYTHON=1
>    make install_pylibfdt SETUP_PREFIX=/path/to/install_prefix
> 
> Version 3 adds support for running setup.py stand-alone:
> 
>    ./pylibfdt/setup.py --install_path
>    [--prefix=/path/to/install_dir]

I've applied 1&2, 3 breaks make check, somehow, so I've left the rest.

> 
> Changes in v3:
> - Add new patch to fix code style in setup.py
> - Update README to suggest using setup.py for an easy install
> - Allow setup.py to be run from the base directory
> - Make setup.py executable
> - Use define..endef instead of line continuations
> - Add new patch to move to Makefile constructs for NO_PYTHON
> - Add a new patch to use setup.py to build the swig file
> 
> Changes in v2:
> - Add new patch to rename libfdt.swig to libfdt.i
> - Add new patch to allow setup.py to operation stand-alone
> - Rebase to master
> 
> Simon Glass (5):
>   pylibfdt: Rename libfdt.swig to libfdt.i
>   pylibfdt: Fix code style in setup.py
>   pylibfdt: Allow setup.py to operate stand-alone
>   pylibfdt: Use Makefile constructs to implement NO_PYTHON
>   pylibfdt: Use setup.py to build the swig file
> 
>  Makefile                           |  13 ++++-
>  README                             |  14 +++--
>  pylibfdt/Makefile.pylibfdt         |  29 ++++------
>  pylibfdt/{libfdt.swig => libfdt.i} |   0
>  pylibfdt/setup.py                  | 113 ++++++++++++++++++++++++++++++++-----
>  5 files changed, 130 insertions(+), 39 deletions(-)
>  rename pylibfdt/{libfdt.swig => libfdt.i} (100%)
>  mode change 100644 => 100755 pylibfdt/setup.py
> 

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Device Tree]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux