* [Buildroot] [PATCH] python: fix to ensure libpython is stripped
@ 2014-01-23 3:40 Przemyslaw Wrzos
2014-01-23 14:06 ` Thomas De Schampheleire
2014-02-13 21:08 ` Thomas Petazzoni
0 siblings, 2 replies; 7+ messages in thread
From: Przemyslaw Wrzos @ 2014-01-23 3:40 UTC (permalink / raw)
To: buildroot
The python and python3 builds mark libpython as read-only which
prevents it from being stripped out correctly for the target.
Signed-off-by: Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
---
package/python/python.mk | 9 +++++++++
package/python3/python3.mk | 9 +++++++++
2 files changed, 18 insertions(+)
NOTE: Resubmitting as part of patchwork cleanup #6.
diff --git a/package/python/python.mk b/package/python/python.mk
index 6a6aaac..69662c3 100644
--- a/package/python/python.mk
+++ b/package/python/python.mk
@@ -162,6 +162,15 @@ endef
PYTHON_POST_INSTALL_TARGET_HOOKS += PYTHON_REMOVE_USELESS_FILES
+#
+# Make sure libpython gets stripped out on target
+#
+define PYTHON_ENSURE_LIBPYTHON_STRIPPED
+ chmod u+w $(TARGET_DIR)/usr/lib/libpython$(PYTHON_VERSION_MAJOR)*.so
+endef
+
+PYTHON_POST_INSTALL_TARGET_HOOKS += PYTHON_ENSURE_LIBPYTHON_STRIPPED
+
PYTHON_AUTORECONF = YES
$(eval $(autotools-package))
diff --git a/package/python3/python3.mk b/package/python3/python3.mk
index b5e9689..6cbe772 100644
--- a/package/python3/python3.mk
+++ b/package/python3/python3.mk
@@ -149,6 +149,15 @@ endef
PYTHON3_POST_INSTALL_TARGET_HOOKS += PYTHON3_REMOVE_USELESS_FILES
+#
+# Make sure libpython gets stripped out on target
+#
+define PYTHON3_ENSURE_LIBPYTHON_STRIPPED
+ chmod u+w $(TARGET_DIR)/usr/lib/libpython$(PYTHON3_VERSION_MAJOR)*.so
+endef
+
+PYTHON3_POST_INSTALL_TARGET_HOOKS += PYTHON3_ENSURE_LIBPYTHON_STRIPPED
+
PYTHON3_AUTORECONF = YES
define PYTHON3_INSTALL_SYMLINK
--
1.8.3.2
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] python: fix to ensure libpython is stripped
2014-01-23 3:40 [Buildroot] [PATCH] python: fix to ensure libpython is stripped Przemyslaw Wrzos
@ 2014-01-23 14:06 ` Thomas De Schampheleire
2014-01-23 23:53 ` Przemyslaw Wrzos
2014-01-27 20:05 ` Thomas De Schampheleire
2014-02-13 21:08 ` Thomas Petazzoni
1 sibling, 2 replies; 7+ messages in thread
From: Thomas De Schampheleire @ 2014-01-23 14:06 UTC (permalink / raw)
To: buildroot
On Thu, Jan 23, 2014 at 4:40 AM, Przemyslaw Wrzos
<przemyslaw.wrzos@calyptech.com> wrote:
> The python and python3 builds mark libpython as read-only which
> prevents it from being stripped out correctly for the target.
>
> Signed-off-by: Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
> ---
> package/python/python.mk | 9 +++++++++
> package/python3/python3.mk | 9 +++++++++
> 2 files changed, 18 insertions(+)
>
> NOTE: Resubmitting as part of patchwork cleanup #6.
>
> diff --git a/package/python/python.mk b/package/python/python.mk
> index 6a6aaac..69662c3 100644
> --- a/package/python/python.mk
> +++ b/package/python/python.mk
> @@ -162,6 +162,15 @@ endef
>
> PYTHON_POST_INSTALL_TARGET_HOOKS += PYTHON_REMOVE_USELESS_FILES
>
> +#
> +# Make sure libpython gets stripped out on target
> +#
> +define PYTHON_ENSURE_LIBPYTHON_STRIPPED
> + chmod u+w $(TARGET_DIR)/usr/lib/libpython$(PYTHON_VERSION_MAJOR)*.so
> +endef
> +
> +PYTHON_POST_INSTALL_TARGET_HOOKS += PYTHON_ENSURE_LIBPYTHON_STRIPPED
> +
> PYTHON_AUTORECONF = YES
>
> $(eval $(autotools-package))
> diff --git a/package/python3/python3.mk b/package/python3/python3.mk
> index b5e9689..6cbe772 100644
> --- a/package/python3/python3.mk
> +++ b/package/python3/python3.mk
> @@ -149,6 +149,15 @@ endef
>
> PYTHON3_POST_INSTALL_TARGET_HOOKS += PYTHON3_REMOVE_USELESS_FILES
>
> +#
> +# Make sure libpython gets stripped out on target
> +#
> +define PYTHON3_ENSURE_LIBPYTHON_STRIPPED
> + chmod u+w $(TARGET_DIR)/usr/lib/libpython$(PYTHON3_VERSION_MAJOR)*.so
> +endef
> +
> +PYTHON3_POST_INSTALL_TARGET_HOOKS += PYTHON3_ENSURE_LIBPYTHON_STRIPPED
> +
> PYTHON3_AUTORECONF = YES
>
> define PYTHON3_INSTALL_SYMLINK
> --
Acked-by: Thomas De Schampheleire <thomas.de_schampheleire@alcatel-lucent.com>
Tested-by: Thomas De Schampheleire <thomas.de_schampheleire@alcatel-lucent.com>
(tested with python2 only)
Note: the library will remain writable, but this doesn't look like a
big problem to me...
By the way: nice find! Saves about 4MB on Python 2!
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] python: fix to ensure libpython is stripped
2014-01-23 14:06 ` Thomas De Schampheleire
@ 2014-01-23 23:53 ` Przemyslaw Wrzos
2014-01-27 20:05 ` Thomas De Schampheleire
1 sibling, 0 replies; 7+ messages in thread
From: Przemyslaw Wrzos @ 2014-01-23 23:53 UTC (permalink / raw)
To: buildroot
Hello Thomas,
On Fri, 24 Jan 2014 at 01:06:08, Thomas De Schampheleire wrote:
> On Thu, Jan 23, 2014 at 4:40 AM, Przemyslaw Wrzos
> <przemyslaw.wrzos@calyptech.com> wrote:
>> The python and python3 builds mark libpython as read-only which
>> prevents it from being stripped out correctly for the target.
>>
>> Signed-off-by: Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
>> ---
>
> Acked-by: Thomas De Schampheleire
> <thomas.de_schampheleire@alcatel-lucent.com> Tested-by: Thomas De
> Schampheleire <thomas.de_schampheleire@alcatel-lucent.com> (tested with
> python2 only)
>
> Note: the library will remain writable, but this doesn't look like a
> big problem to me...
>
> By the way: nice find! Saves about 4MB on Python 2!
Yep, that's the only caveat without stuffing some horrible hack into
buildroot elsewhere to get around it. I couldn't think of a downside
though, and I've been running this on my embedded systems for a year now
without issue.
Cheers,
Przemyslaw Wrzos
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] python: fix to ensure libpython is stripped
2014-01-23 14:06 ` Thomas De Schampheleire
2014-01-23 23:53 ` Przemyslaw Wrzos
@ 2014-01-27 20:05 ` Thomas De Schampheleire
1 sibling, 0 replies; 7+ messages in thread
From: Thomas De Schampheleire @ 2014-01-27 20:05 UTC (permalink / raw)
To: buildroot
On Thu, Jan 23, 2014 at 3:06 PM, Thomas De Schampheleire
<patrickdepinguin@gmail.com> wrote:
> On Thu, Jan 23, 2014 at 4:40 AM, Przemyslaw Wrzos
> <przemyslaw.wrzos@calyptech.com> wrote:
>> The python and python3 builds mark libpython as read-only which
>> prevents it from being stripped out correctly for the target.
>>
>> Signed-off-by: Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
>> ---
>> package/python/python.mk | 9 +++++++++
>> package/python3/python3.mk | 9 +++++++++
>> 2 files changed, 18 insertions(+)
>>
>> NOTE: Resubmitting as part of patchwork cleanup #6.
>>
>> diff --git a/package/python/python.mk b/package/python/python.mk
>> index 6a6aaac..69662c3 100644
>> --- a/package/python/python.mk
>> +++ b/package/python/python.mk
>> @@ -162,6 +162,15 @@ endef
>>
>> PYTHON_POST_INSTALL_TARGET_HOOKS += PYTHON_REMOVE_USELESS_FILES
>>
>> +#
>> +# Make sure libpython gets stripped out on target
>> +#
>> +define PYTHON_ENSURE_LIBPYTHON_STRIPPED
>> + chmod u+w $(TARGET_DIR)/usr/lib/libpython$(PYTHON_VERSION_MAJOR)*.so
>> +endef
>> +
>> +PYTHON_POST_INSTALL_TARGET_HOOKS += PYTHON_ENSURE_LIBPYTHON_STRIPPED
>> +
>> PYTHON_AUTORECONF = YES
>>
>> $(eval $(autotools-package))
>> diff --git a/package/python3/python3.mk b/package/python3/python3.mk
>> index b5e9689..6cbe772 100644
>> --- a/package/python3/python3.mk
>> +++ b/package/python3/python3.mk
>> @@ -149,6 +149,15 @@ endef
>>
>> PYTHON3_POST_INSTALL_TARGET_HOOKS += PYTHON3_REMOVE_USELESS_FILES
>>
>> +#
>> +# Make sure libpython gets stripped out on target
>> +#
>> +define PYTHON3_ENSURE_LIBPYTHON_STRIPPED
>> + chmod u+w $(TARGET_DIR)/usr/lib/libpython$(PYTHON3_VERSION_MAJOR)*.so
>> +endef
>> +
>> +PYTHON3_POST_INSTALL_TARGET_HOOKS += PYTHON3_ENSURE_LIBPYTHON_STRIPPED
>> +
>> PYTHON3_AUTORECONF = YES
>>
>> define PYTHON3_INSTALL_SYMLINK
>> --
>
> Acked-by: Thomas De Schampheleire <thomas.de_schampheleire@alcatel-lucent.com>
> Tested-by: Thomas De Schampheleire <thomas.de_schampheleire@alcatel-lucent.com>
> (tested with python2 only)
>
> Note: the library will remain writable, but this doesn't look like a
> big problem to me...
>
> By the way: nice find! Saves about 4MB on Python 2!
ping on this patch?
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] python: fix to ensure libpython is stripped
2014-01-23 3:40 [Buildroot] [PATCH] python: fix to ensure libpython is stripped Przemyslaw Wrzos
2014-01-23 14:06 ` Thomas De Schampheleire
@ 2014-02-13 21:08 ` Thomas Petazzoni
2014-02-14 0:22 ` Przemyslaw Wrzos
1 sibling, 1 reply; 7+ messages in thread
From: Thomas Petazzoni @ 2014-02-13 21:08 UTC (permalink / raw)
To: buildroot
Dear Przemyslaw Wrzos,
On Thu, 23 Jan 2014 14:40:38 +1100, Przemyslaw Wrzos wrote:
> The python and python3 builds mark libpython as read-only which
> prevents it from being stripped out correctly for the target.
>
> Signed-off-by: Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
> ---
> package/python/python.mk | 9 +++++++++
> package/python3/python3.mk | 9 +++++++++
> 2 files changed, 18 insertions(+)
>
> NOTE: Resubmitting as part of patchwork cleanup #6.
I have integrated this patch into the large set of Python patches I've
posted at
http://lists.busybox.net/pipermail/buildroot/2014-February/089309.html.
However, I agree with Thomas De Schampheleire that this specific patch
should be merged for 2014.02, as it fixes a real problem, and gives a
nice size reduction.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] python: fix to ensure libpython is stripped
2014-02-13 21:08 ` Thomas Petazzoni
@ 2014-02-14 0:22 ` Przemyslaw Wrzos
2014-02-14 8:17 ` Thomas Petazzoni
0 siblings, 1 reply; 7+ messages in thread
From: Przemyslaw Wrzos @ 2014-02-14 0:22 UTC (permalink / raw)
To: buildroot
Hello Thomas,
On Fri, 14 Feb 2014 at 08:08:54, Thomas Petazzoni wrote:
> Dear Przemyslaw Wrzos,
>
> On Thu, 23 Jan 2014 14:40:38 +1100, Przemyslaw Wrzos wrote:
>> The python and python3 builds mark libpython as read-only which
>> prevents it from being stripped out correctly for the target.
>>
>> Signed-off-by: Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
>
> I have integrated this patch into the large set of Python patches I've
> posted at
> http://lists.busybox.net/pipermail/buildroot/2014-February/089309.html.
>
> However, I agree with Thomas De Schampheleire that this specific patch
> should be merged for 2014.02, as it fixes a real problem, and gives a
> nice size reduction.
>
> Thomas
That makes sense. I assume you don't need anything from me at this point.
Cheers,
Przemyslaw Wrzos
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [PATCH] python: fix to ensure libpython is stripped
2014-02-14 0:22 ` Przemyslaw Wrzos
@ 2014-02-14 8:17 ` Thomas Petazzoni
0 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2014-02-14 8:17 UTC (permalink / raw)
To: buildroot
Dear Przemyslaw Wrzos,
On Fri, 14 Feb 2014 11:22:18 +1100, Przemyslaw Wrzos wrote:
> > On Thu, 23 Jan 2014 14:40:38 +1100, Przemyslaw Wrzos wrote:
> >> The python and python3 builds mark libpython as read-only which
> >> prevents it from being stripped out correctly for the target.
> >>
> >> Signed-off-by: Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
> >
> > I have integrated this patch into the large set of Python patches I've
> > posted at
> > http://lists.busybox.net/pipermail/buildroot/2014-February/089309.html.
> >
> > However, I agree with Thomas De Schampheleire that this specific patch
> > should be merged for 2014.02, as it fixes a real problem, and gives a
> > nice size reduction.
> >
> > Thomas
>
> That makes sense. I assume you don't need anything from me at this point.
Nope, at this point your patch is in the queue, it should be handled at
some point :-)
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2014-02-14 8:17 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-01-23 3:40 [Buildroot] [PATCH] python: fix to ensure libpython is stripped Przemyslaw Wrzos
2014-01-23 14:06 ` Thomas De Schampheleire
2014-01-23 23:53 ` Przemyslaw Wrzos
2014-01-27 20:05 ` Thomas De Schampheleire
2014-02-13 21:08 ` Thomas Petazzoni
2014-02-14 0:22 ` Przemyslaw Wrzos
2014-02-14 8:17 ` Thomas Petazzoni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.