All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATHC 0/9] Miscellaneous patch series
@ 2011-10-14 14:51 Will Wagner
  2011-10-14 14:51 ` [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration Will Wagner
                   ` (8 more replies)
  0 siblings, 9 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

This series of patches have a number of different fixes

First one adds option to set the timezone in the default target skeleton

Second one allows gdbserver to be copied from an external toolchain, comments o$

The Qt patches allow compilation for X11, add back Qt Commercial support, all t$

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
@ 2011-10-14 14:51 ` Will Wagner
  2011-10-15  9:51   ` Arnout Vandecappelle
  2011-10-14 14:51 ` [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain Will Wagner
                   ` (7 subsequent siblings)
  8 siblings, 1 reply; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

This adds a config option for the timezone which the user can configure for their needs

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 fs/skeleton/etc/TZ         |    2 +-
 target/generic/Config.in   |    6 ++++++
 target/generic/Makefile.in |    5 +++++
 3 files changed, 12 insertions(+), 1 deletions(-)

diff --git a/fs/skeleton/etc/TZ b/fs/skeleton/etc/TZ
index 9fcb2a3..885c76d 100644
--- a/fs/skeleton/etc/TZ
+++ b/fs/skeleton/etc/TZ
@@ -1 +1 @@
-MST7MDT
+MST7MDT # GENERIC_TIMEZONE
diff --git a/target/generic/Config.in b/target/generic/Config.in
index a91de32..0253f3c 100644
--- a/target/generic/Config.in
+++ b/target/generic/Config.in
@@ -127,6 +127,12 @@ config BR2_TARGET_GENERIC_REMOUNT_ROOTFS_RW
 	  read-only.
 	  If unsure, say Y.
 
+config BR2_TARGET_GENERIC_TIMEZONE
+	string "Timezone configured in /etc/TZ"
+	default "MST7MDT"
+	help
+	  Specify the configured timezone.
+
 endif # BR2_ROOTFS_SKELETON_DEFAULT
 
 config BR2_ROOTFS_POST_BUILD_SCRIPT
diff --git a/target/generic/Makefile.in b/target/generic/Makefile.in
index 4185202..3a33ef4 100644
--- a/target/generic/Makefile.in
+++ b/target/generic/Makefile.in
@@ -2,6 +2,7 @@ TARGET_GENERIC_HOSTNAME:=$(call qstrip,$(BR2_TARGET_GENERIC_HOSTNAME))
 TARGET_GENERIC_ISSUE:=$(call qstrip,$(BR2_TARGET_GENERIC_ISSUE))
 TARGET_GENERIC_GETTY:=$(call qstrip,$(BR2_TARGET_GENERIC_GETTY_PORT))
 TARGET_GENERIC_GETTY_BAUDRATE:=$(call qstrip,$(BR2_TARGET_GENERIC_GETTY_BAUDRATE))
+TARGET_GENERIC_TIMEZONE:=$(call qstrip,$(BR2_TARGET_GENERIC_TIMEZONE))
 
 target-generic-hostname:
 	mkdir -p $(TARGET_DIR)/etc
@@ -31,6 +32,9 @@ target-generic-do-remount-rw:
 target-generic-dont-remount-rw:
 	$(SED) '/^[^#].*# REMOUNT_ROOTFS_RW$$/s~^~#~' $(TARGET_DIR)/etc/inittab
 
+target_generic_timezone:
+	$(SED) '/# GENERIC_TIMEZONE$$/s~^.*#~$(TARGET_GENERIC_TIMEZONE) #~' $(TARGET_DIR)/etc/TZ
+
 ifneq ($(TARGET_GENERIC_HOSTNAME),)
 TARGETS += target-generic-hostname
 endif
@@ -50,4 +54,5 @@ TARGETS += target-generic-do-remount-rw
 else
 TARGETS += target-generic-dont-remount-rw
 endif
+TARGETS += target_generic_timezone
 endif
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
  2011-10-14 14:51 ` [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration Will Wagner
@ 2011-10-14 14:51 ` Will Wagner
  2011-10-15 10:13   ` Arnout Vandecappelle
  2011-10-17 20:32   ` Thomas Petazzoni
  2011-10-14 14:51 ` [Buildroot] [PATCH 3/9] Add openbox window manager Will Wagner
                   ` (6 subsequent siblings)
  8 siblings, 2 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

If you are using an external toolchain that includes gdb for host and gdbserver for target, this option will copy gdbserver onto the target.

Currently has support for ct-ng and codesourcery toolchains

Original idea by Anders Darander taken from http://comments.gmane.org/gmane.comp.lib.uclibc.buildroot/17747

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 toolchain/helpers.mk                     |   30 ++++++++++++++++++++++++++++++
 toolchain/toolchain-external/Config.in   |    6 ++++++
 toolchain/toolchain-external/ext-tool.mk |    8 ++++++++
 3 files changed, 44 insertions(+), 0 deletions(-)

diff --git a/toolchain/helpers.mk b/toolchain/helpers.mk
index 4d90d15..3dfbcf0 100644
--- a/toolchain/helpers.mk
+++ b/toolchain/helpers.mk
@@ -101,6 +101,36 @@ copy_toolchain_sysroot = \
 	find $(STAGING_DIR) -type d | xargs chmod 755
 
 #
+# Copy selected files from the toolchain debug-root to the target dir.
+#
+# $1: The sysroot-feature (this is used for the base of the debug-root)
+# $2: The file to be installed
+# $3: Destination folder on the rootfs.
+# $4: cross-gcc path
+#
+copy_toolchain_debugroot = \
+	SYSROOT_DIR="$(strip $1)" \
+	DEBUGROOT_DIR="$(strip $1)"/../debug-root; \
+	FILE="$(strip $2)"; \
+	DESTDIR="$(strip $3)"; \
+	__CROSS_CC="$(strip $4)" ; \
+\
+       if $$($${__CROSS_CC} -v 2>&1 | grep crosstool-NG 2>&1 1>/dev/null) ; then \
+               echo "A Crosstool-NG generated toolchain is detected."; \
+               if test -f $${DEBUGROOT_DIR}/usr/bin/$${FILE}; then \
+                       $(INSTALL) -m0755 $${DEBUGROOT_DIR}/usr/bin/$${FILE} $(TARGET_DIR)/$${DESTDIR}/$${FILE}; \
+               fi \
+       elif $$($(TARGET_CROSS)gcc -v 2>&1 | grep 'Sourcery G++' 2>&1 1>/dev/null) ; then \
+               echo "A Sourcery G++ generated toolchain is detected."; \
+               if test -f $${SYSROOT_DIR}/usr/bin/$${FILE}; then \
+                       $(INSTALL) -m0755 $${SYSROOT_DIR}/usr/bin/$${FILE} $(TARGET_DIR)/$${DESTDIR}/$${FILE}; \
+               fi \
+       else \
+               echo "Error: Not a known toolchain!"; \
+               exit 1; \
+       fi
+
+#
 # Create lib64 -> lib and usr/lib64 -> usr/lib symbolic links in the
 # target and staging directories. This is needed for some 64 bits
 # toolchains such as the Crosstool-NG toolchains, for which the path
diff --git a/toolchain/toolchain-external/Config.in b/toolchain/toolchain-external/Config.in
index ff7e8db..aa6829c 100644
--- a/toolchain/toolchain-external/Config.in
+++ b/toolchain/toolchain-external/Config.in
@@ -322,6 +322,12 @@ config BR2_TOOLCHAIN_EXTERNAL_CXX
 	  support. If you don't know, leave the default value,
 	  Buildroot will tell you if it's correct or not.
 
+config BR2_TOOLCHAIN_EXTERNAL_GDBSERVER
+	bool "Toolchain has GDB Server binary?"
+	help
+	  Select this option if your external toolchain has
+	  a GDB server binary to be copied to the target.
+
 endif # BR2_TOOLCHAIN_EXTERNAL_CUSTOM
 
 endif # BR2_TOOLCHAIN_EXTERNAL
diff --git a/toolchain/toolchain-external/ext-tool.mk b/toolchain/toolchain-external/ext-tool.mk
index b9d932f..38c0cc5 100644
--- a/toolchain/toolchain-external/ext-tool.mk
+++ b/toolchain/toolchain-external/ext-tool.mk
@@ -64,6 +64,10 @@ LIB_EXTERNAL_LIBS+=libpthread.so
 ifeq ($(BR2_PACKAGE_GDB_SERVER),y)
 LIB_EXTERNAL_LIBS+=libthread_db.so
 endif # gdbserver
+ifeq ($(BR2_TOOLCHAIN_EXTERNAL_GDBSERVER),y)
+LIB_EXTERNAL_LIBS+=libthread_db.so
+LIB_EXTERNAL_GDBSERVER+=gdbserver
+endif # external gdbserver
 endif # ! no threads
 
 # Details about sysroot directory selection.
@@ -300,6 +304,10 @@ $(STAMP_DIR)/ext-toolchain-installed: $(TOOLCHAIN_EXTERNAL_DEPENDENCIES)
 	if [ -L $${ARCH_SYSROOT_DIR}/lib64 ] ; then \
 		$(call create_lib64_symlinks) ; \
 	fi ; \
