Skip to contentSkip to conversation

directfb: add an option for debugging support

PATCH New Cédric Marie 1 participant 2 messages Jun 9, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 directfb: add an option for debugging support

DirectFB debugging support is enabled by default. This new option makes it possible to disable it (--disable-debug-support) and build the library with no debug feature at all.

NB: When debugging support is disabled, the target installation path is suffixed with -pure.

---
 package/directfb/Config.in   | 7 +++++++
 package/directfb/directfb.mk | 4 ++++
 2 files changed, 11 insertions(+)
package/directfb/Config.in +7 -0
index 7759f8f..0015c0c 100644
@@ -20,8 +20,15 @@ config BR2_PACKAGE_DIRECTFB_MULTI
 comment "directfb multi application needs a Linux kernel to be built"
 	depends on !BR2_LINUX_KERNEL
 
+config BR2_PACKAGE_DIRECTFB_DEBUG_SUPPORT
+	bool "directfb debugging support"
+	default y
+	help
+	  Compile DirectFB with debugging support
+
 config BR2_PACKAGE_DIRECTFB_DEBUG
 	bool "directfb debugging"
+	depends on BR2_PACKAGE_DIRECTFB_DEBUG_SUPPORT
 	help
 	  Compile DirectFB with lots of debug output
 
package/directfb/directfb.mk +4 -0
index e3dc506..d8af56e 100644
@@ -39,6 +39,10 @@ DIRECTFB_CONF_OPTS += --enable-multi --enable-fusion
 DIRECTFB_DEPENDENCIES += linux-fusion
 endif
 
+ifneq ($(BR2_PACKAGE_DIRECTFB_DEBUG_SUPPORT),y)
+DIRECTFB_CONF_OPTS += --disable-debug-support
+endif
+
 ifeq ($(BR2_PACKAGE_DIRECTFB_DEBUG),y)
 DIRECTFB_CONF_OPTS += --enable-debug
 endif
Signature · 1 line
2.1.4
#1 lore raw

PATCH 1/1 directfb: add an option for debugging support

Hi,

Sorry for the noise...
Please forget the first version of the patch. The second one looks better to me.

Thank you.

C?dric

Le 09/06/2015 22:32, C?dric Marie a ?crit :

Quoted 49 lines

DirectFB debugging support is enabled by default. This new option makes it possible to disable it (--disable-debug-support) and build the library with no debug feature at all.

NB: When debugging support is disabled, the target installation path is suffixed with -pure.

---
  package/directfb/Config.in   | 7 +++++++
  package/directfb/directfb.mk | 4 ++++
  2 files changed, 11 insertions(+)
package/directfb/Config.in +7 -0
index 7759f8f..0015c0c 100644
@@ -20,8 +20,15 @@ config BR2_PACKAGE_DIRECTFB_MULTI
  comment "directfb multi application needs a Linux kernel to be built"
  	depends on !BR2_LINUX_KERNEL
 
+config BR2_PACKAGE_DIRECTFB_DEBUG_SUPPORT
+	bool "directfb debugging support"
+	default y
+	help
+	  Compile DirectFB with debugging support
+
  config BR2_PACKAGE_DIRECTFB_DEBUG
  	bool "directfb debugging"
+	depends on BR2_PACKAGE_DIRECTFB_DEBUG_SUPPORT
  	help
  	  Compile DirectFB with lots of debug output
 
package/directfb/directfb.mk +4 -0
index e3dc506..d8af56e 100644
@@ -39,6 +39,10 @@ DIRECTFB_CONF_OPTS += --enable-multi --enable-fusion
  DIRECTFB_DEPENDENCIES += linux-fusion
  endif
 
+ifneq ($(BR2_PACKAGE_DIRECTFB_DEBUG_SUPPORT),y)
+DIRECTFB_CONF_OPTS += --disable-debug-support
+endif
+
  ifeq ($(BR2_PACKAGE_DIRECTFB_DEBUG),y)
  DIRECTFB_CONF_OPTS += --enable-debug
  endif
Signature · 1 line
C?dric