Skip to contentSkip to conversation

package/gdb: don't remove support files if python chosen

PATCH New Jonas Zaddach 4 participants 4 messages Apr 10, 2018 – May 1, 2018

lore
Share & tools
lore

Jonas Zaddach

author
#0 lore raw

PATCH 1/1 package/gdb: don't remove support files if python chosen

If one wants to use GDB with python support on the target, you need the support files installed by GDB. These get usually deleted to save some space, so I just wrapped the Makefile code deleting them in a conditional block depending on if python support is active or not.

Signed-off-by Jonas Zaddach <[email protected]>
---
 package/gdb/gdb.mk | 2 ++
 1 file changed, 2 insertions(+)
package/gdb/gdb.mk +2 -0
index 7e86ba0..5c915da 100644
@@ -164,11 +164,13 @@ endif
 
 # This removes some unneeded Python scripts and XML target description
 # files that are not useful for a normal usage of the debugger.
+ifneq ($(BR2_PACKAGE_GDB_PYTHON),y)
 define GDB_REMOVE_UNNEEDED_FILES
 	$(RM) -rf $(TARGET_DIR)/usr/share/gdb
 endef
 
 GDB_POST_INSTALL_TARGET_HOOKS += GDB_REMOVE_UNNEEDED_FILES
+endif
 
 # This installs the gdbserver somewhere into the $(HOST_DIR) so that
 # it becomes an integral part of the SDK, if the toolchain generated
Signature · 1 line
2.7.4
#1 lore raw

PATCH 1/1 package/gdb: don't remove support files if python chosen

Quoted 15 lines · Jonas Zaddach

If one wants to use GDB with python support on the target, you need the support files installed by GDB. These get usually deleted to save some space, so I just wrapped the Makefile code deleting them in a conditional block depending on if python support is active or not.

Signed-off-by Jonas Zaddach <[email protected]>
---
 package/gdb/gdb.mk | 2 ++
 1 file changed, 2 insertions(+)
package/gdb/gdb.mk +0 -0
index 7e86ba0..5c915da 100644

package/gdb/gdb.mk endif · line 166

Quoted 3 lines
package/gdb/gdb.mk +0 -0
@@ -164,11 +164,13 @@ endif
 
 # This removes some unneeded Python scripts and XML target description
 # files that are not useful for a normal usage of the debugger.

Are the XML files also needed, or only the .py files?

It's especially surprising that you'd need those syscalls files for different
architectures than the target...

But anyway, it's just about 200K.

One small nit: I'd leave the comment within the condition. Otherwise:

Reviewed-by Arnout Vandecappelle (Essensium/Mind) <[email protected]>

Regards,
Arnout

Quoted 11 lines
package/gdb/gdb.mk +2 -0
+ifneq ($(BR2_PACKAGE_GDB_PYTHON),y)
 define GDB_REMOVE_UNNEEDED_FILES
 	$(RM) -rf $(TARGET_DIR)/usr/share/gdb
 endef
 
 GDB_POST_INSTALL_TARGET_HOOKS += GDB_REMOVE_UNNEEDED_FILES
+endif
 
 # This installs the gdbserver somewhere into the $(HOST_DIR) so that
 # it becomes an integral part of the SDK, if the toolchain generated
Signature · 6 lines
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
Applied
#2 lore raw

PATCH 1/1 package/gdb: don't remove support files if python chosen

In reply to Jonas Zaddach

Hello,

Quoted 9 lines · Jonas Zaddach

If one wants to use GDB with python support on the target, you need the support files installed by GDB. These get usually deleted to save some space, so I just wrapped the Makefile code deleting them in a conditional block depending on if python support is active or not.

Signed-off-by Jonas Zaddach <[email protected]>
---
 package/gdb/gdb.mk | 2 ++
 1 file changed, 2 insertions(+)

Applied to master after doing the following changes:

[Thomas:
- use positive logic "if python is disabled"
- put the comment inside the condition, as suggested by Arnout]

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
#3 lore raw

PATCH 1/1 package/gdb: don't remove support files if python chosen

In reply to Jonas Zaddach
Quoted 8 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Jonas" == Jonas Zaddach <[email protected]> writes:

If one wants to use GDB with python support on the target, you need the support files installed by GDB. These get usually deleted to save some space, so I just wrapped the Makefile code deleting them in a conditional block depending on if python support is active or not.

Signed-off-by Jonas Zaddach <[email protected]>

Committed to 2018.02.x, thanks.

Signature · 1 line
Bye, Peter Korsgaard