+	if test -n $(LIB_EXTERNAL_GDBSERVER) ; then \
+		echo "Copy gdbserver from the external toolchain to target..." ; \
+		$(call copy_toolchain_debugroot,$${SYSROOT_DIR},$(LIB_EXTERNAL_GDBSERVER),/usr/bin,$(TOOLCHAIN_EXTERNAL_CC)) ; \
+	fi ; \
 	touch $@
 
 # Build toolchain wrapper for preprocessor, C and C++ compiler, and setup
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 3/9] Add openbox window manager
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
  2011-10-14 14:51 ` [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration Will Wagner
  2011-10-14 14:51 ` [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain Will Wagner
@ 2011-10-14 14:51 ` Will Wagner
  2011-10-15 10:51   ` Arnout Vandecappelle
  2011-10-17 20:36   ` Thomas Petazzoni
  2011-10-14 14:51 ` [Buildroot] [PATCH 4/9] New package: Google Breakpad library Will Wagner
                   ` (5 subsequent siblings)
  8 siblings, 2 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

Another X window manager, this one is very minimalist and has few dependancies

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 package/Config.in          |    1 +
 package/openbox/Config.in  |   20 +++
 package/openbox/openbox.mk |   25 ++++
 package/openbox/rc.xml     |  337 ++++++++++++++++++++++++++++++++++++++++++++
 package/openbox/themerc    |  159 +++++++++++++++++++++
 5 files changed, 542 insertions(+), 0 deletions(-)
 create mode 100644 package/openbox/Config.in
 create mode 100644 package/openbox/openbox.mk
 create mode 100644 package/openbox/rc.xml
 create mode 100644 package/openbox/themerc

diff --git a/package/Config.in b/package/Config.in
index 8b9c816..28cd36f 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -128,6 +128,7 @@ source "package/matchbox/Config.in"
 source "package/metacity/Config.in"
 source "package/blackbox/Config.in"
 source "package/fluxbox/Config.in"
+source "package/openbox/Config.in"
 
 comment "X applications"
 source "package/alsamixergui/Config.in"
diff --git a/package/openbox/Config.in b/package/openbox/Config.in
new file mode 100644
index 0000000..f1ae363
--- /dev/null
+++ b/package/openbox/Config.in
@@ -0,0 +1,20 @@
+config BR2_PACKAGE_OPENBOX
+	bool "openbox"
+	depends on BR2_PACKAGE_XORG7
+	select BR2_PACKAGE_XLIB_LIBX11
+	select BR2_PACKAGE_LIBGLIB2
+	select BR2_PACKAGE_PANGO
+	select BR2_PACKAGE_LIBXML2
+	help
+	  Openbox is a highly configurable, next generation
+	  window manager with extensive standards support. 
+
+	  http://openbox.org/wiki/Main_Page
+
+config BR2_PACKAGE_OPENBOX_SIMPLE
+	bool "simple openbox config"
+	depends on BR2_PACKAGE_OPENBOX
+	help
+	  This will remove the default openbox
+	  configuration files and themes,
+	  replacing them with a very simple one
diff --git a/package/openbox/openbox.mk b/package/openbox/openbox.mk
new file mode 100644
index 0000000..cfc6976
--- /dev/null
+++ b/package/openbox/openbox.mk
@@ -0,0 +1,25 @@
+#############################################################
+#
+# openbox
+#
+#############################################################
+
+OPENBOX_VERSION:=3.4.11
+OPENBOX_SOURCE:=openbox-$(OPENBOX_VERSION).tar.gz
+OPENBOX_SITE:=http://openbox.org/dist/openbox/
+
+OPENBOX_DEPENDENCIES:=xlib_libX11 libglib2 pango libxml2
+
+ifeq ($(BR2_PACKAGE_OPENBOX_SIMPLE),y)
+  define OPENBOX_SIMPLE_CONFIG
+    rm $(TARGET_DIR)/etc/xdg/openbox/menu.xml
+    cp package/openbox/rc.xml $(TARGET_DIR)/etc/xdg/openbox/rc.xml
+    rm -rf $(TARGET_DIR)/usr/share/themes/*
+    mkdir -p $(TARGET_DIR)/usr/share/themes/Clearlooks/openbox-3
+    cp package/openbox/themerc $(TARGET_DIR)/usr/share/themes/Clearlooks/openbox-3/
+  endef
+
+  OPENBOX_POST_INSTALL_TARGET_HOOKS += OPENBOX_SIMPLE_CONFIG
+endif
+
+$(eval $(call AUTOTARGETS))
\ No newline at end of file
diff --git a/package/openbox/rc.xml b/package/openbox/rc.xml
new file mode 100644
index 0000000..3b9ac55
--- /dev/null
+++ b/package/openbox/rc.xml
@@ -0,0 +1,337 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<!-- Do not edit this file, it will be overwritten on install.
+        Copy the file to $HOME/.config/openbox/ instead. -->
+
+<openbox_config xmlns="http://openbox.org/3.4/rc">
+
+<resistance>
+  <strength>10</strength>
+  <screen_edge_strength>20</screen_edge_strength>
+</resistance>
+
+<focus>
+  <focusNew>yes</focusNew>
+  <!-- always try to focus new windows when they appear. other rules do
+       apply -->
+  <followMouse>no</followMouse>
+  <!-- move focus to a window when you move the mouse into it -->
+  <focusLast>yes</focusLast>
+  <!-- focus the last used window when changing desktops, instead of the one
+       under the mouse pointer. when followMouse is enabled -->
+  <underMouse>no</underMouse>
+  <!-- move focus under the mouse, even when the mouse is not moving -->
+  <focusDelay>200</focusDelay>
+  <!-- when followMouse is enabled, the mouse must be inside the window for
+       this many milliseconds (1000 = 1 sec) before moving focus to it -->
+  <raiseOnFocus>no</raiseOnFocus>
+  <!-- when followMouse is enabled, and a window is given focus by moving the
+       mouse into it, also raise the window -->
+</focus>
+
+<placement>
+  <policy>Smart</policy>
+  <!-- 'Smart' or 'UnderMouse' -->
+  <center>yes</center>
+  <!-- whether to place windows in the center of the free area found or
+       the top left corner -->
+  <monitor>Active</monitor>
+  <!-- with Smart placement on a multi-monitor system, try to place new windows
+       on: 'Any' - any monitor, 'Mouse' - where the mouse is, 'Active' - where
+       the active window is -->
+  <primaryMonitor>1</primaryMonitor>
+  <!-- The monitor where Openbox should place popup dialogs such as the
+       focus cycling popup, or the desktop switch popup.  It can be an index
+       from 1, specifying a particular monitor.  Or it can be one of the
+       following: 'Mouse' - where the mouse is, or
+                  'Active' - where the active window is -->
+</placement>
+
+<theme>
+  <name>Clearlooks</name>
+  <titleLayout>LC</titleLayout>
+  <!--
+      available characters are NDSLIMC, each can occur@most once.
+      N: window icon
+      L: window label (AKA title).
+      I: iconify
+      M: maximize
+      C: close
+      S: shade (roll up/down)
+      D: omnipresent (on all desktops).
+  -->
+  <keepBorder>yes</keepBorder>
+  <animateIconify>yes</animateIconify>
+  <font place="ActiveWindow">
+    <name>sans</name>
+    <size>8</size>
+    <!-- font size in points -->
+    <weight>bold</weight>
+    <!-- 'bold' or 'normal' -->
+    <slant>normal</slant>
+    <!-- 'italic' or 'normal' -->
+  </font>
+  <font place="InactiveWindow">
+    <name>sans</name>
+    <size>8</size>
+    <!-- font size in points -->
+    <weight>bold</weight>
+    <!-- 'bold' or 'normal' -->
+    <slant>normal</slant>
+    <!-- 'italic' or 'normal' -->
+  </font>
+  <font place="MenuHeader">
+    <name>sans</name>
+    <size>9</size>
+    <!-- font size in points -->
+    <weight>normal</weight>
+    <!-- 'bold' or 'normal' -->
+    <slant>normal</slant>
+    <!-- 'italic' or 'normal' -->
+  </font>
+  <font place="MenuItem">
+    <name>sans</name>
+    <size>9</size>
+    <!-- font size in points -->
+    <weight>normal</weight>
+    <!-- 'bold' or 'normal' -->
+    <slant>normal</slant>
+    <!-- 'italic' or 'normal' -->
+  </font>
+  <font place="OnScreenDisplay">
+    <name>sans</name>
+    <size>9</size>
+    <!-- font size in points -->
+    <weight>bold</weight>
+    <!-- 'bold' or 'normal' -->
+    <slant>normal</slant>
+    <!-- 'italic' or 'normal' -->
+  </font>
+</theme>
+
+<desktops>
+  <!-- this stuff is only used@startup, pagers allow you to change them
+       during a session
+
+       these are default values to use when other ones are not already set
+       by other applications, or saved in your session
+
+       use obconf if you want to change these without having to log out
+       and back in -->
+  <number>1</number>
+  <firstdesk>1</firstdesk>
+  <names>
+    <!-- set names up here if you want to, like this:
+    <name>desktop 1</name>
+    <name>desktop 2</name>
+    -->
+  </names>
+  <popupTime>0</popupTime>
+  <!-- The number of milliseconds to show the popup for when switching
+       desktops.  Set this to 0 to disable the popup. -->
+</desktops>
+
+<resize>
+  <drawContents>yes</drawContents>
+  <popupShow>Nonpixel</popupShow>
+  <!-- 'Always', 'Never', or 'Nonpixel' (xterms and such) -->
+  <popupPosition>Center</popupPosition>
+  <!-- 'Center', 'Top', or 'Fixed' -->
+  <popupFixedPosition>
+    <!-- these are used if popupPosition is set to 'Fixed' -->
+
+    <x>10</x>
+    <!-- positive number for distance from left edge, negative number for
+         distance from right edge, or 'Center' -->
+    <y>10</y>
+    <!-- positive number for distance from top edge, negative number for
+         distance from bottom edge, or 'Center' -->
+  </popupFixedPosition>
+</resize>
+
+<!-- You can reserve a portion of your screen where windows will not cover when
+     they are maximized, or when they are initially placed.
+     Many programs reserve space automatically, but you can use this in other
+     cases. -->
+<margins>
+  <top>0</top>
+  <bottom>0</bottom>
+  <left>0</left>
+  <right>0</right>
+</margins>
+
+<dock>
+  <position>TopLeft</position>
+  <!-- (Top|Bottom)(Left|Right|)|Top|Bottom|Left|Right|Floating -->
+  <floatingX>0</floatingX>
+  <floatingY>0</floatingY>
+  <noStrut>no</noStrut>
+  <stacking>Above</stacking>
+  <!-- 'Above', 'Normal', or 'Below' -->
+  <direction>Vertical</direction>
+  <!-- 'Vertical' or 'Horizontal' -->
+  <autoHide>no</autoHide>
+  <hideDelay>300</hideDelay>
+  <!-- in milliseconds (1000 = 1 second) -->
+  <showDelay>300</showDelay>
+  <!-- in milliseconds (1000 = 1 second) -->
+  <moveButton>Middle</moveButton>
+  <!-- 'Left', 'Middle', 'Right' -->
+</dock>
+
+<keyboard>
+
+  <!-- Keybindings for windows -->
+  <keybind key="H-F4">
+    <action name="Raise"/>
+  </keybind>
+</keyboard>
+
+<mouse>
+  <dragThreshold>8</dragThreshold>
+  <!-- number of pixels the mouse must move before a drag begins -->
+  <doubleClickTime>200</doubleClickTime>
+  <!-- in milliseconds (1000 = 1 second) -->
+  <screenEdgeWarpTime>400</screenEdgeWarpTime>
+  <!-- Time before changing desktops when the pointer touches the edge of the
+       screen while moving a window, in milliseconds (1000 = 1 second).
+       Set this to 0 to disable warping -->
+
+  <context name="Frame">
+    <mousebind button="A-Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+    </mousebind>
+    <mousebind button="A-Left" action="Click">
+      <action name="Unshade"/>
+    </mousebind>
+    <mousebind button="A-Left" action="Drag">
+      <action name="Move"/>
+    </mousebind>
+
+    <mousebind button="A-Right" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+      <action name="Unshade"/>
+    </mousebind>
+    <mousebind button="A-Right" action="Drag">
+      <action name="Resize"/>
+    </mousebind> 
+  </context>
+
+  <context name="Titlebar">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Move"/>
+    </mousebind>
+  </context>
+
+  <context name="Top">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+      <action name="Unshade"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"><edge>top</edge></action>
+    </mousebind>
+  </context>
+
+  <context name="Left">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"><edge>left</edge></action>
+    </mousebind>
+  </context>
+
+  <context name="Right">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"><edge>right</edge></action>
+    </mousebind>
+  </context>
+
+  <context name="Bottom">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"><edge>bottom</edge></action>
+    </mousebind>
+  </context>
+
+  <context name="BLCorner">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"/>
+    </mousebind>
+  </context>
+
+  <context name="BRCorner">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"/>
+    </mousebind>
+  </context>
+
+  <context name="TLCorner">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+      <action name="Unshade"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"/>
+    </mousebind>
+  </context>
+
+  <context name="TRCorner">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+      <action name="Unshade"/>
+    </mousebind>
+    <mousebind button="Left" action="Drag">
+      <action name="Resize"/>
+    </mousebind>
+  </context>
+
+  <context name="Close">
+    <mousebind button="Left" action="Press">
+      <action name="Focus"/>
+      <action name="Raise"/>
+      <action name="Unshade"/>
+    </mousebind>
+    <mousebind button="Left" action="Click">
+      <action name="Close"/>
+    </mousebind>
+  </context>
+</mouse>
+
+<menu>
+  <applicationIcons>no</applicationIcons>
+  <!-- controls if icons appear in the client-list-(combined-)menu -->
+  <manageDesktops>no</manageDesktops>
+  <!-- show the manage desktops section in the client-list-(combined-)menu -->
+</menu>
+
+<applications>
+</applications>
+
+</openbox_config>
diff --git a/package/openbox/themerc b/package/openbox/themerc
new file mode 100644
index 0000000..1dde5fb
--- /dev/null
+++ b/package/openbox/themerc
@@ -0,0 +1,159 @@
+!# Clearlooks-Evolving
+!# Clearlooks as it evolves in gnome-svn...
+!# Last updated 19/01/08
+
+# Fonts
+# these are really halos, but who cares?
+
+*.font: shadow=n
+window.active.label.text.font:shadow=y:shadowtint=25:shadowoffset=1
+window.inactive.label.text.font:shadow=y:shadowtint=00:shadowoffset=0
+menu.items.font:shadow=y:shadowtint=0:shadowoffset=1
+
+!# general stuff
+
+border.width: 1
+padding.width: 3
+padding.height: 2
+window.handle.width: 3
+window.client.padding.width: 0
+menu.overlap: 2
+*.justify: center
+
+!# lets set our damn shadows here, eh?
+
+*.bg.highlight: 50
+*.bg.shadow:    05
+
+window.active.title.bg.highlight: 40
+window.active.title.bg.shadow:    00
+
+window.inactive.title.bg.highlight: 30
+window.inactive.title.bg.shadow:    00
+
+window.*.grip.bg.highlight: 50
+window.*.grip.bg.shadow:    30
+
+window.*.handle.bg.highlight: 50
+window.*.handle.bg.shadow:    30
+
+!# Menu settings
+
+menu.border.color: #aaaaaa
+menu.border.width: 1
+
+menu.title.bg: solid flat
+menu.title.bg.color: #E6E7E6
+menu.title.text.color: #111111
+
+menu.items.bg: Flat Solid
+menu.items.bg.color: #ffffff
+menu.items.text.color: #111111
+menu.items.disabled.text.color: #aaaaaa
+
+menu.items.active.bg: Flat Gradient splitvertical border
+
+menu.items.active.bg.color: #97b8e2
+menu.items.active.bg.color.splitTo: #a8c5e9
+
+menu.items.active.bg.colorTo: #91b3de
+menu.items.active.bg.colorTo.splitTo: #80a7d6
+menu.items.active.bg.border.color: #4b6e99
+menu.items.active.text.color: #ffffff
+
+!# set handles here and only the once?
+
+window.*.handle.bg: Raised solid
+window.*.handle.bg.color: #eaebec
+
+window.*.grip.bg: Raised solid
+window.*.grip.bg.color: #eaebec
+
+!# Active
+
+!# no handles - kind of closer to the clearlooks border, maybe?
+!#window.active.border.color: #455d7c
+
+window.*.border.color: #585a5d
+
+window.active.title.separator.color: #4e76a8
+
+*.title.bg: Raised Gradient splitvertical
+*.title.bg.color: #8CB0DC
+*.title.bg.color.splitTo: #99BAE3
+*.title.bg.colorTo: #86ABD9
+*.title.bg.colorTo.splitTo: #7AA1D1
+
+window.active.label.bg: Parentrelative
+window.active.label.text.color: #ffffff
+
+window.active.button.*.bg: Flat Gradient splitvertical Border
+
+window.active.button.*.bg.color: #92B4DF
+window.active.button.*.bg.color.splitTo: #B0CAEB
+window.active.button.*.bg.colorTo: #86ABD9
+window.active.button.*.bg.colorTo.splitTo: #769FD0
+
+window.active.button.*.bg.border.color: #49678B
+window.active.button.*.image.color: #F4F5F6
+
+window.active.button.hover.bg.color: #b5d3ef
+window.active.button.hover.bg.color.splitTo: #b5d3ef
+window.active.button.hover.bg.colorTo: #9cbae7
+window.active.button.hover.bg.colorTo.splitTo: #8caede
+window.active.button.hover.bg.border.color: #4A658C
+window.active.button.hover.image.color: #ffffff
+
+window.active.button.pressed.bg: Flat solid Border
+window.active.button.pressed.bg.color: #7aa1d2
+
+window.active.button.hover.bg.border.color: #4A658C
+
+!# inactive
+
+!#window.inactive.border.color: #7e8285
+window.inactive.title.separator.color: #96999d
+
+window.inactive.title.bg: Raised Gradient splitvertical
+window.inactive.title.bg.color: #E3E2E0
+window.inactive.title.bg.color.splitTo: #EBEAE9
+window.inactive.title.bg.colorTo: #DEDCDA
+window.inactive.title.bg.colorTo.splitTo: #D5D3D1
+
+window.inactive.label.bg: Parentrelative
+window.inactive.label.text.color: #70747d
+
+window.inactive.button.*.bg: Flat Gradient splitVertical Border
+window.inactive.button.*.bg.color: #ffffff
+window.inactive.button.*.bg.color.splitto: #ffffff
+window.inactive.button.*.bg.colorTo: #F9F8F8
+window.inactive.button.*.bg.colorTo.splitto: #E9E7E6
+window.inactive.button.*.bg.border.color: #928F8B
+window.inactive.button.*.image.color: #6D6C6C
+
+
+!# osd (pop ups and what not, dock?)
+
+osd.border.width: 1
+osd.border.color:  #999
+
+osd.bg: flat border gradient splitvertical
+osd.bg.color: #F0EFEE
+osd.bg.color.splitto: #f5f5f4
+osd.bg.colorTo: #EAEBEC
+osd.bg.colorTo.splitto: #E7E5E4
+
+osd.bg.border.color: #ffffff
+
+osd.label.bg: parentrelative
+osd.label.bg.color: #efefef
+osd.label.bg.border.color: #9c9e9c
+
+osd.label.text.color: #444
+!# yeah whatever, this is fine anyhoo?
+osd.hilight.bg: flat vertical gradient 
+osd.hilight.bg.color: #9ebde5
+osd.hilight.bg.colorTo: #749dcf
+osd.unhilight.bg: flat vertical gradient
+osd.unhilight.bg.color: #BABDB6
+osd.unhilight.bg.colorTo: #efefef
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 4/9] New package: Google Breakpad library
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
                   ` (2 preceding siblings ...)
  2011-10-14 14:51 ` [Buildroot] [PATCH 3/9] Add openbox window manager Will Wagner
@ 2011-10-14 14:51 ` Will Wagner
  2011-10-15 10:31   ` Arnout Vandecappelle
  2011-10-17 20:40   ` Thomas Petazzoni
  2011-10-14 14:51 ` [Buildroot] [PATCH 5/9] Support for compiling Qt for X11 Will Wagner
                   ` (4 subsequent siblings)
  8 siblings, 2 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

The breakpad library is a crash reporting framework

See http://code.google.com/p/google-breakpad/ for more details

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 package/Config.in                                  |    1 +
 package/breakpad/Config.in                         |    7 ++++
 .../breakpad-782_crash_generation_server.patch     |   12 +++++++
 .../breakpad-782_linux_client_cleanup.patch        |   33 ++++++++++++++++++++
 package/breakpad/breakpad-782_linux_exit.patch     |   22 +++++++++++++
 package/breakpad/breakpad.mk                       |   16 +++++++++
 6 files changed, 91 insertions(+), 0 deletions(-)
 create mode 100644 package/breakpad/Config.in
 create mode 100644 package/breakpad/breakpad-782_crash_generation_server.patch
 create mode 100644 package/breakpad/breakpad-782_linux_client_cleanup.patch
 create mode 100644 package/breakpad/breakpad-782_linux_exit.patch
 create mode 100644 package/breakpad/breakpad.mk

diff --git a/package/Config.in b/package/Config.in
index 28cd36f..c14561a 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -355,6 +355,7 @@ source "package/liboil/Config.in"
 source "package/libsigc/Config.in"
 source "package/orc/Config.in"
 source "package/startup-notification/Config.in"
+source "package/breakpad/Config.in"
 endmenu
 
 menu "Text and terminal handling"
diff --git a/package/breakpad/Config.in b/package/breakpad/Config.in
new file mode 100644
index 0000000..64042e0
--- /dev/null
+++ b/package/breakpad/Config.in
@@ -0,0 +1,7 @@
+config BR2_PACKAGE_BREAKPAD
+	bool "breakpad"
+	select BR2_PACKAGE_LIBCURL
+	help
+	  An open-source multi-platform crash reporting system 
+
+	  http://code.google.com/p/google-breakpad/
diff --git a/package/breakpad/breakpad-782_crash_generation_server.patch b/package/breakpad/breakpad-782_crash_generation_server.patch
new file mode 100644
index 0000000..ca6693e
--- /dev/null
+++ b/package/breakpad/breakpad-782_crash_generation_server.patch
@@ -0,0 +1,12 @@
+Index: Makefile.am
+===================================================================
+--- a/Makefile.am	(revision 782)
++++ b/Makefile.am	(working copy)
+@@ -64,6 +64,7 @@
+ 
+ src_client_linux_libbreakpad_client_a_SOURCES = \
+ 	src/client/linux/crash_generation/crash_generation_client.cc \
++	src/client/linux/crash_generation/crash_generation_server.cc \
+ 	src/client/linux/handler/exception_handler.cc \
+ 	src/client/linux/minidump_writer/linux_dumper.cc \
+ 	src/client/linux/minidump_writer/minidump_writer.cc \
diff --git a/package/breakpad/breakpad-782_linux_client_cleanup.patch b/package/breakpad/breakpad-782_linux_client_cleanup.patch
new file mode 100644
index 0000000..d7c4cd7
--- /dev/null
+++ b/package/breakpad/breakpad-782_linux_client_cleanup.patch
@@ -0,0 +1,33 @@
+Index: src/client/linux/crash_generation/crash_generation_client.cc
+===================================================================
+--- a/src/client/linux/crash_generation/crash_generation_client.cc	(revision 778)
++++ b/src/client/linux/crash_generation/crash_generation_client.cc	(working copy)
+@@ -40,6 +40,13 @@
+ 
+ namespace google_breakpad {
+ 
++
++CrashGenerationClient::~CrashGenerationClient()
++{
++  if (server_fd_ >= 0)
++    sys_close(server_fd_);
++}
++
+ bool
+ CrashGenerationClient::RequestDump(const void* blob, size_t blob_size)
+ {
+Index: src/client/linux/crash_generation/crash_generation_client.h
+===================================================================
+--- a/src/client/linux/crash_generation/crash_generation_client.h	(revision 778)
++++ b/src/client/linux/crash_generation/crash_generation_client.h	(working copy)
+@@ -36,9 +36,7 @@
+ 
+ class CrashGenerationClient {
+ public:
+-  ~CrashGenerationClient()
+-  {
+-  }
++  ~CrashGenerationClient();
+ 
+   // Request the crash server to generate a dump.  |blob| is a hack,
+   // see exception_handler.h and minidump_writer.h
diff --git a/package/breakpad/breakpad-782_linux_exit.patch b/package/breakpad/breakpad-782_linux_exit.patch
new file mode 100644
index 0000000..6092cb1
--- /dev/null
+++ b/package/breakpad/breakpad-782_linux_exit.patch
@@ -0,0 +1,22 @@
+Index: src/client/linux/crash_generation/crash_generation_server.cc
+===================================================================
+--- a/src/client/linux/crash_generation/crash_generation_server.cc	(revision 778)
++++ b/src/client/linux/crash_generation/crash_generation_server.cc	(working copy)
+@@ -298,7 +298,17 @@
+ CrashGenerationServer::ClientEvent(short revents)
+ {
+   if (POLLHUP & revents)
++  {
++    if (exit_callback_) {
++      ClientInfo info;
++
++      info.crash_server_ = this;
++      info.pid_ = -1;
++
++      exit_callback_(exit_context_, &info);
++    }
+     return false;
++  }
+   assert(POLLIN & revents);
+ 
+   // A process has crashed and has signaled us by writing a datagram
diff --git a/package/breakpad/breakpad.mk b/package/breakpad/breakpad.mk
new file mode 100644
index 0000000..4880867
--- /dev/null
+++ b/package/breakpad/breakpad.mk
@@ -0,0 +1,16 @@
+#############################################################
+#
+# breakpad
+#
+#############################################################
+
+BREAKPAD_SITE:=http://google-breakpad.googlecode.com/svn/trunk/
+BREAKPAD_SITE_METHOD:=svn
+BREAKPAD_VERSION:=842
+BREAKPAD_DEPENDENCIES:=libcurl
+
+BREAKPAD_LIBTOOL_PATCH = NO
+BREAKPAD_INSTALL_STAGING = YES
+BREAKPAD_INSTALL_TARGET = NO
+
+$(eval $(call AUTOTARGETS))
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 5/9] Support for compiling Qt for X11
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
                   ` (3 preceding siblings ...)
  2011-10-14 14:51 ` [Buildroot] [PATCH 4/9] New package: Google Breakpad library Will Wagner
@ 2011-10-14 14:51 ` Will Wagner
  2011-10-14 14:51 ` [Buildroot] [PATCH 6/9] Qt commercial support Will Wagner
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

This patch enables you to cross compile the X11 version of Qt. Although cross compilation of Qt is not officially supported it works well and allows Qt to be build for a target running X11

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 package/qt/Config.in |   12 ++++++++++++
 package/qt/qt.mk     |   44 +++++++++++++++++++++++++++++++++++++++-----
 2 files changed, 51 insertions(+), 5 deletions(-)

diff --git a/package/qt/Config.in b/package/qt/Config.in
index ab231d9..513f135 100644
--- a/package/qt/Config.in
+++ b/package/qt/Config.in
@@ -11,6 +11,14 @@ menuconfig BR2_PACKAGE_QT
 
 if BR2_PACKAGE_QT
 
+comment "Qt will be built for use with an X server"
+	depends on BR2_PACKAGE_XSERVER_XORG_SERVER
+
+config BR2_PACKAGE_QT_X11
+	bool
+	depends on BR2_PACKAGE_XSERVER_XORG_SERVER
+	default y
+
 config BR2_PACKAGE_QT_DEBUG
 	bool "Compile with debug support"
 	help
@@ -85,6 +93,7 @@ config BR2_PACKAGE_QT_GUI_MODULE
 	  video output, or you don't require Qt GUI, say n.
 
 if BR2_PACKAGE_QT_GUI_MODULE
+if !BR2_PACKAGE_QT_X11
 menu "Pixel depths"
 comment "Deselecting each option leads to Qt's default (8,16,32)"
 
@@ -165,6 +174,7 @@ config BR2_PACKAGE_QT_SYSTEMFREETYPE
 	  Use shared libfreetype from the target system.
 	  See http://www.freetype.org/
 endchoice
+endif # !BR2_PACKAGE_QT_X11
 
 config BR2_PACKAGE_QT_GIF
 	bool "Enable GIF support"
@@ -255,9 +265,11 @@ endchoice
 
 source "package/qt/Config.sql.in"
 if BR2_PACKAGE_QT_GUI_MODULE
+if !BR2_PACKAGE_QT_X11
 source "package/qt/Config.gfx.in"
 source "package/qt/Config.mouse.in"
 source "package/qt/Config.keyboard.in"
+endif # !BR2_PACKAGE_QT_X11
 
 config BR2_PACKAGE_QT_PHONON
 	bool "Phonon Module"
diff --git a/package/qt/qt.mk b/package/qt/qt.mk
index cc035bf..a36fcb6 100644
--- a/package/qt/qt.mk
+++ b/package/qt/qt.mk
@@ -96,6 +96,10 @@ ifneq ($(QT_PIXEL_DEPTHS),)
 QT_CONFIGURE_OPTS += -depths $(subst $(space),$(comma),$(strip $(QT_PIXEL_DEPTHS)))
 endif
 
+ifneq ($(BR2_PACKAGE_QT_X11),y)
+### Embedded options
+QT_CONFIGURE_OPTS += -no-gfx-qnx -no-kbd-qnx -no-mouse-qnx
+
 ### Display drivers
 ifeq ($(BR2_PACKAGE_QT_GFX_LINUXFB),y)
 QT_CONFIGURE_OPTS += -qt-gfx-linuxfb
@@ -129,7 +133,6 @@ else
 QT_CONFIGURE_OPTS += -no-gfx-directfb
 endif
 
-
 ### Mouse drivers
 ifeq ($(BR2_PACKAGE_QT_MOUSE_PC),y)
 QT_CONFIGURE_OPTS += -qt-mouse-pc
@@ -178,6 +181,11 @@ else
 QT_CONFIGURE_OPTS += -no-kbd-qvfb
 endif
 
+else
+### X11 options
+QT_CONFIGURE_OPTS += -no-gtkstyle -no-opengl -no-openvg -no-glib --disable-script
+endif
+
 ifeq ($(BR2_PACKAGE_QT_DEBUG),y)
 QT_CONFIGURE_OPTS += -debug
 else
@@ -216,7 +224,12 @@ else
 QT_EMB_PLATFORM = generic
 endif
 
+ifneq ($(BR2_PACKAGE_QT_X11),y)
 QT_CONFIGURE_OPTS += -embedded $(QT_EMB_PLATFORM)
+else
+QT_CONFIGURE_OPTS += -xplatform linux-$(QT_EMB_PLATFORM)-g++ -arch i386
+QT_DEPENDENCIES   += xserver_xorg-server
+endif
 
 ifneq ($(BR2_PACKAGE_QT_GUI_MODULE),y)
 QT_CONFIGURE_OPTS += -no-gui
@@ -415,6 +428,12 @@ else
 QT_CONFIGURE_OPTS += -no-declarative
 endif
 
+ifeq ($(BR2_PACKAGE_XLIB_LIBXINERAMA),y)
+QT_CONFIGURE_OPTS += -xinerama
+else
+QT_CONFIGURE_OPTS += -no-xinerama
+endif
+
 # ccache and precompiled headers don't play well together
 ifeq ($(BR2_CCACHE),y)
 QT_CONFIGURE_OPTS += -no-pch
@@ -431,7 +450,11 @@ endif
 # End of workaround.
 
 # Variable for other Qt applications to use
+ifneq ($(BR2_PACKAGE_QT_X11),y)
 QT_QMAKE:=$(HOST_DIR)/usr/bin/qmake -spec qws/linux-$(QT_EMB_PLATFORM)-g++
+else
+QT_QMAKE:=$(HOST_DIR)/usr/bin/qmake -spec linux-$(QT_EMB_PLATFORM)-g++
+endif
 
 ################################################################################
 # QT_QMAKE_SET -- helper macro to set QMAKE_<variable> = <value> in
@@ -445,10 +468,24 @@ QT_QMAKE:=$(HOST_DIR)/usr/bin/qmake -spec qws/linux-$(QT_EMB_PLATFORM)-g++
 # E.G. use like this:
 # $(call QT_QMAKE_SET,variable,value,directory)
 ################################################################################
+ifneq ($(BR2_PACKAGE_QT_X11),y)
 define QT_QMAKE_SET
 	$(SED) '/QMAKE_$(1)/d' $(3)/mkspecs/qws/linux-$(QT_EMB_PLATFORM)-g++/qmake.conf
 	$(SED) '/include.*qws.conf/aQMAKE_$(1) = $(2)' $(3)/mkspecs/qws/linux-$(QT_EMB_PLATFORM)-g++/qmake.conf
 endef
+else
+define QT_QMAKE_SET
+	$(SED) '/QMAKE_$(1)/d' $(3)/mkspecs/linux-$(QT_EMB_PLATFORM)-g++/qmake.conf
+	$(SED) '/include.*linux.conf/aQMAKE_$(1) = $(2)' $(3)/mkspecs/linux-$(QT_EMB_PLATFORM)-g++/qmake.conf
+endef
+endif
+
+ifeq ($(BR2_PACKAGE_QT_X11),y)
+define QT_CONFIGURE_MKSPECS
+	cp -R $(@D)/mkspecs/linux-g++ $(@D)/mkspecs/linux-$(QT_EMB_PLATFORM)-g++
+	$(call QT_QMAKE_SET,INCDIR_X11,$(STAGING_DIR)/usr/X11R6/include,$(@D))
+endef
+endif
 
 ifneq ($(BR2_INET_IPV6),y)
 define QT_CONFIGURE_IPV6
@@ -465,6 +502,7 @@ endif
 
 define QT_CONFIGURE_CMDS
 	-[ -f $(@D)/Makefile ] && $(MAKE) -C $(@D) confclean
+	$(QT_CONFIGURE_MKSPECS)
 	$(QT_CONFIGURE_IPV6)
 	$(QT_CONFIGURE_CONFIG_FILE)
 	# Fix compiler path
@@ -489,10 +527,6 @@ define QT_CONFIGURE_CMDS
 		$(if $(VERBOSE),-verbose,-silent) \
 		-force-pkg-config \
 		$(QT_CONFIGURE_OPTS) \
-		-no-gfx-qnx \
-		-no-kbd-qnx \
-		-no-mouse-qnx \
-		-no-xinerama \
 		-no-cups \
 		-no-nis \
 		-no-accessibility \
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 6/9] Qt commercial support
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
                   ` (4 preceding siblings ...)
  2011-10-14 14:51 ` [Buildroot] [PATCH 5/9] Support for compiling Qt for X11 Will Wagner
@ 2011-10-14 14:51 ` Will Wagner
  2011-10-15 10:58   ` Arnout Vandecappelle
  2011-10-14 14:51 ` [Buildroot] [PATCH 7/9] Improve Qt ttf support Will Wagner
                   ` (2 subsequent siblings)
  8 siblings, 1 reply; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

This patch allows the commercial version of Qt to be built. It does not attempt to download the source code, instead it uses the new file: method so the user has to download the source manually

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 package/qt/Config.in |   21 +++++++++++++++++++--
 package/qt/qt.mk     |   11 +++++++++--
 2 files changed, 28 insertions(+), 4 deletions(-)

diff --git a/package/qt/Config.in b/package/qt/Config.in
index 513f135..4640937 100644
--- a/package/qt/Config.in
+++ b/package/qt/Config.in
@@ -11,6 +11,23 @@ menuconfig BR2_PACKAGE_QT
 
 if BR2_PACKAGE_QT
 
+choice
+	prompt "License type"
+	default BR2_PACKAGE_QT_LICENSE_OPENSOURCE
+	help
+	  Selects the license type: Opensource or Commercial
+
+config BR2_PACKAGE_QT_LICENSE_OPENSOURCE
+	bool "Opensource License"
+
+config BR2_PACKAGE_QT_LICENSE_COMMERCIAL
+	bool "Commercial License"
+	help
+	  You will need a valid license file for this to work.
+	  You will also need to download the source code manually.
+
+endchoice
+
 comment "Qt will be built for use with an X server"
 	depends on BR2_PACKAGE_XSERVER_XORG_SERVER
 
@@ -53,9 +70,9 @@ config BR2_PACKAGE_QT_STATIC
 endchoice
 
 config BR2_PACKAGE_QT_LICENSE_APPROVED
-	bool "Approve free license"
+	bool "Approve license"
 	help
-	  Select this if you approve one of the available free licenses for the
+	  Select this if you approve one of the available licenses for the
 	  Qt4 library.
 	  By doing this you will not be asked while the library is compiled.
 	  Please read and understand the license terms before approving this.
diff --git a/package/qt/qt.mk b/package/qt/qt.mk
index a36fcb6..78d821a 100644
--- a/package/qt/qt.mk
+++ b/package/qt/qt.mk
@@ -13,13 +13,20 @@
 ######################################################################
 
 QT_VERSION = 4.7.4
-QT_SOURCE  = qt-everywhere-opensource-src-$(QT_VERSION).tar.gz
+QT_LICENSE_TYPE = opensource
+ifeq ($(BR2_PACKAGE_QT_LICENSE_COMMERCIAL),y)
+QT_LICENSE_TYPE = commercial
+endif
+QT_SOURCE  = qt-everywhere-$(QT_LICENSE_TYPE)-src-$(QT_VERSION).tar.gz
 QT_SITE    = http://get.qt.nokia.com/qt/source
+ifeq ($(BR2_PACKAGE_QT_LICENSE_COMMERCIAL),y)
+QT_SITE    = file://$(HOME)
+endif
 
 QT_INSTALL_STAGING = YES
 
 ifeq ($(BR2_PACKAGE_QT_LICENSE_APPROVED),y)
-QT_CONFIGURE_OPTS += -opensource -confirm-license
+QT_CONFIGURE_OPTS += -$(QT_LICENSE_TYPE) -confirm-license
 endif
 
 QT_CONFIG_FILE=$(call qstrip,$(BR2_PACKAGE_QT_CONFIG_FILE))
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 7/9] Improve Qt ttf support
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
                   ` (5 preceding siblings ...)
  2011-10-14 14:51 ` [Buildroot] [PATCH 6/9] Qt commercial support Will Wagner
@ 2011-10-14 14:51 ` Will Wagner
  2011-10-14 14:52 ` [Buildroot] [PATCH 8/9] qt: fix config when specifying custom config file Will Wagner
  2011-10-14 14:52 ` [Buildroot] [PATCH 9/9] Mesa3d does not build with --enable-static set Will Wagner
  8 siblings, 0 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:51 UTC (permalink / raw)
  To: buildroot

This patch adds a config option to only copy specific fonts onto the target rather than all true type fonts

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 package/qt/Config.in |   13 +++++++++++++
 package/qt/qt.mk     |    8 +++++++-
 2 files changed, 20 insertions(+), 1 deletions(-)

diff --git a/package/qt/Config.in b/package/qt/Config.in
index 4640937..9b0d4d2 100644
--- a/package/qt/Config.in
+++ b/package/qt/Config.in
@@ -166,6 +166,19 @@ config BR2_PACKAGE_QT_FONT_JAPANESE
 config BR2_PACKAGE_QT_FONT_UNIFONT
 	bool "unicode"
 
+config BR2_PACKAGE_QT_FONT_ALL_TRUETYPE
+	bool "install all truetype fonts"
+	depends on !BR2_PACKAGE_QT_NOFREETYPE
+	default y
+
+config BR2_PACKAGE_QT_FONT_DEJAVU
+	bool "dejavu ttf"
+	depends on !BR2_PACKAGE_QT_NOFREETYPE && !BR2_PACKAGE_QT_FONT_ALL_TRUETYPE
+
+config BR2_PACKAGE_QT_FONT_VERA
+	bool "vera ttf"
+	depends on !BR2_PACKAGE_QT_NOFREETYPE && !BR2_PACKAGE_QT_FONT_ALL_TRUETYPE
+
 endmenu
 
 choice
diff --git a/package/qt/qt.mk b/package/qt/qt.mk
index 78d821a..1137218 100644
--- a/package/qt/qt.mk
+++ b/package/qt/qt.mk
@@ -303,6 +303,10 @@ QT_FONTS = $(addprefix $(STAGING_DIR)/usr/lib/fonts/, $(addsuffix *.qpf, \
 	   $(if $(BR2_PACKAGE_QT_FONT_JAPANESE),japanese) \
 	   $(if $(BR2_PACKAGE_QT_FONT_UNIFONT),unifont)))
 
+QT_FONTS += $(addprefix $(STAGING_DIR)/usr/lib/fonts/, $(addsuffix *.ttf, \
+          $(if $(BR2_PACKAGE_QT_FONT_DEJAVU),DejaVu) \
+          $(if $(BR2_PACKAGE_QT_FONT_VERA),Vera)))
+
 ifeq ($(BR2_PACKAGE_QT_QTFREETYPE),y)
 QT_CONFIGURE_OPTS += -qt-freetype
 else
@@ -647,19 +651,21 @@ define QT_INSTALL_TARGET_PLUGINS
 endef
 
 # Fonts installation
-ifneq ($(QT_FONTS),)
+ifneq ($(strip $(QT_FONTS)),)
 define QT_INSTALL_TARGET_FONTS
 	mkdir -p $(TARGET_DIR)/usr/lib/fonts
 	cp -dpf $(QT_FONTS) $(TARGET_DIR)/usr/lib/fonts
 endef
 endif
 
+ifeq ($(BR2_PACKAGE_QT_FONT_ALL_TRUETYPE),y)
 ifeq ($(BR2_PACKAGE_QT_QTFREETYPE)$(BR2_PACKAGE_QT_SYSTEMFREETYPE),y)
 define QT_INSTALL_TARGET_FONTS_TTF
 	mkdir -p $(TARGET_DIR)/usr/lib/fonts
 	cp -dpf $(STAGING_DIR)/usr/lib/fonts/*.ttf $(TARGET_DIR)/usr/lib/fonts
 endef
 endif
+endif
 
 define QT_INSTALL_TARGET_CMDS
 	$(QT_INSTALL_TARGET_LIBS)
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 8/9] qt: fix config when specifying custom config file
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
                   ` (6 preceding siblings ...)
  2011-10-14 14:51 ` [Buildroot] [PATCH 7/9] Improve Qt ttf support Will Wagner
@ 2011-10-14 14:52 ` Will Wagner
  2011-10-14 14:52 ` [Buildroot] [PATCH 9/9] Mesa3d does not build with --enable-static set Will Wagner
  8 siblings, 0 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:52 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 package/qt/qt.mk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/package/qt/qt.mk b/package/qt/qt.mk
index 1137218..e900363 100644
--- a/package/qt/qt.mk
+++ b/package/qt/qt.mk
@@ -32,7 +32,7 @@ endif
 QT_CONFIG_FILE=$(call qstrip,$(BR2_PACKAGE_QT_CONFIG_FILE))
 
 ifneq ($(QT_CONFIG_FILE),)
-QT_CONFIGURE_OPTS += -config buildroot
+QT_CONFIGURE_OPTS += -qconfig buildroot
 endif
 
 QT_CFLAGS = $(TARGET_CFLAGS)
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 9/9] Mesa3d does not build with --enable-static set
  2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
                   ` (7 preceding siblings ...)
  2011-10-14 14:52 ` [Buildroot] [PATCH 8/9] qt: fix config when specifying custom config file Will Wagner
@ 2011-10-14 14:52 ` Will Wagner
  8 siblings, 0 replies; 23+ messages in thread
From: Will Wagner @ 2011-10-14 14:52 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Will Wagner <will_wagner@carallon.com>
---
 package/x11r7/mesa3d/mesa3d.mk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/package/x11r7/mesa3d/mesa3d.mk b/package/x11r7/mesa3d/mesa3d.mk
index 1fc1631..d1d6ec4 100644
--- a/package/x11r7/mesa3d/mesa3d.mk
+++ b/package/x11r7/mesa3d/mesa3d.mk
@@ -8,7 +8,7 @@ MESA3D_SOURCE:=MesaLib-$(MESA3D_VERSION).tar.gz
 MESA3D_SITE:=ftp://ftp.freedesktop.org/pub/mesa/$(MESA3D_VERSION)
 
 MESA3D_AUTORECONF = NO
-MESA3D_CONF_OPT = --disable-egl --disable-glu --disable-glw --disable-glut --disable-gallium --with-driver=dri --with-dri-drivers=swrast
+MESA3D_CONF_OPT = --disable-static --disable-egl --disable-glu --disable-glw --disable-glut --disable-gallium --with-driver=dri --with-dri-drivers=swrast
 MESA3D_INSTALL_STAGING = YES
 
 MESA3D_DEPENDENCIES = xproto_glproto xlib_libXxf86vm xlib_libXdamage xlib_libXfixes xproto_dri2proto libdrm expat
-- 
1.7.2.5

^ permalink raw reply related	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration
  2011-10-14 14:51 ` [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration Will Wagner
@ 2011-10-15  9:51   ` Arnout Vandecappelle
  2011-10-17 20:01     ` Thomas Petazzoni
  0 siblings, 1 reply; 23+ messages in thread
From: Arnout Vandecappelle @ 2011-10-15  9:51 UTC (permalink / raw)
  To: buildroot


On Friday 14 October 2011 16:51:53, Will Wagner wrote:
> +target_generic_timezone:
> +       $(SED) '/# GENERIC_TIMEZONE$$/s~^.*#~$(TARGET_GENERIC_TIMEZONE) #~' $(TARGET_DIR)/etc/TZ
> +

 Why not just create the file here?  I.e.

	echo $(TARGET_GENERIC_TIMEZONE) > $(TARGET_DIR)/etc/TZ


 Perhaps even move it out of the SKELETON_DEFAULT condition.

 Regards,
 Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20111015/10eb0c59/attachment.html>

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain
  2011-10-14 14:51 ` [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain Will Wagner
@ 2011-10-15 10:13   ` Arnout Vandecappelle
  2011-10-17 20:32   ` Thomas Petazzoni
  1 sibling, 0 replies; 23+ messages in thread
From: Arnout Vandecappelle @ 2011-10-15 10:13 UTC (permalink / raw)
  To: buildroot


On Friday 14 October 2011 16:51:54, Will Wagner wrote:
> If you are using an external toolchain that includes gdb for host and gdbserver for target, this option will copy gdbserver onto the target.
> 
> Currently has support for ct-ng and codesourcery toolchains
> 
> Original idea by Anders Darander taken from http://comments.gmane.org/gmane.comp.lib.uclibc.buildroot/17747
> 
> Signed-off-by: Will Wagner <will_wagner@carallon.com>
> ---
>  toolchain/helpers.mk                     |   30 ++++++++++++++++++++++++++++++
>  toolchain/toolchain-external/Config.in   |    6 ++++++
>  toolchain/toolchain-external/ext-tool.mk |    8 ++++++++
>  3 files changed, 44 insertions(+), 0 deletions(-)
> 
> diff --git a/toolchain/helpers.mk b/toolchain/helpers.mk
> index 4d90d15..3dfbcf0 100644
> --- a/toolchain/helpers.mk
> +++ b/toolchain/helpers.mk
> @@ -101,6 +101,36 @@ copy_toolchain_sysroot = \
>  	find $(STAGING_DIR) -type d | xargs chmod 755
>  
>  #
> +# Copy selected files from the toolchain debug-root to the target dir.
> +#
> +# $1: The sysroot-feature (this is used for the base of the debug-root)
> +# $2: The file to be installed
> +# $3: Destination folder on the rootfs.
> +# $4: cross-gcc path
> +#
> +copy_toolchain_debugroot = \

 This macro is only called from one place and I don't see a reason why
it should ever be called from anywhere else.  So why not put it directly
in the ext-toolchain-installed target?

> +	SYSROOT_DIR="$(strip $1)" \
> +	DEBUGROOT_DIR="$(strip $1)"/../debug-root; \
> +	FILE="$(strip $2)"; \
> +	DESTDIR="$(strip $3)"; \
> +	__CROSS_CC="$(strip $4)" ; \
> +\
> +       if $$($${__CROSS_CC} -v 2>&1 | grep crosstool-NG 2>&1 1>/dev/null) ; then \
> +               echo "A Crosstool-NG generated toolchain is detected."; \
> +               if test -f $${DEBUGROOT_DIR}/usr/bin/$${FILE}; then \
> +                       $(INSTALL) -m0755 $${DEBUGROOT_DIR}/usr/bin/$${FILE} $(TARGET_DIR)/$${DESTDIR}/$${FILE}; \
> +               fi \
> +       elif $$($(TARGET_CROSS)gcc -v 2>&1 | grep 'Sourcery G++' 2>&1 1>/dev/null) ; then \
> +               echo "A Sourcery G++ generated toolchain is detected."; \
> +               if test -f $${SYSROOT_DIR}/usr/bin/$${FILE}; then \
> +                       $(INSTALL) -m0755 $${SYSROOT_DIR}/usr/bin/$${FILE} $(TARGET_DIR)/$${DESTDIR}/$${FILE}; \
> +               fi \
> +       else \
> +               echo "Error: Not a known toolchain!"; \
> +               exit 1; \
> +       fi

 Isn't it simpler to just try the possible locations?  I think we can
safely assume that if there is a file called 'gdbserver' in a bin directory,
it will be the gdbserver we want.

 So something like

	found=false; \
	for dir in $${DEBUGROOT_DIR} $${SYSROOT_DIR}; do \
		if test -f $${dir}/usr/bin/$${FILE}; then
			$(INSTALL) -m0755 $${SYSROOT_DIR}/usr/bin/$${FILE} $(TARGET_DIR)/$${DESTDIR}/$${FILE}; \
			found=true; \
			break; \
		fi; \
	done; \
	if [ -z "$${found}" ]; then \
		echo "gdbserver executable not found!"; \
		exit 1; \
	fi

> +
> +#
>  # Create lib64 -> lib and usr/lib64 -> usr/lib symbolic links in the
>  # target and staging directories. This is needed for some 64 bits
>  # toolchains such as the Crosstool-NG toolchains, for which the path
> diff --git a/toolchain/toolchain-external/Config.in b/toolchain/toolchain-external/Config.in
> index ff7e8db..aa6829c 100644
> --- a/toolchain/toolchain-external/Config.in
> +++ b/toolchain/toolchain-external/Config.in
> @@ -322,6 +322,12 @@ config BR2_TOOLCHAIN_EXTERNAL_CXX
>  	  support. If you don't know, leave the default value,
>  	  Buildroot will tell you if it's correct or not.
>  
> +config BR2_TOOLCHAIN_EXTERNAL_GDBSERVER
> +	bool "Toolchain has GDB Server binary?"
> +	help
> +	  Select this option if your external toolchain has
> +	  a GDB server binary to be copied to the target.
> +

 Either this one should depend on BR2_PACKAGE_GDB_SERVER or vice versa.

>  endif # BR2_TOOLCHAIN_EXTERNAL_CUSTOM
>  
>  endif # BR2_TOOLCHAIN_EXTERNAL
> diff --git a/toolchain/toolchain-external/ext-tool.mk b/toolchain/toolchain-external/ext-tool.mk
> index b9d932f..38c0cc5 100644
> --- a/toolchain/toolchain-external/ext-tool.mk
> +++ b/toolchain/toolchain-external/ext-tool.mk
> @@ -64,6 +64,10 @@ LIB_EXTERNAL_LIBS+=libpthread.so
>  ifeq ($(BR2_PACKAGE_GDB_SERVER),y)
>  LIB_EXTERNAL_LIBS+=libthread_db.so
>  endif # gdbserver
> +ifeq ($(BR2_TOOLCHAIN_EXTERNAL_GDBSERVER),y)
> +LIB_EXTERNAL_LIBS+=libthread_db.so
> +LIB_EXTERNAL_GDBSERVER+=gdbserver
> +endif # external gdbserver
>  endif # ! no threads
>  
>  # Details about sysroot directory selection.
> @@ -300,6 +304,10 @@ $(STAMP_DIR)/ext-toolchain-installed: $(TOOLCHAIN_EXTERNAL_DEPENDENCIES)
>  	if [ -L $${ARCH_SYSROOT_DIR}/lib64 ] ; then \
>  		$(call create_lib64_symlinks) ; \
>  	fi ; \
> +	if test -n $(LIB_EXTERNAL_GDBSERVER) ; then \
> +		echo "Copy gdbserver from the external toolchain to target..." ; \
> +		$(call copy_toolchain_debugroot,$${SYSROOT_DIR},$(LIB_EXTERNAL_GDBSERVER),/usr/bin,$(TOOLCHAIN_EXTERNAL_CC)) ; \
> +	fi ; \
>  	touch $@
>  
>  # Build toolchain wrapper for preprocessor, C and C++ compiler, and setup
> 


 Regards,
 Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20111015/57daba3a/attachment-0001.html>

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 4/9] New package: Google Breakpad library
  2011-10-14 14:51 ` [Buildroot] [PATCH 4/9] New package: Google Breakpad library Will Wagner
@ 2011-10-15 10:31   ` Arnout Vandecappelle
  2011-10-17 20:40   ` Thomas Petazzoni
  1 sibling, 0 replies; 23+ messages in thread
From: Arnout Vandecappelle @ 2011-10-15 10:31 UTC (permalink / raw)
  To: buildroot


On Friday 14 October 2011 16:51:56, Will Wagner wrote:
> diff --git a/package/breakpad/breakpad-782_crash_generation_server.patch b/package/breakpad/breakpad-782_crash_generation_server.patch
> new file mode 100644
> index 0000000..ca6693e
> --- /dev/null
> +++ b/package/breakpad/breakpad-782_crash_generation_server.patch
> @@ -0,0 +1,12 @@
> +Index: Makefile.am
> +===================================================================
> +--- a/Makefile.am      (revision 782)
> ++++ b/Makefile.am      (working copy)
> +@@ -64,6 +64,7 @@

 Patches should have a commit-log-like message at the top, including a
Signed-off-by line.

 Regards,
 Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20111015/86aaf34c/attachment.html>

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 3/9] Add openbox window manager
  2011-10-14 14:51 ` [Buildroot] [PATCH 3/9] Add openbox window manager Will Wagner
@ 2011-10-15 10:51   ` Arnout Vandecappelle
  2011-10-17 20:36   ` Thomas Petazzoni
  1 sibling, 0 replies; 23+ messages in thread
From: Arnout Vandecappelle @ 2011-10-15 10:51 UTC (permalink / raw)
  To: buildroot


On Friday 14 October 2011 16:51:55, Will Wagner wrote:
> +<!-- Do not edit this file, it will be overwritten on install.
> +        Copy the file to $HOME/.config/openbox/ instead. -->

 This comment seems a bit out-of-place...


 Regards,
 Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20111015/242912a1/attachment-0001.html>

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 6/9] Qt commercial support
  2011-10-14 14:51 ` [Buildroot] [PATCH 6/9] Qt commercial support Will Wagner
@ 2011-10-15 10:58   ` Arnout Vandecappelle
  2011-10-15 11:41     ` Jean-Christophe PLAGNIOL-VILLARD
  0 siblings, 1 reply; 23+ messages in thread
From: Arnout Vandecappelle @ 2011-10-15 10:58 UTC (permalink / raw)
  To: buildroot


On Friday 14 October 2011 16:51:58, Will Wagner wrote:
[snip]
> +ifeq ($(BR2_PACKAGE_QT_LICENSE_COMMERCIAL),y)
> +QT_SITE    = file://$(HOME)
> +endif

 The download location of the tar file should be a configuration option.
$(DL_DIR) seems like a better default for it.


 Regards,
 Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20111015/abde127d/attachment.html>

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 6/9] Qt commercial support
  2011-10-15 10:58   ` Arnout Vandecappelle
@ 2011-10-15 11:41     ` Jean-Christophe PLAGNIOL-VILLARD
  0 siblings, 0 replies; 23+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2011-10-15 11:41 UTC (permalink / raw)
  To: buildroot

On 12:58 Sat 15 Oct     , Arnout Vandecappelle wrote:
>    On Friday 14 October 2011 16:51:58, Will Wagner wrote:

Could you stop to post in HTML please

Best Regards,
J.

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration
  2011-10-15  9:51   ` Arnout Vandecappelle
@ 2011-10-17 20:01     ` Thomas Petazzoni
  2011-10-18  8:32       ` Arnout Vandecappelle
  0 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2011-10-17 20:01 UTC (permalink / raw)
  To: buildroot

Le Sat, 15 Oct 2011 11:51:42 +0200,
Arnout Vandecappelle <arnout@mind.be> a ?crit :

>  Why not just create the file here?  I.e.
> 
> 	echo $(TARGET_GENERIC_TIMEZONE) > $(TARGET_DIR)/etc/TZ
> 
>  Perhaps even move it out of the SKELETON_DEFAULT condition.

Agreed.

However, it would be good to mention in the config option help text
that this only works with uClibc. I don't ask to do it in this patch
(because it's a lot more work), but it would be great to make this
works for glibc as well (where it's not /etc/TZ, but /etc/timezone,
with a different syntax).

Regards,

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] 23+ messages in thread

* [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain
  2011-10-14 14:51 ` [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain Will Wagner
  2011-10-15 10:13   ` Arnout Vandecappelle
@ 2011-10-17 20:32   ` Thomas Petazzoni
  2011-10-18  8:23     ` Arnout Vandecappelle
  1 sibling, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2011-10-17 20:32 UTC (permalink / raw)
  To: buildroot

Hello,

Le Fri, 14 Oct 2011 15:51:54 +0100,
Will Wagner <will_wagner@carallon.com> a ?crit :

> If you are using an external toolchain that includes gdb for host and gdbserver for target, this option will copy gdbserver onto the target.
> 
> Currently has support for ct-ng and codesourcery toolchains
> 
> Original idea by Anders Darander taken from http://comments.gmane.org/gmane.comp.lib.uclibc.buildroot/17747
> 
> Signed-off-by: Will Wagner <will_wagner@carallon.com>

You should wrap the commit log (ditto for all your patches)

See also

 [PATCH 3/4] debug: provide an option to copy the gdbserver to the
 target

that I posted on July, 11th 2011.

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] 23+ messages in thread

* [Buildroot] [PATCH 3/9] Add openbox window manager
  2011-10-14 14:51 ` [Buildroot] [PATCH 3/9] Add openbox window manager Will Wagner
  2011-10-15 10:51   ` Arnout Vandecappelle
@ 2011-10-17 20:36   ` Thomas Petazzoni
  1 sibling, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2011-10-17 20:36 UTC (permalink / raw)
  To: buildroot

Le Fri, 14 Oct 2011 15:51:55 +0100,
Will Wagner <will_wagner@carallon.com> a ?crit :

> +OPENBOX_VERSION:=3.4.11
> +OPENBOX_SOURCE:=openbox-$(OPENBOX_VERSION).tar.gz
> +OPENBOX_SITE:=http://openbox.org/dist/openbox/
> +
> +OPENBOX_DEPENDENCIES:=xlib_libX11 libglib2 pango libxml2

You should use = instead of :=, and put a space before and after =.

> +ifeq ($(BR2_PACKAGE_OPENBOX_SIMPLE),y)

Perhaps this should be called
BR2_PACKAGE_OPENBOX_INSTALL_SIMPLE_CONFIG ?

> +  define OPENBOX_SIMPLE_CONFIG

and OPENBOX_INSTALL_SIMPLE_CONFIG

> +    rm $(TARGET_DIR)/etc/xdg/openbox/menu.xml
> +    cp package/openbox/rc.xml $(TARGET_DIR)/etc/xdg/openbox/rc.xml

install -D -m 0644 package/openbox/rc.xml $(TARGET_DIR)/etc/xdg/openbox/rc.xml

> +    rm -rf $(TARGET_DIR)/usr/share/themes/*

Couldn't this possibly erase some 'themes' from other applications ?

> +    mkdir -p $(TARGET_DIR)/usr/share/themes/Clearlooks/openbox-3
> +    cp package/openbox/themerc $(TARGET_DIR)/usr/share/themes/Clearlooks/openbox-3/

install -D -m 0644 package/openbox/themerc $(TARGET_DIR)/usr/share/themes/Clearlooks/openbox-3/themerc

Otherwise, looks good.

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] 23+ messages in thread

* [Buildroot] [PATCH 4/9] New package: Google Breakpad library
  2011-10-14 14:51 ` [Buildroot] [PATCH 4/9] New package: Google Breakpad library Will Wagner
  2011-10-15 10:31   ` Arnout Vandecappelle
@ 2011-10-17 20:40   ` Thomas Petazzoni
  1 sibling, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2011-10-17 20:40 UTC (permalink / raw)
  To: buildroot

Hello,

Le Fri, 14 Oct 2011 15:51:56 +0100,
Will Wagner <will_wagner@carallon.com> a ?crit :

> The breakpad library is a crash reporting framework
> 
> See http://code.google.com/p/google-breakpad/ for more details
> 
> Signed-off-by: Will Wagner <will_wagner@carallon.com>
> ---
>  package/Config.in                                  |    1 +
>  package/breakpad/Config.in                         |    7 ++++
>  .../breakpad-782_crash_generation_server.patch     |   12 +++++++
>  .../breakpad-782_linux_client_cleanup.patch        |   33 ++++++++++++++++++++
>  package/breakpad/breakpad-782_linux_exit.patch     |   22 +++++++++++++

As Arnout said, all patches require a proper description +
Signed-off-by line. See for example
package/libgtk2/host-libgtk2-2.20.1-reduce-dependencies.patch.

> +#############################################################
> +#
> +# breakpad
> +#
> +#############################################################
> +
> +BREAKPAD_SITE:=http://google-breakpad.googlecode.com/svn/trunk/
> +BREAKPAD_SITE_METHOD:=svn
> +BREAKPAD_VERSION:=842
> +BREAKPAD_DEPENDENCIES:=libcurl

= instead of :=, spaces before and after =.

> +BREAKPAD_LIBTOOL_PATCH = NO

Does it really break something?

> +BREAKPAD_INSTALL_STAGING = YES
> +BREAKPAD_INSTALL_TARGET = NO

This is a strange combination. A comment would be good to explain why a
library does not need to be installed on the target.

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] 23+ messages in thread

