* [Buildroot] [PATCH 1/4] TCL: change BR2_PACKAGE_TCL_SHLIB_ONLY option
2013-06-20 15:53 [Buildroot] [PATCH 0/4] make usb_modeswitch work with udev Richard Genoud
@ 2013-06-20 15:53 ` Richard Genoud
2013-06-20 15:59 ` Thomas Petazzoni
2013-06-20 15:53 ` [Buildroot] [PATCH 2/4] tcl: add a tclsh symlink Richard Genoud
` (5 subsequent siblings)
6 siblings, 1 reply; 25+ messages in thread
From: Richard Genoud @ 2013-06-20 15:53 UTC (permalink / raw)
To: buildroot
It's not convenient to have an option (defaulted to yes) that removes a
software.
For instance, usb_modeswitching_data needs the tclsh interpreter, so it
would have to un-select this option, but select TCL.
Having an option that adds the tclsh binary (defaulted to no to keep the
same behaviour) is way more convenient.
Moreover, it seems that it was intended liked that at the begining
because usb_modeswitching_data already selects BR2_PACKAGE_TCL_TCLSH
which wasn't declared anywhere
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/Config.in | 11 +++++------
package/tcl/tcl.mk | 2 +-
2 files changed, 6 insertions(+), 7 deletions(-)
diff --git a/package/tcl/Config.in b/package/tcl/Config.in
index 7a4d887..8af7980 100644
--- a/package/tcl/Config.in
+++ b/package/tcl/Config.in
@@ -16,12 +16,11 @@ config BR2_PACKAGE_TCL_DEL_ENCODINGS
It saves approx. 1.4 Mb of space.
-config BR2_PACKAGE_TCL_SHLIB_ONLY
- bool "install only shared library"
- default y
+config BR2_PACKAGE_TCL_TCLSH
+ bool "Install also tclsh binary, not only the shared library"
+ default n
depends on BR2_PACKAGE_TCL
help
- Install only TCL shared library and not binary tcl
- interpreter(tclsh8.4).
+ Install the TCL interpreter binary file(tclsh8.4).
- Saves ~14kb.
+ Adds ~14kb.
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 144fefe..1c23420 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -26,7 +26,7 @@ define TCL_POST_INSTALL_CLEANUP
-if [ "$(BR2_PACKAGE_TCL_DEL_ENCODINGS)" = "y" ]; then \
rm -Rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*; \
fi
- -if [ "$(BR2_PACKAGE_TCL_SHLIB_ONLY)" = "y" ]; then \
+ -if [ "$(BR2_PACKAGE_TCL_TCLSH)" != "y" ]; then \
rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR); \
fi
endef
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 1/4] TCL: change BR2_PACKAGE_TCL_SHLIB_ONLY option
2013-06-20 15:53 ` [Buildroot] [PATCH 1/4] TCL: change BR2_PACKAGE_TCL_SHLIB_ONLY option Richard Genoud
@ 2013-06-20 15:59 ` Thomas Petazzoni
0 siblings, 0 replies; 25+ messages in thread
From: Thomas Petazzoni @ 2013-06-20 15:59 UTC (permalink / raw)
To: buildroot
Dear Richard Genoud,
On Thu, 20 Jun 2013 17:53:27 +0200, Richard Genoud wrote:
> @@ -26,7 +26,7 @@ define TCL_POST_INSTALL_CLEANUP
> -if [ "$(BR2_PACKAGE_TCL_DEL_ENCODINGS)" = "y" ]; then \
> rm -Rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*; \
> fi
> - -if [ "$(BR2_PACKAGE_TCL_SHLIB_ONLY)" = "y" ]; then \
> + -if [ "$(BR2_PACKAGE_TCL_TCLSH)" != "y" ]; then \
> rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR); \
> fi
> endef
Whenever possible, I think we prefer to use make conditional rather
than shell conditions. So something like:
ifeq ($(BR2_PACKAGE_TCL_DEL_ENCODINGS),y)
define TCL_REMOVE_ENCODINGS
rm -rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*
endef
TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_ENCODINGS
endif
ifeq ($(BR2_PACKAGE_TCL_TCLSH),y)
define TCL_SYMLINK_TCLSH
ln -s tclsh$(TCL_VERSION_MAJOR) $(TARGET_DIR)/usr/bin/tclsh
endef
TCL_POST_INSTALL_TARGET_HOOKS += TCL_SYMLINK_TCLSH
else
define TCL_REMOVE_TCLSH
rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR)
endef
TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_TCLSH
endif
And while you're at it, you could also remove the useless (and
incorrect) :
-$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libtcl8.4.so
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 2/4] tcl: add a tclsh symlink
2013-06-20 15:53 [Buildroot] [PATCH 0/4] make usb_modeswitch work with udev Richard Genoud
2013-06-20 15:53 ` [Buildroot] [PATCH 1/4] TCL: change BR2_PACKAGE_TCL_SHLIB_ONLY option Richard Genoud
@ 2013-06-20 15:53 ` Richard Genoud
2013-06-20 15:53 ` [Buildroot] [PATCH 3/4] usb_modeswitch: bump to 1.2.6 Richard Genoud
` (4 subsequent siblings)
6 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-20 15:53 UTC (permalink / raw)
To: buildroot
tclsh is installed with its version number (tclshx.y) but scripts (like
usb_modeswitch_dispatcher) are calling tclsh, not tclshx.y.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 1c23420..d81ad50 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -28,6 +28,8 @@ define TCL_POST_INSTALL_CLEANUP
fi
-if [ "$(BR2_PACKAGE_TCL_TCLSH)" != "y" ]; then \
rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR); \
+ else \
+ ln -s tclsh$(TCL_VERSION_MAJOR) $(TARGET_DIR)/usr/bin/tclsh ; \
fi
endef
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 3/4] usb_modeswitch: bump to 1.2.6
2013-06-20 15:53 [Buildroot] [PATCH 0/4] make usb_modeswitch work with udev Richard Genoud
2013-06-20 15:53 ` [Buildroot] [PATCH 1/4] TCL: change BR2_PACKAGE_TCL_SHLIB_ONLY option Richard Genoud
2013-06-20 15:53 ` [Buildroot] [PATCH 2/4] tcl: add a tclsh symlink Richard Genoud
@ 2013-06-20 15:53 ` Richard Genoud
2013-06-20 16:00 ` Thomas Petazzoni
2013-06-20 15:53 ` [Buildroot] [PATCH 4/4] usb_modeswitch_data: bump to 20130610 Richard Genoud
` (3 subsequent siblings)
6 siblings, 1 reply; 25+ messages in thread
From: Richard Genoud @ 2013-06-20 15:53 UTC (permalink / raw)
To: buildroot
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
.../usb_modeswitch/usb_modeswitch-makefile.patch | 31 --------------------
package/usb_modeswitch/usb_modeswitch.mk | 2 +-
2 files changed, 1 insertion(+), 32 deletions(-)
delete mode 100644 package/usb_modeswitch/usb_modeswitch-makefile.patch
diff --git a/package/usb_modeswitch/usb_modeswitch-makefile.patch b/package/usb_modeswitch/usb_modeswitch-makefile.patch
deleted file mode 100644
index b792741..0000000
--- a/package/usb_modeswitch/usb_modeswitch-makefile.patch
+++ /dev/null
@@ -1,31 +0,0 @@
-Removed the -s from the install command. Without this, usb_modeswitch would
-fail to install.
-
-Signed-off-by: J.C. Woltz <jwoltz@gmail.com>
-
-Index: b/Makefile
-===================================================================
---- a/Makefile
-+++ b/Makefile
-@@ -46,7 +46,7 @@
- cd jim && $(MAKE) distclean
-
- install-common: all
-- install -D -s --mode=755 usb_modeswitch $(SBINDIR)/usb_modeswitch
-+ install -D --mode=755 usb_modeswitch $(SBINDIR)/usb_modeswitch
- install -D --mode=755 usb_modeswitch.sh $(UDEVDIR)/usb_modeswitch
- install -D --mode=644 usb_modeswitch.conf $(ETCDIR)/usb_modeswitch.conf
- install -D --mode=644 usb_modeswitch.1 $(MANDIR)/usb_modeswitch.1
-@@ -67,10 +67,10 @@
- install: install-common install-script
-
- install-shared: dispatcher-dynamic install-common
-- install -D -s --mode=755 usb_modeswitch_dispatcher $(SBINDIR)/usb_modeswitch_dispatcher
-+ install -D --mode=755 usb_modeswitch_dispatcher $(SBINDIR)/usb_modeswitch_dispatcher
-
- install-static: dispatcher-static install-common
-- install -D -s --mode=755 usb_modeswitch_dispatcher $(SBINDIR)/usb_modeswitch_dispatcher
-+ install -D --mode=755 usb_modeswitch_dispatcher $(SBINDIR)/usb_modeswitch_dispatcher
-
- uninstall:
- $(RM) $(SBINDIR)/usb_modeswitch
diff --git a/package/usb_modeswitch/usb_modeswitch.mk b/package/usb_modeswitch/usb_modeswitch.mk
index 653109d..35f90bb 100644
--- a/package/usb_modeswitch/usb_modeswitch.mk
+++ b/package/usb_modeswitch/usb_modeswitch.mk
@@ -4,7 +4,7 @@
#
#############################################################
-USB_MODESWITCH_VERSION = 1.2.5
+USB_MODESWITCH_VERSION = 1.2.6
USB_MODESWITCH_SOURCE = usb-modeswitch-$(USB_MODESWITCH_VERSION).tar.bz2
USB_MODESWITCH_SITE = http://www.draisberghof.de/usb_modeswitch
USB_MODESWITCH_DEPENDENCIES = libusb-compat
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 3/4] usb_modeswitch: bump to 1.2.6
2013-06-20 15:53 ` [Buildroot] [PATCH 3/4] usb_modeswitch: bump to 1.2.6 Richard Genoud
@ 2013-06-20 16:00 ` Thomas Petazzoni
2013-06-20 17:02 ` Richard Genoud
0 siblings, 1 reply; 25+ messages in thread
From: Thomas Petazzoni @ 2013-06-20 16:00 UTC (permalink / raw)
To: buildroot
Dear Richard Genoud,
On Thu, 20 Jun 2013 17:53:29 +0200, Richard Genoud wrote:
> Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
> ---
> .../usb_modeswitch/usb_modeswitch-makefile.patch | 31 --------------------
> package/usb_modeswitch/usb_modeswitch.mk | 2 +-
> 2 files changed, 1 insertion(+), 32 deletions(-)
> delete mode 100644 package/usb_modeswitch/usb_modeswitch-makefile.patch
When bumping to a new version leads to the removal of some patches, it
is always good to mention /why/ they are removed (they got upstreamed,
they are no longer needed because..).
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 3/4] usb_modeswitch: bump to 1.2.6
2013-06-20 16:00 ` Thomas Petazzoni
@ 2013-06-20 17:02 ` Richard Genoud
2013-06-20 17:04 ` Gustavo Zacarias
0 siblings, 1 reply; 25+ messages in thread
From: Richard Genoud @ 2013-06-20 17:02 UTC (permalink / raw)
To: buildroot
2013/6/20 Thomas Petazzoni <thomas.petazzoni@free-electrons.com>:
> Dear Richard Genoud,
>
> On Thu, 20 Jun 2013 17:53:29 +0200, Richard Genoud wrote:
>> Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
>> ---
>> .../usb_modeswitch/usb_modeswitch-makefile.patch | 31 --------------------
>> package/usb_modeswitch/usb_modeswitch.mk | 2 +-
>> 2 files changed, 1 insertion(+), 32 deletions(-)
>> delete mode 100644 package/usb_modeswitch/usb_modeswitch-makefile.patch
>
> When bumping to a new version leads to the removal of some patches, it
> is always good to mention /why/ they are removed (they got upstreamed,
> they are no longer needed because..).
You're right, I've been a little lazy on this !
I'll resend the whole set tomorrow.
Thanks,
Richard.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 3/4] usb_modeswitch: bump to 1.2.6
2013-06-20 17:02 ` Richard Genoud
@ 2013-06-20 17:04 ` Gustavo Zacarias
0 siblings, 0 replies; 25+ messages in thread
From: Gustavo Zacarias @ 2013-06-20 17:04 UTC (permalink / raw)
To: buildroot
On 06/20/2013 02:02 PM, Richard Genoud wrote:
> 2013/6/20 Thomas Petazzoni <thomas.petazzoni@free-electrons.com>:
>> Dear Richard Genoud,
>>
>> On Thu, 20 Jun 2013 17:53:29 +0200, Richard Genoud wrote:
>>> Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
>>> ---
>>> .../usb_modeswitch/usb_modeswitch-makefile.patch | 31 --------------------
>>> package/usb_modeswitch/usb_modeswitch.mk | 2 +-
>>> 2 files changed, 1 insertion(+), 32 deletions(-)
>>> delete mode 100644 package/usb_modeswitch/usb_modeswitch-makefile.patch
>>
>> When bumping to a new version leads to the removal of some patches, it
>> is always good to mention /why/ they are removed (they got upstreamed,
>> they are no longer needed because..).
>
> You're right, I've been a little lazy on this !
> I'll resend the whole set tomorrow.
Never mind about 3-4/4 since it's been already done by Patrick Ziegler
and committed already :)
Regards.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 4/4] usb_modeswitch_data: bump to 20130610
2013-06-20 15:53 [Buildroot] [PATCH 0/4] make usb_modeswitch work with udev Richard Genoud
` (2 preceding siblings ...)
2013-06-20 15:53 ` [Buildroot] [PATCH 3/4] usb_modeswitch: bump to 1.2.6 Richard Genoud
@ 2013-06-20 15:53 ` Richard Genoud
2013-06-21 11:56 ` [Buildroot] [PATCH 0/6] tcl rework (was: make usb_modeswitch work with udev) richard.genoud at gmail.com
` (2 subsequent siblings)
6 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-20 15:53 UTC (permalink / raw)
To: buildroot
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
.../usb_modeswitch_data-fix-parallel-install.patch | 6 +++---
.../usb_modeswitch_data/usb_modeswitch_data-makefile.patch | 12 ++++++------
package/usb_modeswitch_data/usb_modeswitch_data.mk | 2 +-
3 files changed, 10 insertions(+), 10 deletions(-)
diff --git a/package/usb_modeswitch_data/usb_modeswitch_data-fix-parallel-install.patch b/package/usb_modeswitch_data/usb_modeswitch_data-fix-parallel-install.patch
index ba30499..9319d0d 100644
--- a/package/usb_modeswitch_data/usb_modeswitch_data-fix-parallel-install.patch
+++ b/package/usb_modeswitch_data/usb_modeswitch_data-fix-parallel-install.patch
@@ -7,10 +7,10 @@ Index: b/Makefile
===================================================================
--- a/Makefile
+++ b/Makefile
-@@ -22,6 +22,7 @@
- install -D --mode=644 40-usb_modeswitch.rules $(RULESDIR)/40-usb_modeswitch.rules
+@@ -26,6 +26,7 @@
+ ./gen-rules.tcl
- db-install:
+ db-install: files-install
+ install -d $(PREFIX)/share/usb_modeswitch
install --mode=644 -t $(PREFIX)/share/usb_modeswitch ./usb_modeswitch.d/*
diff --git a/package/usb_modeswitch_data/usb_modeswitch_data-makefile.patch b/package/usb_modeswitch_data/usb_modeswitch_data-makefile.patch
index 769e514..5632abd 100644
--- a/package/usb_modeswitch_data/usb_modeswitch_data-makefile.patch
+++ b/package/usb_modeswitch_data/usb_modeswitch_data-makefile.patch
@@ -4,19 +4,19 @@ cross-compiler environment, it is not wanted to reload the host udev rules.
Signed-off-by: J.C. Woltz <jwoltz@gmail.com>
--- usb_modeswitch_data-20120120.orig/Makefile 2012-01-20 17:25:32.000000000 -0500
+++ usb_modeswitch_data-20120120/Makefile 2012-02-06 14:20:47.000000000 -0500
-@@ -12,9 +12,9 @@
-
+@@ -13,9 +13,9 @@
clean:
+ $(RM) 40-usb_modeswitch.rules
--install: files-install db-install rules-reload
-+install: files-install db-install
+-install: all files-install db-install rules-reload
++install: all files-install db-install
-install-packed: files-install db-install-packed rules-reload
+install-packed: files-install db-install-packed
- files-install:
+ files-install:
install -d $(PREFIX)/share/usb_modeswitch
-@@ -50,7 +50,7 @@
+@@ -54,7 +54,7 @@
fi \
fi
diff --git a/package/usb_modeswitch_data/usb_modeswitch_data.mk b/package/usb_modeswitch_data/usb_modeswitch_data.mk
index 479137c..26432c1 100644
--- a/package/usb_modeswitch_data/usb_modeswitch_data.mk
+++ b/package/usb_modeswitch_data/usb_modeswitch_data.mk
@@ -4,7 +4,7 @@
#
#############################################################
-USB_MODESWITCH_DATA_VERSION = 20121109
+USB_MODESWITCH_DATA_VERSION = 20130610
USB_MODESWITCH_DATA_SOURCE = usb-modeswitch-data-$(USB_MODESWITCH_DATA_VERSION).tar.bz2
USB_MODESWITCH_DATA_SITE = http://www.draisberghof.de/usb_modeswitch
USB_MODESWITCH_DATA_DEPENDENCIES = usb_modeswitch
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 0/6] tcl rework (was: make usb_modeswitch work with udev)
2013-06-20 15:53 [Buildroot] [PATCH 0/4] make usb_modeswitch work with udev Richard Genoud
` (3 preceding siblings ...)
2013-06-20 15:53 ` [Buildroot] [PATCH 4/4] usb_modeswitch_data: bump to 20130610 Richard Genoud
@ 2013-06-21 11:56 ` richard.genoud at gmail.com
[not found] ` <1371815822-29453-1-git-send-email-y>
2013-06-24 8:40 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Richard Genoud
6 siblings, 0 replies; 25+ messages in thread
From: richard.genoud at gmail.com @ 2013-06-21 11:56 UTC (permalink / raw)
To: buildroot
From: Richard Genoud <richard.genoud@gmail.com>
At first, the goal of this patch set was to add a symlink
tclsh->tclsh8.4 to make usb_modeswitch_dispatcher happy (it has a
shebang #!/usr/bin/tclsh).
While doing that, I saw some other problems (libtcl not stripped and
not into staging). Also, Thomas suggested (and coded it actually !) that
make condition should be used.
Hence, a bigger patchset.
Changes since v1:
- drop usb_modeswitch{,_data} version bumps patches as they are
already mainlined (thanks Gustavo for the heads up !)
- use make conditions rather than shell conditions
- drop the explicit strip
- install the library into stating
- make the library writable
Richard Genoud (6):
tcl: change BR2_PACKAGE_TCL_SHLIB_ONLY option
tcl: use make conditions
tcl: remove strip command
tcl: install the libtcl into staging
tcl: add a tclsh symlink
tcl: make the libtcl writable in order to be stripped
package/tcl/Config.in | 11 +++++------
package/tcl/tcl.mk | 33 ++++++++++++++++++++++++---------
2 files changed, 29 insertions(+), 15 deletions(-)
--
1.7.10.4
^ permalink raw reply [flat|nested] 25+ messages in thread[parent not found: <1371815822-29453-1-git-send-email-y>]
* [Buildroot] [PATCH 1/6] tcl: change BR2_PACKAGE_TCL_SHLIB_ONLY option
[not found] ` <1371815822-29453-1-git-send-email-y>
@ 2013-06-21 11:56 ` richard.genoud at gmail.com
2013-06-22 9:15 ` Patrick Ziegler
2013-06-21 11:56 ` [Buildroot] [PATCH 2/6] tcl: use make conditions richard.genoud at gmail.com
` (4 subsequent siblings)
5 siblings, 1 reply; 25+ messages in thread
From: richard.genoud at gmail.com @ 2013-06-21 11:56 UTC (permalink / raw)
To: buildroot
From: Richard Genoud <richard.genoud@gmail.com>
It's not convenient to have an option (defaulted to yes) that removes a
software.
For instance, usb_modeswitching_data needs the tclsh interpreter, so it
would have to un-select this option, but select TCL.
Having an option that adds the tclsh binary (defaulted to no to keep the
same behaviour) is way more convenient.
Moreover, it seems that it was intended liked that at the begining
because usb_modeswitching_data already selects BR2_PACKAGE_TCL_TCLSH
which wasn't declared anywhere.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/Config.in | 11 +++++------
package/tcl/tcl.mk | 2 +-
2 files changed, 6 insertions(+), 7 deletions(-)
diff --git a/package/tcl/Config.in b/package/tcl/Config.in
index 7a4d887..8af7980 100644
--- a/package/tcl/Config.in
+++ b/package/tcl/Config.in
@@ -16,12 +16,11 @@ config BR2_PACKAGE_TCL_DEL_ENCODINGS
It saves approx. 1.4 Mb of space.
-config BR2_PACKAGE_TCL_SHLIB_ONLY
- bool "install only shared library"
- default y
+config BR2_PACKAGE_TCL_TCLSH
+ bool "Install also tclsh binary, not only the shared library"
+ default n
depends on BR2_PACKAGE_TCL
help
- Install only TCL shared library and not binary tcl
- interpreter(tclsh8.4).
+ Install the TCL interpreter binary file(tclsh8.4).
- Saves ~14kb.
+ Adds ~14kb.
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 0c8fa33..ad23b80 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -27,7 +27,7 @@ define TCL_POST_INSTALL_CLEANUP
-if [ "$(BR2_PACKAGE_TCL_DEL_ENCODINGS)" = "y" ]; then \
rm -Rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*; \
fi
- -if [ "$(BR2_PACKAGE_TCL_SHLIB_ONLY)" = "y" ]; then \
+ -if [ "$(BR2_PACKAGE_TCL_TCLSH)" != "y" ]; then \
rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR); \
fi
endef
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 1/6] tcl: change BR2_PACKAGE_TCL_SHLIB_ONLY option
2013-06-21 11:56 ` [Buildroot] [PATCH 1/6] tcl: change BR2_PACKAGE_TCL_SHLIB_ONLY option richard.genoud at gmail.com
@ 2013-06-22 9:15 ` Patrick Ziegler
2013-06-24 7:49 ` Richard Genoud
0 siblings, 1 reply; 25+ messages in thread
From: Patrick Ziegler @ 2013-06-22 9:15 UTC (permalink / raw)
To: buildroot
Dear Richard Genoud,
Am 21.06.2013 13:56, schrieb richard.genoud at gmail.com:
> From: Richard Genoud <richard.genoud@gmail.com>
>
> It's not convenient to have an option (defaulted to yes) that removes a
> software.
> For instance, usb_modeswitching_data needs the tclsh interpreter, so it
> would have to un-select this option, but select TCL.
>
> Having an option that adds the tclsh binary (defaulted to no to keep the
> same behaviour) is way more convenient.
> Moreover, it seems that it was intended liked that at the begining
> because usb_modeswitching_data already selects BR2_PACKAGE_TCL_TCLSH
> which wasn't declared anywhere.
>
> Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
If you change an existing option, please have a look on the
Config.in.legacy file. You should add the old option to that file, so
the user gets informed about the change.
Finally you should check if another package uses this option for its own
configuration and change this to the new option.
BR2_PACKAGE_TCL_SHLIB_ONLY is used by usb_modeswitch (not
usb_modeswitch_data anymore) to switch between the binary wrapper and
the tcl wrapper script.
Best regards
Patrick
--
Dipl.-Inf. (FH) Patrick Ziegler
University Of Applied Sciences
Kaiserslautern
Amerikastrasse 1
D-66482 Zweibruecken
Germany
Phone: +49 631 3724 5526
Mail: patrick.ziegler at fh-kl.de
http://www.fh-kl.de
http://www.fh-kl.de/fachbereiche/imst/iuk-knowhow.html
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 1/6] tcl: change BR2_PACKAGE_TCL_SHLIB_ONLY option
2013-06-22 9:15 ` Patrick Ziegler
@ 2013-06-24 7:49 ` Richard Genoud
0 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-24 7:49 UTC (permalink / raw)
To: buildroot
2013/6/22 Patrick Ziegler <patrick.ziegler@fh-kl.de>:
> Dear Richard Genoud,
>
> Am 21.06.2013 13:56, schrieb richard.genoud at gmail.com:
>> From: Richard Genoud <richard.genoud@gmail.com>
>>
>> It's not convenient to have an option (defaulted to yes) that removes a
>> software.
>> For instance, usb_modeswitching_data needs the tclsh interpreter, so it
>> would have to un-select this option, but select TCL.
>>
>> Having an option that adds the tclsh binary (defaulted to no to keep the
>> same behaviour) is way more convenient.
>> Moreover, it seems that it was intended liked that at the begining
>> because usb_modeswitching_data already selects BR2_PACKAGE_TCL_TCLSH
>> which wasn't declared anywhere.
>>
>> Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
>
> If you change an existing option, please have a look on the
> Config.in.legacy file. You should add the old option to that file, so
> the user gets informed about the change.
>
> Finally you should check if another package uses this option for its own
> configuration and change this to the new option.
> BR2_PACKAGE_TCL_SHLIB_ONLY is used by usb_modeswitch (not
> usb_modeswitch_data anymore) to switch between the binary wrapper and
> the tcl wrapper script.
>
I didn't saw your patch on that !
It seems that you already correct the problem with the tclsh interpreter.
So I can drop this patch.
Thanks !
Best regards,
Richard.
^ permalink raw reply [flat|nested] 25+ messages in thread
* [Buildroot] [PATCH 2/6] tcl: use make conditions
[not found] ` <1371815822-29453-1-git-send-email-y>
2013-06-21 11:56 ` [Buildroot] [PATCH 1/6] tcl: change BR2_PACKAGE_TCL_SHLIB_ONLY option richard.genoud at gmail.com
@ 2013-06-21 11:56 ` richard.genoud at gmail.com
2013-06-21 11:56 ` [Buildroot] [PATCH 3/6] tcl: remove strip command richard.genoud at gmail.com
` (3 subsequent siblings)
5 siblings, 0 replies; 25+ messages in thread
From: richard.genoud at gmail.com @ 2013-06-21 11:56 UTC (permalink / raw)
To: buildroot
From: Richard Genoud <richard.genoud@gmail.com>
As suggested by Thomas Petazzoni, make conditions should be used rather
than shell condition in Makefiles.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 21 ++++++++++++++-------
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index ad23b80..2526ca1 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -24,15 +24,22 @@ HOST_TCL_CONF_OPT = \
define TCL_POST_INSTALL_CLEANUP
-$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libtcl8.4.so
- -if [ "$(BR2_PACKAGE_TCL_DEL_ENCODINGS)" = "y" ]; then \
- rm -Rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*; \
- fi
- -if [ "$(BR2_PACKAGE_TCL_TCLSH)" != "y" ]; then \
- rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR); \
- fi
endef
-
TCL_POST_INSTALL_TARGET_HOOKS += TCL_POST_INSTALL_CLEANUP
+ifeq ($(BR2_PACKAGE_TCL_DEL_ENCODINGS),y)
+define TCL_REMOVE_ENCODINGS
+ rm -rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*
+endef
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_ENCODINGS
+endif
+
+ifneq ($(BR2_PACKAGE_TCL_TCLSH),y)
+define TCL_REMOVE_TCLSH
+ rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR)
+endef
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_TCLSH
+endif
+
$(eval $(autotools-package))
$(eval $(host-autotools-package))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 3/6] tcl: remove strip command
[not found] ` <1371815822-29453-1-git-send-email-y>
2013-06-21 11:56 ` [Buildroot] [PATCH 1/6] tcl: change BR2_PACKAGE_TCL_SHLIB_ONLY option richard.genoud at gmail.com
2013-06-21 11:56 ` [Buildroot] [PATCH 2/6] tcl: use make conditions richard.genoud at gmail.com
@ 2013-06-21 11:56 ` richard.genoud at gmail.com
2013-06-21 11:57 ` [Buildroot] [PATCH 4/6] tcl: install the libtcl into staging richard.genoud at gmail.com
` (2 subsequent siblings)
5 siblings, 0 replies; 25+ messages in thread
From: richard.genoud at gmail.com @ 2013-06-21 11:56 UTC (permalink / raw)
To: buildroot
From: Richard Genoud <richard.genoud@gmail.com>
Stripping is done at the end of the buildroot process, for all files at
once.
There's no need to do it here.
(Suggested by Thomas Petazzoni)
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 5 -----
1 file changed, 5 deletions(-)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 2526ca1..a927a02 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -22,11 +22,6 @@ HOST_TCL_CONF_OPT = \
--disable-langinfo \
--disable-framework
-define TCL_POST_INSTALL_CLEANUP
- -$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libtcl8.4.so
-endef
-TCL_POST_INSTALL_TARGET_HOOKS += TCL_POST_INSTALL_CLEANUP
-
ifeq ($(BR2_PACKAGE_TCL_DEL_ENCODINGS),y)
define TCL_REMOVE_ENCODINGS
rm -rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 4/6] tcl: install the libtcl into staging
[not found] ` <1371815822-29453-1-git-send-email-y>
` (2 preceding siblings ...)
2013-06-21 11:56 ` [Buildroot] [PATCH 3/6] tcl: remove strip command richard.genoud at gmail.com
@ 2013-06-21 11:57 ` richard.genoud at gmail.com
2013-06-21 11:57 ` [Buildroot] [PATCH 5/6] tcl: add a tclsh symlink richard.genoud at gmail.com
2013-06-21 11:57 ` [Buildroot] [PATCH 6/6] tcl: make the libtcl writable in order to be stripped richard.genoud at gmail.com
5 siblings, 0 replies; 25+ messages in thread
From: richard.genoud at gmail.com @ 2013-06-21 11:57 UTC (permalink / raw)
To: buildroot
From: Richard Genoud <richard.genoud@gmail.com>
The tcl package contains a lib, so it should go into staging.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index a927a02..90989d0 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -12,6 +12,7 @@ TCL_SITE = http://downloads.sourceforge.net/project/tcl/Tcl/$(TCL_VERSION_MAJOR)
TCL_LICENSE = tcl license
TCL_LICENSE_FILES = license.terms
TCL_SUBDIR = unix
+TCL_INSTALL_STAGING = YES
TCL_CONF_OPT = \
--disable-symbols \
--disable-langinfo \
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 5/6] tcl: add a tclsh symlink
[not found] ` <1371815822-29453-1-git-send-email-y>
` (3 preceding siblings ...)
2013-06-21 11:57 ` [Buildroot] [PATCH 4/6] tcl: install the libtcl into staging richard.genoud at gmail.com
@ 2013-06-21 11:57 ` richard.genoud at gmail.com
2013-06-21 11:57 ` [Buildroot] [PATCH 6/6] tcl: make the libtcl writable in order to be stripped richard.genoud at gmail.com
5 siblings, 0 replies; 25+ messages in thread
From: richard.genoud at gmail.com @ 2013-06-21 11:57 UTC (permalink / raw)
To: buildroot
From: Richard Genoud <richard.genoud@gmail.com>
tclsh is installed with its version number (tclshx.y) but scripts (like
usb_modeswitch_dispatcher) are calling tclsh, not tclshx.y.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 90989d0..b50daeb 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -30,7 +30,12 @@ endef
TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_ENCODINGS
endif
-ifneq ($(BR2_PACKAGE_TCL_TCLSH),y)
+ifeq ($(BR2_PACKAGE_TCL_TCLSH),y)
+define TCL_SYMLINK_TCLSH
+ ln -s tclsh$(TCL_VERSION_MAJOR) $(TARGET_DIR)/usr/bin/tclsh
+endef
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_SYMLINK_TCLSH
+else
define TCL_REMOVE_TCLSH
rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR)
endef
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCH 6/6] tcl: make the libtcl writable in order to be stripped
[not found] ` <1371815822-29453-1-git-send-email-y>
` (4 preceding siblings ...)
2013-06-21 11:57 ` [Buildroot] [PATCH 5/6] tcl: add a tclsh symlink richard.genoud at gmail.com
@ 2013-06-21 11:57 ` richard.genoud at gmail.com
5 siblings, 0 replies; 25+ messages in thread
From: richard.genoud at gmail.com @ 2013-06-21 11:57 UTC (permalink / raw)
To: buildroot
From: Richard Genoud <richard.genoud@gmail.com>
The libtcl is installed with 0555 and that prevents it from being
stripped.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index b50daeb..55d083a 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -42,5 +42,12 @@ endef
TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_TCLSH
endif
+# library get installed read only, so strip fails
+define TCL_FIXUP_RO_LIB
+ chmod +w $(TARGET_DIR)/usr/lib/libtcl*
+endef
+
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_FIXUP_RO_LIB
+
$(eval $(autotools-package))
$(eval $(host-autotools-package))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread
* [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev)
2013-06-20 15:53 [Buildroot] [PATCH 0/4] make usb_modeswitch work with udev Richard Genoud
` (5 preceding siblings ...)
[not found] ` <1371815822-29453-1-git-send-email-y>
@ 2013-06-24 8:40 ` Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 1/5] tcl: use make conditions Richard Genoud
` (5 more replies)
6 siblings, 6 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-24 8:40 UTC (permalink / raw)
To: buildroot
At first, the goal of this patch set was to add a symlink
tclsh->tclsh8.4 to make usb_modeswitch_dispatcher happy (it has a
shebang #!/usr/bin/tclsh).
While doing that, I saw some other problems (libtcl not stripped and
not into staging). Also, Thomas suggested (and coded it actually !) that
make condition should be used.
Hence, a bigger patchset.
Changes since v2:
- drop the change of BR2_PACKAGE_TCL_SHLIB_ONLY option as commit
d3e4db4 (usb_modeswitch: bump to version 1.2.6) already fixes
the problem.
Changes since v1:
- drop usb_modeswitch{,_data} version bumps patches as they are
already mainlined (thanks Gustavo for the heads up !)
- use make conditions rather than shell conditions
- drop the explicit strip
- install the library into stating
- make the library writable
Richard Genoud (5):
tcl: use make conditions
tcl: remove strip command
tcl: install the libtcl into staging
tcl: add a tclsh symlink
tcl: make the libtcl writable in order to be stripped
package/tcl/tcl.mk | 33 ++++++++++++++++++++++++---------
1 file changed, 24 insertions(+), 9 deletions(-)
--
1.7.10.4
^ permalink raw reply [flat|nested] 25+ messages in thread* [Buildroot] [PATCHv3 1/5] tcl: use make conditions
2013-06-24 8:40 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Richard Genoud
@ 2013-06-24 8:40 ` Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 2/5] tcl: remove strip command Richard Genoud
` (4 subsequent siblings)
5 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-24 8:40 UTC (permalink / raw)
To: buildroot
As suggested by Thomas Petazzoni, make conditions should be used rather
than shell condition in Makefiles.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 21 ++++++++++++++-------
1 file changed, 14 insertions(+), 7 deletions(-)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 0c8fa33..fc62abd 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -24,15 +24,22 @@ HOST_TCL_CONF_OPT = \
define TCL_POST_INSTALL_CLEANUP
-$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libtcl8.4.so
- -if [ "$(BR2_PACKAGE_TCL_DEL_ENCODINGS)" = "y" ]; then \
- rm -Rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*; \
- fi
- -if [ "$(BR2_PACKAGE_TCL_SHLIB_ONLY)" = "y" ]; then \
- rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR); \
- fi
endef
-
TCL_POST_INSTALL_TARGET_HOOKS += TCL_POST_INSTALL_CLEANUP
+ifeq ($(BR2_PACKAGE_TCL_DEL_ENCODINGS),y)
+define TCL_REMOVE_ENCODINGS
+ rm -rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*
+endef
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_ENCODINGS
+endif
+
+ifeq ($(BR2_PACKAGE_TCL_SHLIB_ONLY),y)
+define TCL_REMOVE_TCLSH
+ rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR)
+endef
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_TCLSH
+endif
+
$(eval $(autotools-package))
$(eval $(host-autotools-package))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCHv3 2/5] tcl: remove strip command
2013-06-24 8:40 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 1/5] tcl: use make conditions Richard Genoud
@ 2013-06-24 8:40 ` Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 3/5] tcl: install the libtcl into staging Richard Genoud
` (3 subsequent siblings)
5 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-24 8:40 UTC (permalink / raw)
To: buildroot
Stripping is done at the end of the buildroot process, for all files at
once.
There's no need to do it here.
(Suggested by Thomas Petazzoni)
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 5 -----
1 file changed, 5 deletions(-)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index fc62abd..dbdb40f 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -22,11 +22,6 @@ HOST_TCL_CONF_OPT = \
--disable-langinfo \
--disable-framework
-define TCL_POST_INSTALL_CLEANUP
- -$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(TARGET_DIR)/usr/lib/libtcl8.4.so
-endef
-TCL_POST_INSTALL_TARGET_HOOKS += TCL_POST_INSTALL_CLEANUP
-
ifeq ($(BR2_PACKAGE_TCL_DEL_ENCODINGS),y)
define TCL_REMOVE_ENCODINGS
rm -rf $(TARGET_DIR)/usr/lib/tcl$(TCL_VERSION_MAJOR)/encoding/*
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCHv3 3/5] tcl: install the libtcl into staging
2013-06-24 8:40 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 1/5] tcl: use make conditions Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 2/5] tcl: remove strip command Richard Genoud
@ 2013-06-24 8:40 ` Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 4/5] tcl: add a tclsh symlink Richard Genoud
` (2 subsequent siblings)
5 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-24 8:40 UTC (permalink / raw)
To: buildroot
The tcl package contains a lib, so it should go into staging.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index dbdb40f..83d74ec 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -12,6 +12,7 @@ TCL_SITE = http://downloads.sourceforge.net/project/tcl/Tcl/$(TCL_VERSION_MAJOR)
TCL_LICENSE = tcl license
TCL_LICENSE_FILES = license.terms
TCL_SUBDIR = unix
+TCL_INSTALL_STAGING = YES
TCL_CONF_OPT = \
--disable-symbols \
--disable-langinfo \
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCHv3 4/5] tcl: add a tclsh symlink
2013-06-24 8:40 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Richard Genoud
` (2 preceding siblings ...)
2013-06-24 8:40 ` [Buildroot] [PATCHv3 3/5] tcl: install the libtcl into staging Richard Genoud
@ 2013-06-24 8:40 ` Richard Genoud
2013-06-24 8:40 ` [Buildroot] [PATCHv3 5/5] tcl: make the libtcl writable in order to be stripped Richard Genoud
2013-07-28 14:29 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Thomas Petazzoni
5 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-24 8:40 UTC (permalink / raw)
To: buildroot
tclsh is installed with its version number (tclshx.y) but scripts (like
usb_modeswitch_dispatcher) are calling tclsh, not tclshx.y.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 83d74ec..3fd4047 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -35,6 +35,11 @@ define TCL_REMOVE_TCLSH
rm -f $(TARGET_DIR)/usr/bin/tclsh$(TCL_VERSION_MAJOR)
endef
TCL_POST_INSTALL_TARGET_HOOKS += TCL_REMOVE_TCLSH
+else
+define TCL_SYMLINK_TCLSH
+ ln -s tclsh$(TCL_VERSION_MAJOR) $(TARGET_DIR)/usr/bin/tclsh
+endef
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_SYMLINK_TCLSH
endif
$(eval $(autotools-package))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCHv3 5/5] tcl: make the libtcl writable in order to be stripped
2013-06-24 8:40 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Richard Genoud
` (3 preceding siblings ...)
2013-06-24 8:40 ` [Buildroot] [PATCHv3 4/5] tcl: add a tclsh symlink Richard Genoud
@ 2013-06-24 8:40 ` Richard Genoud
2013-07-28 14:29 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Thomas Petazzoni
5 siblings, 0 replies; 25+ messages in thread
From: Richard Genoud @ 2013-06-24 8:40 UTC (permalink / raw)
To: buildroot
The libtcl is installed with 0555 and that prevents it from being
stripped.
Signed-off-by: Richard Genoud <richard.genoud@gmail.com>
---
package/tcl/tcl.mk | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/package/tcl/tcl.mk b/package/tcl/tcl.mk
index 3fd4047..9f766e0 100644
--- a/package/tcl/tcl.mk
+++ b/package/tcl/tcl.mk
@@ -42,5 +42,12 @@ endef
TCL_POST_INSTALL_TARGET_HOOKS += TCL_SYMLINK_TCLSH
endif
+# library get installed read only, so strip fails
+define TCL_FIXUP_RO_LIB
+ chmod +w $(TARGET_DIR)/usr/lib/libtcl*
+endef
+
+TCL_POST_INSTALL_TARGET_HOOKS += TCL_FIXUP_RO_LIB
+
$(eval $(autotools-package))
$(eval $(host-autotools-package))
--
1.7.10.4
^ permalink raw reply related [flat|nested] 25+ messages in thread* [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev)
2013-06-24 8:40 ` [Buildroot] [PATCHv3 0/5] tcl rework (was: make usb_modeswitch work with udev) Richard Genoud
` (4 preceding siblings ...)
2013-06-24 8:40 ` [Buildroot] [PATCHv3 5/5] tcl: make the libtcl writable in order to be stripped Richard Genoud
@ 2013-07-28 14:29 ` Thomas Petazzoni
5 siblings, 0 replies; 25+ messages in thread
From: Thomas Petazzoni @ 2013-07-28 14:29 UTC (permalink / raw)
To: buildroot
Dear Richard Genoud,
On Mon, 24 Jun 2013 10:40:51 +0200, Richard Genoud wrote:
> Richard Genoud (5):
> tcl: use make conditions
> tcl: remove strip command
> tcl: install the libtcl into staging
> tcl: add a tclsh symlink
> tcl: make the libtcl writable in order to be stripped
>
> package/tcl/tcl.mk | 33 ++++++++++++++++++++++++---------
> 1 file changed, 24 insertions(+), 9 deletions(-)
Committed, thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 25+ messages in thread