From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Sun, 26 Apr 2015 19:06:20 +0200 Subject: [Buildroot] [PATCH] python: Allow pyo only and variants In-Reply-To: <1430065819-12745-1-git-send-email-maxime.hadjinlian@gmail.com> References: <1430065819-12745-1-git-send-email-maxime.hadjinlian@gmail.com> Message-ID: <20150426190620.4647bb75@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Dear Maxime Hadjinlian, On Sun, 26 Apr 2015 18:30:19 +0200, Maxime Hadjinlian wrote: > Enable the fact that you may want to use only pyo files on you target. > The main differences between the file format are as follow: > > - py: The source file > - pyc: It's a compiled bytecode, makes import faster. > - pyo: It's an optimized bytecode, mainly it removes the asserts > > Also adding the various "mix" options a user may want between theses > three options. > > Signed-off-by: Maxime Hadjinlian What about Python 3 ? > choice > prompt "python module format to install" > - default BR2_PACKAGE_PYTHON_PYC_ONLY > + default BR2_PACKAGE_PYTHON_PYO_ONLY > help > - Select Python module format to install on target (py, pyc or both) > + Select Python module format to install on target (py, pyc, pyo or a mix) > > config BR2_PACKAGE_PYTHON_PY_ONLY > bool ".py sources only" > @@ -28,9 +28,21 @@ config BR2_PACKAGE_PYTHON_PY_ONLY > config BR2_PACKAGE_PYTHON_PYC_ONLY > bool ".pyc compiled sources only" > > +config BR2_PACKAGE_PYTHON_PYO_ONLY > + bool ".pyo compiled sources only" > + > config BR2_PACKAGE_PYTHON_PY_PYC > bool ".py sources and .pyc compiled" > > +config BR2_PACKAGE_PYTHON_PY_PYO > + bool ".py sources and .pyo compiled" > + > +config BR2_PACKAGE_PYTHON_PYC_PYO > + bool ".pyc and .pyo compiled sources" > + > +config BR2_PACKAGE_PYTHON_PY_PYC_PYO > + bool ".py sources and both .pyc, .pyo compiled" Hum do we really want/need a choice with all possible combinations? > + > endchoice > > menu "core python modules" > diff --git a/package/python/python.mk b/package/python/python.mk > index 4a3e71c..a175f19 100644 > --- a/package/python/python.mk > +++ b/package/python/python.mk > @@ -33,8 +33,7 @@ HOST_PYTHON_CONF_OPTS += \ > --disable-test-modules \ > --disable-bz2 \ > --disable-ssl \ > - --disable-ossaudiodev \ > - --disable-pyo-build > + --disable-ossaudiodev > > # Make sure that LD_LIBRARY_PATH overrides -rpath. > # This is needed because libpython may be installed at the same time that > @@ -142,8 +141,7 @@ PYTHON_CONF_OPTS += \ > --disable-gdbm \ > --disable-tk \ > --disable-nis \ > - --disable-dbm \ > - --disable-pyo-build > + --disable-dbm I think we want to keep --disable-pyc-build or --disable-pyo-build when possible, since it saves quite a bit of time when building Python. > # This is needed to make sure the Python build process doesn't try to > # regenerate those files with the pgen program. Otherwise, it builds > @@ -217,16 +215,40 @@ PYTHON_PATH = $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR)/sysconfigdata/ > $(eval $(autotools-package)) > $(eval $(host-autotools-package)) > > +ifeq ($(BR2_PACKAGE_PYTHON_PYO_ONLY),y) > +define PYTHON_FINALIZE_TARGET > + find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) \( -name '*.py' -or -name '*.pyc' \) -print0 | xargs -0 rm -f > +endef > +endif > + > ifeq ($(BR2_PACKAGE_PYTHON_PYC_ONLY),y) > define PYTHON_FINALIZE_TARGET > - find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) -name '*.py' -print0 | xargs -0 rm -f > + find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) \( -name '*.py' -or -name '*.pyo' \) -print0 | xargs -0 rm -f > endef > endif > > ifeq ($(BR2_PACKAGE_PYTHON_PY_ONLY),y) > define PYTHON_FINALIZE_TARGET > + find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) \( -name '*.pyc' -or -name '*.pyo' \) -print0 | xargs -0 rm -f > +endef > +endif > + > +ifeq ($(BR2_PACKAGE_PYTHON_PY_PYC),y) > +define PYTHON_FINALIZE_TARGET > + find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) -name '*.pyo' -print0 | xargs -0 rm -f > +endef > +endif > + > +ifeq ($(BR2_PACKAGE_PYTHON_PY_PYO),y) > +define PYTHON_FINALIZE_TARGET > find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) -name '*.pyc' -print0 | xargs -0 rm -f > endef > endif > > +ifeq ($(BR2_PACKAGE_PYTHON_PYC_PYO),y) > +define PYTHON_FINALIZE_TARGET > + find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) -name '*.py' -print0 | xargs -0 rm -f > +endef > +endif I'm sure we can do something smarter here. # arg1: extension of files to remove define PYTHON_FINALIZE_REMOVE_HELPER find $(TARGET_DIR)/usr/lib/python$(PYTHON_VERSION_MAJOR) -name '*.$(1)' -print0 | xargs -0 rm -f endef ifeq ($(BR2_PACKAGE_PYTHON_REMOVE_PY),y) $(call PYTHON_FINALIZE_REMOVE_HELPER,py) endif ifeq ($(BR2_PACKAGE_PYTHON_REMOVE_PYC),y) $(call PYTHON_FINALIZE_REMOVE_HELPER,pyc) endif ifeq ($(BR2_PACKAGE_PYTHON_REMOVE_PYO),y) $(call PYTHON_FINALIZE_REMOVE_HELPER,pyo) endif BR2_PACKAGE_PYTHON_REMOVE_* being hidden Config.in options. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com