* [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain
  2011-10-17 20:32   ` Thomas Petazzoni
@ 2011-10-18  8:23     ` Arnout Vandecappelle
  2011-12-05 18:19       ` Will Wagner
  0 siblings, 1 reply; 23+ messages in thread
From: Arnout Vandecappelle @ 2011-10-18  8:23 UTC (permalink / raw)
  To: buildroot


On Monday 17 October 2011 22:32:26, Thomas Petazzoni wrote:
> See also
> 
>  [PATCH 3/4] debug: provide an option to copy the gdbserver to the
>  target
> 
> that I posted on July, 11th 2011.

 Grmbl, I missed that one and I already started improving Will's patch...  I just love wasting my time like that :-/

 [I lost my mail archive so it's a bit difficult to your to the original patch, so I'll put my comment here.]

 I notice that Will has
+ifeq ($(BR2_TOOLCHAIN_EXTERNAL_GDBSERVER),y)
+LIB_EXTERNAL_LIBS+=libthread_db.so
which was not there in your patch.  Is it not needed?


 Regards,
 Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration
  2011-10-17 20:01     ` Thomas Petazzoni
@ 2011-10-18  8:32       ` Arnout Vandecappelle
  0 siblings, 0 replies; 23+ messages in thread
From: Arnout Vandecappelle @ 2011-10-18  8:32 UTC (permalink / raw)
  To: buildroot


 [In a private e-mail, Will indicated to me that he doesn't really have time to work on these patches, so if anyone feels like picking them up...]

On Monday 17 October 2011 22:01:30, Thomas Petazzoni wrote:
> >  Perhaps even move it out of the SKELETON_DEFAULT condition.
> 
> Agreed.
> 
> However, it would be good to mention in the config option help text
> that this only works with uClibc. I don't ask to do it in this patch
> (because it's a lot more work), but it would be great to make this
> works for glibc as well (where it's not /etc/TZ, but /etc/timezone,
> with a different syntax).

 Actually, /etc/timezone can contain something like MST7MDT too.  It's just more descriptive to put America/Denver.

 And AFAIK, /etc/timezone is not even used by glibc, only the environment variable TZ or /etc/localtime.

 Regards,
 Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
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:  31BB CF53 8660 6F88 345D  54CC A836 5879 20D7 CF43

^ permalink raw reply	[flat|nested] 23+ messages in thread

* [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain
  2011-10-18  8:23     ` Arnout Vandecappelle
@ 2011-12-05 18:19       ` Will Wagner
  0 siblings, 0 replies; 23+ messages in thread
From: Will Wagner @ 2011-12-05 18:19 UTC (permalink / raw)
  To: buildroot

On 18/10/2011 09:23, Arnout Vandecappelle wrote:
>
> On Monday 17 October 2011 22:32:26, Thomas Petazzoni wrote:
>> See also
>>
>>   [PATCH 3/4] debug: provide an option to copy the gdbserver to the
>>   target
>>
>> that I posted on July, 11th 2011.
>
>   Grmbl, I missed that one and I already started improving Will's patch...  I just love wasting my time like that :-/
>
>   [I lost my mail archive so it's a bit difficult to your to the original patch, so I'll put my comment here.]
>
>   I notice that Will has
> +ifeq ($(BR2_TOOLCHAIN_EXTERNAL_GDBSERVER),y)
> +LIB_EXTERNAL_LIBS+=libthread_db.so
> which was not there in your patch.  Is it not needed?

Arnout, did you ever finish your changes here? I'd love to see a version 
of this committed to buildroot.

Any testing/review I can do?

Thanks
Will


-- 
------------------------------------------------------------------------
Will Wagner                                     will_wagner at carallon.com
Development Manager                      Office Tel: +44 (0)20 7371 2032
Carallon Ltd, Studio G20, Shepherds Building, Rockley Rd, London W14 0DA
------------------------------------------------------------------------

^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, other threads:[~2011-12-05 18:19 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-14 14:51 [Buildroot] [PATHC 0/9] Miscellaneous patch series Will Wagner
2011-10-14 14:51 ` [Buildroot] [PATCH 1/9] Allow timezone to be set in configuration Will Wagner
2011-10-15  9:51   ` Arnout Vandecappelle
2011-10-17 20:01     ` Thomas Petazzoni
2011-10-18  8:32       ` Arnout Vandecappelle
2011-10-14 14:51 ` [Buildroot] [PATCH 2/9] Copy gdbserver from external toolchain Will Wagner
2011-10-15 10:13   ` Arnout Vandecappelle
2011-10-17 20:32   ` Thomas Petazzoni
2011-10-18  8:23     ` Arnout Vandecappelle
2011-12-05 18:19       ` Will Wagner
2011-10-14 14:51 ` [Buildroot] [PATCH 3/9] Add openbox window manager Will Wagner
2011-10-15 10:51   ` Arnout Vandecappelle
2011-10-17 20:36   ` Thomas Petazzoni
2011-10-14 14:51 ` [Buildroot] [PATCH 4/9] New package: Google Breakpad library Will Wagner
2011-10-15 10:31   ` Arnout Vandecappelle
2011-10-17 20:40   ` Thomas Petazzoni
2011-10-14 14:51 ` [Buildroot] [PATCH 5/9] Support for compiling Qt for X11 Will Wagner
2011-10-14 14:51 ` [Buildroot] [PATCH 6/9] Qt commercial support Will Wagner
2011-10-15 10:58   ` Arnout Vandecappelle
2011-10-15 11:41     ` Jean-Christophe PLAGNIOL-VILLARD
2011-10-14 14:51 ` [Buildroot] [PATCH 7/9] Improve Qt ttf support Will Wagner
2011-10-14 14:52 ` [Buildroot] [PATCH 8/9] qt: fix config when specifying custom config file Will Wagner
2011-10-14 14:52 ` [Buildroot] [PATCH 9/9] Mesa3d does not build with --enable-static set Will Wagner

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.