Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes
@ 2010-05-15 23:15 Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 01/12] iconv: disallow selection when BR2_ENABLE_LOCALE is enabled Thomas Petazzoni
                   ` (12 more replies)
  0 siblings, 13 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

The following changes since commit 5e25bc482134bee62e3cc9d55af694aaa977d5b5:
  Thomas Petazzoni (1):
        external toolchain: check BR2_INSTALL_LIBSTDCPP

are available in the git repository at:

  git://git.busybox.net/~tpetazzoni/git/buildroot 2010-05-15-fixes

Thomas Petazzoni (12):
      iconv: disallow selection when BR2_ENABLE_LOCALE is enabled
      lmbench: needs BR2_INET_RPC
      startup-notification: add dependency on xlib_libX11
      ntp: fix build failure
      samba: only require libiconv if !BR2_ENABLE_LOCALE
      pango: run pango-querymodules on the target instead of on the host
      cairo: remove host variant
      mutt: fix build and convert to autotools
      lvm2: do not override CC and similar variables
      uemacs: don't forget CFLAGS at link time
      vpnc: pass CFLAGS at link time
      netsnmp: disable build with external toolchain

 package/cairo/cairo.mk                             |    3 -
 package/libiconv/Config.in                         |    1 +
 package/lmbench/Config.in                          |    2 +
 package/lvm2/lvm2.mk                               |    2 +-
 package/mutt/mutt-1.5.16-makedoc-hostcc.patch      |   32 ---------
 package/mutt/mutt-1.5.17+20080114-nodoc.patch      |   21 ++++++
 package/mutt/mutt.mk                               |   73 +++-----------------
 package/netsnmp/Config.in                          |    3 +
 package/ntp/ntp-4.2.4_p5-nano.patch                |   36 ++++++++++
 package/pango/S25pango                             |   20 ++++++
 package/pango/pango.mk                             |   18 +----
 package/samba/Config.in                            |    2 +-
 package/samba/samba.mk                             |    2 +-
 package/startup-notification/Config.in             |    2 +
 .../startup-notification/startup-notification.mk   |    1 +
 package/uemacs/uemacs.mk                           |    2 +-
 package/vpnc/vpnc.mk                               |    4 +-
 17 files changed, 105 insertions(+), 119 deletions(-)
 delete mode 100644 package/mutt/mutt-1.5.16-makedoc-hostcc.patch
 create mode 100644 package/mutt/mutt-1.5.17+20080114-nodoc.patch
 create mode 100644 package/ntp/ntp-4.2.4_p5-nano.patch
 create mode 100644 package/pango/S25pango

Thanks,
-- 
Thomas Petazzoni

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

* [Buildroot] [PATCH 01/12] iconv: disallow selection when BR2_ENABLE_LOCALE is enabled
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 02/12] lmbench: needs BR2_INET_RPC Thomas Petazzoni
                   ` (11 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

A C library compiled with locale support already has an iconv()
implementation. In that case, libiconv is useless. Therefore, disallow
the selection of libiconv when the configuration says that the
toolchain has locale support.

This fixes build issues similar to the one we had with gettext:
libiconv headers are installed, but -liconv is not added because the
./configure script detects that the C library supports iconv.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/libiconv/Config.in |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/package/libiconv/Config.in b/package/libiconv/Config.in
index b600944..ee4fd62 100644
--- a/package/libiconv/Config.in
+++ b/package/libiconv/Config.in
@@ -1,5 +1,6 @@
 config BR2_PACKAGE_LIBICONV
 	bool "libiconv"
+	depends on !BR2_ENABLE_LOCALE
 	help
 	  unicode conversion library
 
-- 
1.6.3.3

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

* [Buildroot] [PATCH 02/12] lmbench: needs BR2_INET_RPC
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 01/12] iconv: disallow selection when BR2_ENABLE_LOCALE is enabled Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-18 11:39   ` Peter Korsgaard
  2010-05-15 23:15 ` [Buildroot] [PATCH 03/12] startup-notification: add dependency on xlib_libX11 Thomas Petazzoni
                   ` (10 subsequent siblings)
  12 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

This package uses several pmap_*() functions in the C library, that
are only available if the C library has RPC support.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/lmbench/Config.in |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/package/lmbench/Config.in b/package/lmbench/Config.in
index 704b3ff..80d38f1 100644
--- a/package/lmbench/Config.in
+++ b/package/lmbench/Config.in
@@ -1,5 +1,7 @@
 config BR2_PACKAGE_LMBENCH
 	bool "lmbench"
+	# Uses pmap_set, pmap__unset, pmap_getport, etc.
+	depends on BR2_INET_RPC
 	help
 	  LMbench is a suite of simple, portable,
 	  ANSI/C microbenchmarks for UNIX/POSIX.
-- 
1.6.3.3

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

* [Buildroot] [PATCH 03/12] startup-notification: add dependency on xlib_libX11
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 01/12] iconv: disallow selection when BR2_ENABLE_LOCALE is enabled Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 02/12] lmbench: needs BR2_INET_RPC Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 04/12] ntp: fix build failure Thomas Petazzoni
                   ` (9 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/startup-notification/Config.in             |    2 ++
 .../startup-notification/startup-notification.mk   |    1 +
 2 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/package/startup-notification/Config.in b/package/startup-notification/Config.in
index 23fdf74..277d300 100644
--- a/package/startup-notification/Config.in
+++ b/package/startup-notification/Config.in
@@ -1,5 +1,7 @@
 config BR2_PACKAGE_STARTUP_NOTIFICATION
 	bool "startup-notification"
+	select BR2_PACKAGE_XLIB_LIBX11
+	depends on BR2_PACKAGE_XORG7
 	help
 	  Startup-notification is a library used to monitor application startup.
 
diff --git a/package/startup-notification/startup-notification.mk b/package/startup-notification/startup-notification.mk
index 62f067a..f70fcc1 100644
--- a/package/startup-notification/startup-notification.mk
+++ b/package/startup-notification/startup-notification.mk
@@ -9,6 +9,7 @@ STARTUP_NOTIFICATION_SITE = http://freedesktop.org/software/startup-notification
 STARTUP_NOTIFICATION_AUTORECONF = NO
 STARTUP_NOTIFICATION_INSTALL_STAGING = YES
 STARTUP_NOTIFICATION_INSTALL_TARGET = YES
+STARTUP_NOTIFICATION = xlib_libX11
 
 STARTUP_NOTIFICATION_CONF_ENV = lf_cv_sane_realloc=yes
 
-- 
1.6.3.3

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

* [Buildroot] [PATCH 04/12] ntp: fix build failure
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (2 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 03/12] startup-notification: add dependency on xlib_libX11 Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 05/12] samba: only require libiconv if !BR2_ENABLE_LOCALE Thomas Petazzoni
                   ` (8 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

This patch adds a patch to NTP to fix the following build failure,
using a workaround patch found on the Fedora bugzilla:

ntp_loopfilter.c: In function 'local_clock':
ntp_loopfilter.c:571: error: 'MOD_NANO' undeclared (first use in this function)
ntp_loopfilter.c:571: error: (Each undeclared identifier is reported only once
ntp_loopfilter.c:571: error: for each function it appears in.)
ntp_loopfilter.c: In function 'loop_config':
ntp_loopfilter.c:896: error: 'MOD_NANO' undeclared (first use in this function)

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/ntp/ntp-4.2.4_p5-nano.patch |   36 +++++++++++++++++++++++++++++++++++
 1 files changed, 36 insertions(+), 0 deletions(-)
 create mode 100644 package/ntp/ntp-4.2.4_p5-nano.patch

diff --git a/package/ntp/ntp-4.2.4_p5-nano.patch b/package/ntp/ntp-4.2.4_p5-nano.patch
new file mode 100644
index 0000000..f4097ee
--- /dev/null
+++ b/package/ntp/ntp-4.2.4_p5-nano.patch
@@ -0,0 +1,36 @@
+Fixes the MOD_NANO build failure
+
+ntp_loopfilter.c: In function 'local_clock':
+ntp_loopfilter.c:571: error: 'MOD_NANO' undeclared (first use in this function)
+ntp_loopfilter.c:571: error: (Each undeclared identifier is reported only once
+ntp_loopfilter.c:571: error: for each function it appears in.)
+ntp_loopfilter.c: In function 'loop_config':
+ntp_loopfilter.c:896: error: 'MOD_NANO' undeclared (first use in this function)
+
+Taken from
+http://cvs.fedoraproject.org/viewvc/devel/ntp/ntp-4.2.4p7-nano.patch?revision=1.1&content-type=text/plain&view=co
+
+See the discussion at
+https://bugs.launchpad.net/ubuntu/+source/ntp/+bug/412242 for more
+details and links. The below fix is not a correct fix, but only a
+workaround. The real fix required changes to kernel/libc headers.
+
+Index: ntp-4.2.4p5/include/ntp_syscall.h
+===================================================================
+--- ntp-4.2.4p5.orig/include/ntp_syscall.h	2010-05-14 17:34:48.000000000 +0200
++++ ntp-4.2.4p5/include/ntp_syscall.h	2010-05-14 17:35:13.000000000 +0200
+@@ -14,6 +14,14 @@
+ # include <sys/timex.h>
+ #endif
+ 
++#if defined(ADJ_NANO) && !defined(MOD_NANO)
++#define MOD_NANO ADJ_NANO
++#endif
++
++#if defined(ADJ_TAI) && !defined(MOD_TAI)
++#define MOD_TAI ADJ_TAI
++#endif
++
+ #ifndef NTP_SYSCALLS_LIBC
+ #ifdef NTP_SYSCALLS_STD
+ # define ntp_adjtime(t)		syscall(SYS_ntp_adjtime, (t))
-- 
1.6.3.3

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

* [Buildroot] [PATCH 05/12] samba: only require libiconv if !BR2_ENABLE_LOCALE
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (3 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 04/12] ntp: fix build failure Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 06/12] pango: run pango-querymodules on the target instead of on the host Thomas Petazzoni
                   ` (7 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/samba/Config.in |    2 +-
 package/samba/samba.mk  |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/samba/Config.in b/package/samba/Config.in
index 720ecd7..a7a0b34 100644
--- a/package/samba/Config.in
+++ b/package/samba/Config.in
@@ -1,6 +1,6 @@
 config BR2_PACKAGE_SAMBA
 	bool "samba"
-	select BR2_PACKAGE_LIBICONV
+	select BR2_PACKAGE_LIBICONV if !BR2_ENABLE_LOCALE
 	help
 	  Provides print services to all manner of SMB/CIFS clients,
 	  including the numerous versions of Microsoft Windows
diff --git a/package/samba/samba.mk b/package/samba/samba.mk
index 4115363..39968a9 100644
--- a/package/samba/samba.mk
+++ b/package/samba/samba.mk
@@ -16,7 +16,7 @@ SAMBA_INSTALL_TARGET = YES
 
 
 SAMBA_DEPENDENCIES = \
-	libiconv \
+	$(if $(BR2_ENABLE_LOCALE),,libiconv) \
 	$(if $(BR2_PACKAGE_SAMBA_RPCCLIENT),readline) \
 	$(if $(BR2_PACKAGE_SAMBA_SMBCLIENT),readline) \
 	$(if $(BR2_PACKAGE_SAMBA_AVAHI),avahi) \
-- 
1.6.3.3

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

* [Buildroot] [PATCH 06/12] pango: run pango-querymodules on the target instead of on the host
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (4 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 05/12] samba: only require libiconv if !BR2_ENABLE_LOCALE Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 07/12] cairo: remove host variant Thomas Petazzoni
                   ` (6 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

We needed to build pango for the host in order to compile
pango-querymodules for the host, which was used to produce
/etc/pango/pango.modules. Unfortunately:

 * This produces an incorrect /etc/pango/pango.modules (no modules
   detected in my case), probably because the host pango-querymodules
   was looking at host pango modules

 * This requires to build pango for the host, which requires to build
   cairo for the host, which requires to build X11 for the host.

To make things work and remove the dependency between pango and
host-pango, we introduce a S25pango script that creates
/etc/pango/pango.modules on startup if it doesn't exist, just as we do
with libgtk2 for /etc/gtk-2.0/gdk-pixbuf.loaders.

Since host-pango is no longer needed, we remove all definitions
related to it.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/pango/S25pango |   20 ++++++++++++++++++++
 package/pango/pango.mk |   18 ++----------------
 2 files changed, 22 insertions(+), 16 deletions(-)
 create mode 100644 package/pango/S25pango

diff --git a/package/pango/S25pango b/package/pango/S25pango
new file mode 100644
index 0000000..9af9307
--- /dev/null
+++ b/package/pango/S25pango
@@ -0,0 +1,20 @@
+#!/bin/sh
+#
+# run pango-querymodules if needed
+
+FILE=/etc/pango/pango.modules
+
+case "$1" in
+     start|"")
+	if [ ! -f $FILE ] ; then
+	   mkdir -p /etc/pango
+	   /usr/bin/pango-querymodules > "$FILE"
+	fi
+	;;
+    stop)
+	;;
+    *)
+	echo "Usage: $0 {start|stop}" >&2
+	exit 1
+	;;
+esac
\ No newline at end of file
diff --git a/package/pango/pango.mk b/package/pango/pango.mk
index 812a4c8..c640045 100644
--- a/package/pango/pango.mk
+++ b/package/pango/pango.mk
@@ -40,16 +40,7 @@ PANGO_CONF_ENV = ac_cv_func_posix_getpwuid_r=yes glib_cv_stack_grows=no \
 PANGO_CONF_OPT = --enable-shared --enable-static \
 		--enable-explicit-deps=no --disable-debug
 
-HOST_PANGO_CONF_OPT = \
-		--disable-static \
-		$(if $(BR2_PACKAGE_XORG7),--with-x,--without-x) \
-		--disable-debug \
-
-PANGO_DEPENDENCIES = $(if $(BR2_NEEDS_GETTEXT),gettext libintl) host-pkg-config host-pango libglib2 cairo
-
-HOST_PANGO_DEPENDENCIES = host-pkg-config host-cairo host-libglib2 host-autoconf host-automake
-
-HOST_PANGO_AUTORECONF = YES
+PANGO_DEPENDENCIES = $(if $(BR2_NEEDS_GETTEXT),gettext libintl) host-pkg-config libglib2 cairo
 
 ifeq ($(BR2_PACKAGE_XORG7),y)
         PANGO_CONF_OPT += --with-x \
@@ -61,12 +52,7 @@ else
 endif
 
 $(eval $(call AUTOTARGETS,package,pango))
-$(eval $(call AUTOTARGETS,package,pango,host))
 
 $(PANGO_HOOK_POST_INSTALL):
-	mkdir -p $(TARGET_DIR)/etc/pango
-	$(PANGO_HOST_BINARY) > $(TARGET_DIR)/etc/pango/pango.modules
-	$(SED) 's~$(HOST_DIR)~~g' $(TARGET_DIR)/etc/pango/pango.modules
+	$(INSTALL) -m 755 package/pango/S25pango $(TARGET_DIR)/etc/init.d/
 	touch $@
-
-PANGO_HOST_BINARY:=$(HOST_DIR)/usr/bin/pango-querymodules
-- 
1.6.3.3

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

* [Buildroot] [PATCH 07/12] cairo: remove host variant
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (5 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 06/12] pango: run pango-querymodules on the target instead of on the host Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 08/12] mutt: fix build and convert to autotools Thomas Petazzoni
                   ` (5 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

Since we no longer need to build pango on the host, cairo on the host
is no longer needed.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/cairo/cairo.mk |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/package/cairo/cairo.mk b/package/cairo/cairo.mk
index 0ca99aa..9b72ffa 100644
--- a/package/cairo/cairo.mk
+++ b/package/cairo/cairo.mk
@@ -79,8 +79,6 @@ else
 	CAIRO_CONF_OPT += --disable-svg
 endif
 
-HOST_CAIRO_DEPENDENCIES = host-pkg-config host-pixman host-fontconfig
-
 HOST_CAIRO_CONF_OPT = \
 		--enable-ps \
 		--enable-pdf \
@@ -90,4 +88,3 @@ HOST_CAIRO_CONF_OPT = \
 		--disable-svg
 
 $(eval $(call AUTOTARGETS,package,cairo))
-$(eval $(call AUTOTARGETS,package,cairo,host))
-- 
1.6.3.3

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

* [Buildroot] [PATCH 08/12] mutt: fix build and convert to autotools
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (6 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 07/12] cairo: remove host variant Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 09/12] lvm2: do not override CC and similar variables Thomas Petazzoni
                   ` (4 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

mutt has been upgraded, but the mutt-1.5.16-makedoc-hostcc.patch
wasn't upgraded accordingly, causing build failures in the
documentation. Therefore, we add a new patch
mutt-1.5.17+20080114-nodoc.patch that disables the construction of the
documentation.

As this patch modifies Makefile.am, we need to autoreconf the
package. The simplest and cleanest way to do this is to convert the
package to the autotools infrastructure.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/mutt/mutt-1.5.16-makedoc-hostcc.patch |   32 -----------
 package/mutt/mutt-1.5.17+20080114-nodoc.patch |   21 +++++++
 package/mutt/mutt.mk                          |   73 ++++---------------------
 3 files changed, 32 insertions(+), 94 deletions(-)
 delete mode 100644 package/mutt/mutt-1.5.16-makedoc-hostcc.patch
 create mode 100644 package/mutt/mutt-1.5.17+20080114-nodoc.patch

diff --git a/package/mutt/mutt-1.5.16-makedoc-hostcc.patch b/package/mutt/mutt-1.5.16-makedoc-hostcc.patch
deleted file mode 100644
index de5cc2d..0000000
--- a/package/mutt/mutt-1.5.16-makedoc-hostcc.patch
+++ /dev/null
@@ -1,32 +0,0 @@
---- mutt-1.5.16.oorig/Makefile.in	2007-09-02 22:16:23.000000000 +0200
-+++ mutt-1.5.16/Makefile.in	2007-09-02 22:19:49.000000000 +0200
-@@ -164,12 +164,6 @@
- DBX = @DBX@
- DEBUGGER = @DEBUGGER@
- 
--# $(makedoc_OBJECTS): $(makedoc_SOURCES)
--# 	$(HOST_CC) $(DEFS) $(AM_CPPFLAGS) $(AM_CPPFLAGS) $(CPPFLAGS) $(AM_CFLAGS) -c $<
--
--# makedoc: $(makedoc_OBJECTS) $(makedoc_DEPENDENCIES)
--# 	@rm -rf makedoc
--# 	$(HOST_CC) $(AM_CFLAGS) $(LDFLAGS) $(makedoc_LDFLAGS) $(makedoc_OBJECTS) $(makedoc_LDADD) -o makedoc
- DEFS = -DPKGDATADIR=\"$(pkgdatadir)\" -DSYSCONFDIR=\"$(sysconfdir)\" \
- 	-DBINDIR=\"$(bindir)\" -DMUTTLOCALEDIR=\"$(datadir)/locale\" \
- 	-DHAVE_CONFIG_H=1
-@@ -447,9 +441,15 @@
- 
- clean-binPROGRAMS:
- 	-test -z "$(bin_PROGRAMS)" || rm -f $(bin_PROGRAMS)
-+
-+$(makedoc_OBJECTS): $(makedoc_SOURCES)
-+	$(CC_FOR_BUILD) -DHAVE_UNISTD_H -DHAVE_GETOPT_H -DHAVE_STRERROR -c $< ;#$(DEFS) $(AM_CPPFLAGS) $(AM_CPPFLAGS) $(CPPFLAGS) $(AM_CFLAGS) -c $<
-+#makedoc: $(makedoc_OBJECTS) $(makedoc_DEPENDENCIES)
-+#	@rm -rf makedoc
-+#	$(CC_FOR_BUILD) $(AM_CFLAGS) $(LDFLAGS) $(makedoc_LDFLAGS) $(makedoc_OBJECTS) $(makedoc_LDADD) -o makedoc
- makedoc$(EXEEXT): $(makedoc_OBJECTS) $(makedoc_DEPENDENCIES) 
- 	@rm -f makedoc$(EXEEXT)
--	$(LINK) $(makedoc_LDFLAGS) $(makedoc_OBJECTS) $(makedoc_LDADD) $(LIBS)
-+	$(CC_FOR_BUILD) $(makedoc_LDFLAGS) $(makedoc_OBJECTS) $(makedoc_LDADD) $(LIBS) -o $@
- mutt$(EXEEXT): $(mutt_OBJECTS) $(mutt_DEPENDENCIES) 
- 	@rm -f mutt$(EXEEXT)
- 	$(LINK) $(mutt_LDFLAGS) $(mutt_OBJECTS) $(mutt_LDADD) $(LIBS)
diff --git a/package/mutt/mutt-1.5.17+20080114-nodoc.patch b/package/mutt/mutt-1.5.17+20080114-nodoc.patch
new file mode 100644
index 0000000..6fd556d
--- /dev/null
+++ b/package/mutt/mutt-1.5.17+20080114-nodoc.patch
@@ -0,0 +1,21 @@
+Disable the documentation build
+
+The documentation tool is compiled for the target, but executed on the
+host. We don't bother fixing this, as we don't care of the
+documentation in Buildroot.
+
+Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
+
+Index: mutt-1.5.17+20080114/Makefile.am
+===================================================================
+--- mutt-1.5.17+20080114.orig/Makefile.am	2010-05-15 18:15:08.000000000 +0200
++++ mutt-1.5.17+20080114/Makefile.am	2010-05-15 18:15:17.000000000 +0200
+@@ -9,7 +9,7 @@
+ IMAP_INCLUDES = -I$(top_srcdir)/imap
+ endif
+ 
+-SUBDIRS = m4 po intl doc contrib $(IMAP_SUBDIR)
++SUBDIRS = m4 po intl contrib $(IMAP_SUBDIR)
+ 
+ bin_SCRIPTS = muttbug flea @SMIMEAUX_TARGET@
+ 
diff --git a/package/mutt/mutt.mk b/package/mutt/mutt.mk
index 3ad0f2f..d021aef 100644
--- a/package/mutt/mutt.mk
+++ b/package/mutt/mutt.mk
@@ -7,70 +7,19 @@ MUTT_VERSION:=1.5.17+20080114
 MUTT_SOURCE:=mutt_$(MUTT_VERSION).orig.tar.gz
 MUTT_PATCH:=mutt_$(MUTT_VERSION)-1.diff.gz
 MUTT_SITE:=$(BR2_DEBIAN_MIRROR)/debian/pool/main/m/mutt/
-MUTT_DIR:=$(BUILD_DIR)/mutt-$(MUTT_VERSION)
-MUTT_CAT:=$(ZCAT)
-MUTT_BINARY:=mutt
-MUTT_TARGET_BINARY:=usr/bin/mutt
-
-$(DL_DIR)/$(MUTT_SOURCE):
-	$(call DOWNLOAD,$(MUTT_SITE),$(MUTT_SOURCE))
-
-$(DL_DIR)/$(MUTT_PATCH):
-	$(call DOWNLOAD,$(MUTT_SITE),$(MUTT_PATCH))
-
-$(MUTT_DIR)/.unpacked: $(DL_DIR)/$(MUTT_SOURCE) $(DL_DIR)/$(MUTT_PATCH)
-	$(MUTT_CAT) $(DL_DIR)/$(MUTT_SOURCE) | tar -C $(BUILD_DIR) $(TAR_OPTIONS) -
-	toolchain/patch-kernel.sh $(MUTT_DIR) package/mutt/ mutt-$(MUTT_VERSION)\*.patch
-ifneq ($(MUTT_PATCH),)
-	(cd $(MUTT_DIR) && $(MUTT_CAT) $(DL_DIR)/$(MUTT_PATCH) | patch -p1)
-	if [ -d $(MUTT_DIR)/debian/patches ]; then \
-		toolchain/patch-kernel.sh $(MUTT_DIR) $(MUTT_DIR)/debian/patches \*.patch; \
-	fi
-endif
-	touch $@
-
-$(MUTT_DIR)/.configured: $(MUTT_DIR)/.unpacked
-	(cd $(MUTT_DIR); rm -rf config.cache; \
-		$(TARGET_CONFIGURE_OPTS) \
-		$(TARGET_CONFIGURE_ARGS) \
-		./configure $(QUIET) \
-		--target=$(GNU_TARGET_NAME) \
-		--host=$(GNU_TARGET_NAME) \
-		--build=$(GNU_HOST_NAME) \
-		--prefix=/usr \
-		$(DISABLE_LARGEFILE) \
-		$(DISABLE_IPV6) \
-		$(DISABLE_NLS) \
+MUTT_DEPENDENCIES=ncurses
+MUTT_CONF_OPT = \
 		--disable-smtp \
 		--disable-iconv \
-		--without-wc-funcs \
-	)
-	touch $@
-
-$(MUTT_DIR)/$(MUTT_BINARY): $(MUTT_DIR)/.configured
-	$(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(MUTT_DIR)
-
-$(TARGET_DIR)/$(MUTT_TARGET_BINARY): $(MUTT_DIR)/$(MUTT_BINARY)
-	cp -dpf $(MUTT_DIR)/$(MUTT_BINARY) $@
-	$(STRIPCMD) $(STRIP_STRIP_ALL) $@
+		--without-wc-funcs
+MUTT_AUTORECONF=YES
 
-mutt-source: $(DL_DIR)/$(MUTT_SOURCE) $(DL_DIR)/$(MUTT_PATCH)
+define MUTT_APPLY_DEBIAN_PATCHES
+        if [ -d $(@D)/debian/patches ]; then \
+                toolchain/patch-kernel.sh $(@D) $(@D)/debian/patches \*.patch; \
+        fi
+endef
 
-mutt-unpacked: $(MUTT_DIR)/.unpacked
+MUTT_POST_PATCH_HOOKS += MUTT_APPLY_DEBIAN_PATCHES
 
-mutt: ncurses $(TARGET_DIR)/$(MUTT_TARGET_BINARY)
-
-mutt-clean:
-	-$(MAKE) -C $(MUTT_DIR) clean
-	rm -f $(TARGET_DIR)/$(MUTT_TARGET_BINARY)
-
-mutt-dirclean:
-	rm -rf $(MUTT_DIR)
-#############################################################
-#
-# Toplevel Makefile options
-#
-#############################################################
-ifeq ($(BR2_PACKAGE_MUTT),y)
-TARGETS+=mutt
-endif
+$(eval $(call AUTOTARGETS,package,mutt))
\ No newline at end of file
-- 
1.6.3.3

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

* [Buildroot] [PATCH 09/12] lvm2: do not override CC and similar variables
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (7 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 08/12] mutt: fix build and convert to autotools Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-18 11:43   ` Peter Korsgaard
  2010-05-15 23:15 ` [Buildroot] [PATCH 10/12] uemacs: don't forget CFLAGS at link time Thomas Petazzoni
                   ` (3 subsequent siblings)
  12 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

Since lvm2 uses autoconf, there's no point in overriding CC and other
variables at build time. This causes build breakage as CFLAGS aren't
used at link time, therefore --sysroot lacks, which breaks the build
with external toolchains.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/lvm2/lvm2.mk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/package/lvm2/lvm2.mk b/package/lvm2/lvm2.mk
index bc6f458..2bd6b8b 100644
--- a/package/lvm2/lvm2.mk
+++ b/package/lvm2/lvm2.mk
@@ -74,7 +74,7 @@ $(LVM2_DIR)/.configured: $(LVM2_DIR)/.unpacked
 
 
 $(LVM2_DIR)/.built: $(LVM2_DIR)/.configured
-	$(MAKE1) CC=$(TARGET_CC) RANLIB=$(TARGET_RANLIB) AR=$(TARGET_AR) -C $(LVM2_DIR) DESTDIR=$(STAGING_DIR)
+	$(MAKE1) -C $(LVM2_DIR) DESTDIR=$(STAGING_DIR)
 	$(MAKE1) -C $(LVM2_DIR) DESTDIR=$(STAGING_DIR) install
 	# Fixup write permissions so that the files can be overwritten
 	# several times in the $(TARGET_DIR)
-- 
1.6.3.3

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

* [Buildroot] [PATCH 10/12] uemacs: don't forget CFLAGS at link time
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (8 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 09/12] lvm2: do not override CC and similar variables Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 11/12] vpnc: pass " Thomas Petazzoni
                   ` (2 subsequent siblings)
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

Without CFLAGS, the build breaks because --sysroot is missing at link
time, and this option is absolutely mandatory for external toolchain
builds.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/uemacs/uemacs.mk |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/package/uemacs/uemacs.mk b/package/uemacs/uemacs.mk
index 6e2f4c4..539ad51 100644
--- a/package/uemacs/uemacs.mk
+++ b/package/uemacs/uemacs.mk
@@ -23,7 +23,7 @@ $(UEMACS_DIR)/.unpacked: $(DL_DIR)/$(UEMACS_SOURCE)
 
 $(UEMACS_DIR)/$(UEMACS_BINARY): $(UEMACS_DIR)/.unpacked
 	$(MAKE) -C $(UEMACS_DIR) \
-	CC="$(TARGET_CC)" DEFINES="-DAUTOCONF -DPOSIX -DUSG" CFLAGS+="$(TARGET_CFLAGS) " LIBS=-lncurses
+	CC="$(TARGET_CC)" DEFINES="-DAUTOCONF -DPOSIX -DUSG" CFLAGS+="$(TARGET_CFLAGS) " LIBS="$(TARGET_CFLAGS) -lncurses"
 	$(STRIPCMD) $(UEMACS_DIR)/$(UEMACS_BINARY)
 
 $(TARGET_DIR)/$(UEMACS_TARGET_BINARY): $(UEMACS_DIR)/$(UEMACS_BINARY)
-- 
1.6.3.3

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

* [Buildroot] [PATCH 11/12] vpnc: pass CFLAGS at link time
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (9 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 10/12] uemacs: don't forget CFLAGS at link time Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-15 23:15 ` [Buildroot] [PATCH 12/12] netsnmp: disable build with external toolchain Thomas Petazzoni
  2010-05-18 11:56 ` [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Peter Korsgaard
  12 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

External toolchain builds absolutely require the --sysroot option to
be passed at link time.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/vpnc/vpnc.mk |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/vpnc/vpnc.mk b/package/vpnc/vpnc.mk
index 5d0f87a..c4f23fa 100644
--- a/package/vpnc/vpnc.mk
+++ b/package/vpnc/vpnc.mk
@@ -26,7 +26,7 @@ $(VPNC_BINARY): $(VPNC_DIR)/.unpacked
 	rm -f $@
 	$(MAKE) $(TARGET_CONFIGURE_OPTS) INCLUDE=$(STAGING_DIR)/usr/include \
 		CFLAGS="$(TARGET_CFLAGS)" \
-		LDFLAGS+=-lgcrypt LDFLAGS+=-lgpg-error \
+		LDFLAGS+=-lgcrypt LDFLAGS+=-lgpg-error LDFLAGS+="$(TARGET_CFLAGS)" \
 		CC="$(TARGET_CC)" -C $(VPNC_DIR)
 
 $(VPNC_TARGET_BINARY): $(VPNC_BINARY)
@@ -38,7 +38,7 @@ $(VPNC_TARGET_BINARY): $(VPNC_BINARY)
 		MANDIR=/usr/share/man \
 		VERSION=$(VPNC_VERSION) \
 		INCLUDE=$(STAGING_DIR)/usr/include \
-		LDFLAGS="-lgcrypt -lgpg-error" \
+		LDFLAGS="-lgcrypt -lgpg-error $(TARGET_CFLAGS)" \
 		-C $(VPNC_DIR) install
 	$(STRIPCMD) $(STRIP_STRIP_UNNEEDED) $(VPNC_TARGET_BINARY)
 
-- 
1.6.3.3

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

* [Buildroot] [PATCH 12/12] netsnmp: disable build with external toolchain
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (10 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 11/12] vpnc: pass " Thomas Petazzoni
@ 2010-05-15 23:15 ` Thomas Petazzoni
  2010-05-18 11:47   ` Peter Korsgaard
  2010-05-18 11:56 ` [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Peter Korsgaard
  12 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-15 23:15 UTC (permalink / raw)
  To: buildroot

There is a build problem with netsnmp on external toolchain that isn't
easy to fix: libtool strips the --sysroot that we are passing it,
which make the link step fail.

In preparation for the release, just make sure this package isn't
visible to external toolchain users. Of course, on the long run, we
should fix this.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 package/netsnmp/Config.in |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/package/netsnmp/Config.in b/package/netsnmp/Config.in
index fd54eb4..91e8efb 100644
--- a/package/netsnmp/Config.in
+++ b/package/netsnmp/Config.in
@@ -1,5 +1,8 @@
 config BR2_PACKAGE_NETSNMP
 	bool "netsnmp"
+	# Build with external toolchains is broken, libtool strips
+	# --sysroot at link time
+	depends on !BR2_TOOLCHAIN_EXTERNAL
 	help
 	  Suite of applications used to implement SNMP v1, SNMP v2c, and
 	  SNMP v3 using both IPv4 and IPv6.
-- 
1.6.3.3

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

* [Buildroot] [PATCH 02/12] lmbench: needs BR2_INET_RPC
  2010-05-15 23:15 ` [Buildroot] [PATCH 02/12] lmbench: needs BR2_INET_RPC Thomas Petazzoni
@ 2010-05-18 11:39   ` Peter Korsgaard
  0 siblings, 0 replies; 23+ messages in thread
From: Peter Korsgaard @ 2010-05-18 11:39 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> This package uses several pmap_*() functions in the C library, that
 Thomas> are only available if the C library has RPC support.

A comment "lmbench needs a toolchain with RPC support" would be good,
but I can add that myself.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [PATCH 09/12] lvm2: do not override CC and similar variables
  2010-05-15 23:15 ` [Buildroot] [PATCH 09/12] lvm2: do not override CC and similar variables Thomas Petazzoni
@ 2010-05-18 11:43   ` Peter Korsgaard
  2010-05-19 11:24     ` Thomas Petazzoni
  0 siblings, 1 reply; 23+ messages in thread
From: Peter Korsgaard @ 2010-05-18 11:43 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> Since lvm2 uses autoconf, there's no point in overriding CC and
 Thomas> other variables at build time. This causes build breakage as
 Thomas> CFLAGS aren't used at link time, therefore --sysroot lacks,
 Thomas> which breaks the build with external toolchains.

Thanks. Is there anything stopping us from converting it to
Makefile.autotools.in format?

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [PATCH 12/12] netsnmp: disable build with external toolchain
  2010-05-15 23:15 ` [Buildroot] [PATCH 12/12] netsnmp: disable build with external toolchain Thomas Petazzoni
@ 2010-05-18 11:47   ` Peter Korsgaard
  0 siblings, 0 replies; 23+ messages in thread
From: Peter Korsgaard @ 2010-05-18 11:47 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> There is a build problem with netsnmp on external toolchain that isn't
 Thomas> easy to fix: libtool strips the --sysroot that we are passing it,
 Thomas> which make the link step fail.

 Thomas> In preparation for the release, just make sure this package isn't
 Thomas> visible to external toolchain users. Of course, on the long run, we
 Thomas> should fix this.

Hmm, yeah. We'll need to do the same for BR2_PACKAGE_QUAGGA_SNMP then.

Maybe a comment "netsnmp is broken with external toolchain" would be
good.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes
  2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
                   ` (11 preceding siblings ...)
  2010-05-15 23:15 ` [Buildroot] [PATCH 12/12] netsnmp: disable build with external toolchain Thomas Petazzoni
@ 2010-05-18 11:56 ` Peter Korsgaard
  2010-05-18 12:16   ` Thomas Petazzoni
  12 siblings, 1 reply; 23+ messages in thread
From: Peter Korsgaard @ 2010-05-18 11:56 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> The following changes since commit 5e25bc482134bee62e3cc9d55af694aaa977d5b5:
 Thomas>   Thomas Petazzoni (1):
 Thomas>         external toolchain: check BR2_INSTALL_LIBSTDCPP

 Thomas> are available in the git repository at:

 Thomas>   git://git.busybox.net/~tpetazzoni/git/buildroot 2010-05-15-fixes

 Thomas> Thomas Petazzoni (12):
 Thomas>       iconv: disallow selection when BR2_ENABLE_LOCALE is enabled
 Thomas>       lmbench: needs BR2_INET_RPC
 Thomas>       startup-notification: add dependency on xlib_libX11
 Thomas>       ntp: fix build failure
 Thomas>       samba: only require libiconv if !BR2_ENABLE_LOCALE
 Thomas>       pango: run pango-querymodules on the target instead of on the host
 Thomas>       cairo: remove host variant
 Thomas>       mutt: fix build and convert to autotools
 Thomas>       lvm2: do not override CC and similar variables
 Thomas>       uemacs: don't forget CFLAGS at link time
 Thomas>       vpnc: pass CFLAGS at link time
 Thomas>       netsnmp: disable build with external toolchain

I wanted to pull this, but that's not what I got:

From git://git.busybox.net/~tpetazzoni/git/buildroot
 * branch            2010-05-15-fixes -> FETCH_HEAD
Removing package/mutt/mutt-1.5.16-makedoc-hostcc.patch
Merge made by recursive.
 package/avahi/Config.in                            |    5 +-
 package/avahi/avahi.mk                             |    2 +-
 package/cairo/cairo.mk                             |    3 -
 package/gettext/Config.in                          |    2 +
 package/gmpc/Config.in                             |    4 +-
 package/gmpc/gmpc.mk                               |    3 +-
 package/grep/Config.in                             |    4 +-
 package/grep/grep.mk                               |    2 +-
 package/hal/Config.in                              |    4 +-
 package/hal/hal.mk                                 |    2 +-
 package/libcap/libcap.mk                           |    4 +-
 ...cgicc-3.2.9-disable-documentation-option.patch} |    0
 package/libcgicc/libcgicc.mk                       |    2 +
 package/libglib2/Config.in                         |    4 +-
 package/libglib2/libglib2.mk                       |    2 +-
 package/libiconv/Config.in                         |    1 +
 package/libidn/libidn.mk                           |    2 +-
 package/libsoup/libsoup.mk                         |    2 +-
 package/lmbench/Config.in                          |    2 +
 package/ltrace/ltrace.mk                           |    3 +-
 package/lvm2/lvm2.mk                               |    2 +-
 package/make/Config.in                             |    4 +-
 package/make/make.mk                               |    2 +-
 package/multimedia/libmpd/Config.in                |    2 -
 package/mutt/mutt-1.5.16-makedoc-hostcc.patch      |   32 ---------
 package/mutt/mutt-1.5.17+20080114-nodoc.patch      |   21 ++++++
 package/mutt/mutt.mk                               |   73 +++-----------------
 package/netsnmp/Config.in                          |    3 +
 package/ntp/ntp-4.2.4_p5-nano.patch                |   36 ++++++++++
 package/pango/S25pango                             |   20 ++++++
 package/pango/pango.mk                             |   18 +----
 package/php/Config.ext                             |    2 +-
 package/php/php.mk                                 |    2 +-
 package/psmisc/Config.in                           |    4 +-
 package/psmisc/psmisc.mk                           |    8 +--
 package/samba/Config.in                            |    2 +-
 package/samba/samba.mk                             |    2 +-
 package/sshfs/Config.in                            |    4 +-
 package/sshfs/sshfs.mk                             |    2 +-
 package/startup-notification/Config.in             |    2 +
 .../startup-notification/startup-notification.mk   |    1 +
 package/uemacs/uemacs.mk                           |    2 +-
 package/util-linux/Config.in                       |    4 +-
 package/util-linux/util-linux.mk                   |    4 +-
 package/vpnc/vpnc.mk                               |    4 +-
 toolchain/Config.in.2                              |    6 ++
 toolchain/external-toolchain/ext-tool.mk           |   12 +++
 47 files changed, 164 insertions(+), 163 deletions(-)

E.G. the branch also contained your older pull request, including the
avahi ipv6 dependency that I NACKed.

Could you fix that and send a new pull request, please?

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes
  2010-05-18 11:56 ` [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Peter Korsgaard
@ 2010-05-18 12:16   ` Thomas Petazzoni
  2010-05-18 12:20     ` Peter Korsgaard
  0 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-18 12:16 UTC (permalink / raw)
  To: buildroot

On Tue, 18 May 2010 13:56:51 +0200
Peter Korsgaard <jacmet@uclibc.org> wrote:

> I wanted to pull this, but that's not what I got:
> 
> E.G. the branch also contained your older pull request, including the
> avahi ipv6 dependency that I NACKed.
> 
> Could you fix that and send a new pull request, please?

Argh, yes, of course. In fact, it's because I started my second branch
on top of the first, because some of the fixes of the second branch
depend on fixes on the first branch.

The easiest is probably that I merge both branches, get rid of the
unwanted Avahi IPv6 fix, and send a new pull request for the whole
thing. Would that be ok for you ?

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] [pull request] Pull request for branch 2010-05-15-fixes
  2010-05-18 12:16   ` Thomas Petazzoni
@ 2010-05-18 12:20     ` Peter Korsgaard
  0 siblings, 0 replies; 23+ messages in thread
From: Peter Korsgaard @ 2010-05-18 12:20 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

Hi,

 >> Could you fix that and send a new pull request, please?

 Thomas> Argh, yes, of course. In fact, it's because I started my second branch
 Thomas> on top of the first, because some of the fixes of the second branch
 Thomas> depend on fixes on the first branch.

 Thomas> The easiest is probably that I merge both branches, get rid of the
 Thomas> unwanted Avahi IPv6 fix, and send a new pull request for the whole
 Thomas> thing. Would that be ok for you ?

Yes, that would be perfect - Thanks.

It would be even better if you fix the minor comments I had while
you're at it, but I can do that as well otherwise.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [PATCH 09/12] lvm2: do not override CC and similar variables
  2010-05-18 11:43   ` Peter Korsgaard
@ 2010-05-19 11:24     ` Thomas Petazzoni
  2010-05-19 11:51       ` Peter Korsgaard
  0 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-19 11:24 UTC (permalink / raw)
  To: buildroot

On Tue, 18 May 2010 13:43:14 +0200
Peter Korsgaard <jacmet@uclibc.org> wrote:

> Thanks. Is there anything stopping us from converting it to
> Makefile.autotools.in format?

Probably not. In this particular case, I didn't do it because we are in
the stabilization phase and I didn't want to introduce too much
unneeded changes. For mutt, I had to do it because it actually fixed
some problems.

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 09/12] lvm2: do not override CC and similar variables
  2010-05-19 11:24     ` Thomas Petazzoni
@ 2010-05-19 11:51       ` Peter Korsgaard
  0 siblings, 0 replies; 23+ messages in thread
From: Peter Korsgaard @ 2010-05-19 11:51 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> On Tue, 18 May 2010 13:43:14 +0200
 Thomas> Peter Korsgaard <jacmet@uclibc.org> wrote:

 >> Thanks. Is there anything stopping us from converting it to
 >> Makefile.autotools.in format?

 Thomas> Probably not. In this particular case, I didn't do it because
 Thomas> we are in the stabilization phase and I didn't want to
 Thomas> introduce too much unneeded changes. For mutt, I had to do it
 Thomas> because it actually fixed some problems.

OK.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes
@ 2010-05-21  8:08 Thomas Petazzoni
  2010-05-21 18:38 ` Peter Korsgaard
  0 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2010-05-21  8:08 UTC (permalink / raw)
  To: buildroot

Hello,

Here is a new pull request for the set of fixes I have for
2010.05. This new pull request gathers changes from both of the
previous 2010-05-14-fixes and 2010-05-15-fixes pull requests.

Changes since the previous pull requests made following Peter's
comments:

 * BR2_NEEDS_GETTEXT is now only set to 'y' if the C library doesn't
   contain gettext/libintl *and* BR2_ENABLE_LOCALE is set. This fixes
   the comment of Peter who wanted to compile Avahi without gettext is
   a !BR2_ENABLE_LOCALE case. I've tested this change by compiling
   "ltrace flex libcgicc avahi gmpc grep hal libglib2 libidn libsoup
   make libmpd php php_ext_gettext psmisc util-linux" with an ARM
   internal toolchain with locale enabled, an ARM Buildroot uClibc
   external toolchain with locale disabled and ARM Crosstool-NG eglibc
   external toolchain.

 * Add comment on lmbench requiring RPC support

 * Add comment on netsnmp not working with external toolchains +
   adding the same on BR2_PACKAGE_QUAGGA_SNMP.

The following changes since commit f3122259e2cbbc0be0fc02820a146b4e4f48bef6:
  Peter Korsgaard (1):
        mtd-utils: buildsystem is broken with parallel make (regarding libubi)

are available in the git repository at:

  git://git.busybox.net/~tpetazzoni/git/buildroot 2010-05-15-fixes

Thomas Petazzoni (19):
      Add a BR2_NEEDS_GETTEXT option
      Make all package using gettext rely on BR2_NEEDS_GETTEXT
      libcap: fix build failure
      libcgicc: fix package after autotools infrastructure conversion
      ltrace: fix build failure
      gettext: make only available if BR2_NEEDS_GETTEXT is set
      external toolchain: check BR2_INSTALL_LIBSTDCPP
      iconv: disallow selection when BR2_ENABLE_LOCALE is enabled
      lmbench: needs BR2_INET_RPC
      startup-notification: add dependency on xlib_libX11
      ntp: fix build failure
      samba: only require libiconv if !BR2_ENABLE_LOCALE
      pango: run pango-querymodules on the target instead of on the host
      cairo: remove host variant
      mutt: fix build and convert to autotools
      lvm2: do not override CC and similar variables
      uemacs: don't forget CFLAGS at link time
      vpnc: pass CFLAGS at link time
      netsnmp: disable build with external toolchain

 package/avahi/Config.in                            |    4 +-
 package/avahi/avahi.mk                             |    2 +-
 package/cairo/cairo.mk                             |    3 -
 package/gettext/Config.in                          |    2 +
 package/gmpc/Config.in                             |    4 +-
 package/gmpc/gmpc.mk                               |    3 +-
 package/grep/Config.in                             |    4 +-
 package/grep/grep.mk                               |    2 +-
 package/hal/Config.in                              |    4 +-
 package/hal/hal.mk                                 |    2 +-
 package/libcap/libcap.mk                           |    4 +-
 ...cgicc-3.2.9-disable-documentation-option.patch} |    0
 package/libcgicc/libcgicc.mk                       |    2 +
 package/libglib2/Config.in                         |    4 +-
 package/libglib2/libglib2.mk                       |    2 +-
 package/libiconv/Config.in                         |    1 +
 package/libidn/libidn.mk                           |    2 +-
 package/libsoup/libsoup.mk                         |    2 +-
 package/lmbench/Config.in                          |    5 ++
 package/ltrace/ltrace.mk                           |    3 +-
 package/lvm2/lvm2.mk                               |    2 +-
 package/make/Config.in                             |    4 +-
 package/make/make.mk                               |    2 +-
 package/multimedia/libmpd/Config.in                |    2 -
 package/mutt/mutt-1.5.16-makedoc-hostcc.patch      |   32 ---------
 package/mutt/mutt-1.5.17+20080114-nodoc.patch      |   21 ++++++
 package/mutt/mutt.mk                               |   73 +++-----------------
 package/netsnmp/Config.in                          |    6 ++
 package/ntp/ntp-4.2.4_p5-nano.patch                |   36 ++++++++++
 package/pango/S25pango                             |   20 ++++++
 package/pango/pango.mk                             |   18 +----
 package/php/Config.ext                             |    2 +-
 package/php/php.mk                                 |    2 +-
 package/psmisc/Config.in                           |    4 +-
 package/psmisc/psmisc.mk                           |    8 +--
 package/quagga/Config.in                           |    4 +
 package/samba/Config.in                            |    2 +-
 package/samba/samba.mk                             |    2 +-
 package/sshfs/Config.in                            |    4 +-
 package/sshfs/sshfs.mk                             |    2 +-
 package/startup-notification/Config.in             |    2 +
 .../startup-notification/startup-notification.mk   |    1 +
 package/uemacs/uemacs.mk                           |    2 +-
 package/util-linux/Config.in                       |    4 +-
 package/util-linux/util-linux.mk                   |    4 +-
 package/vpnc/vpnc.mk                               |    4 +-
 toolchain/Config.in.2                              |    8 ++
 toolchain/external-toolchain/ext-tool.mk           |   12 +++
 48 files changed, 175 insertions(+), 163 deletions(-)
 rename package/libcgicc/{cgicc-3.2.9-disable-documentation-option.patch => libcgicc-3.2.9-disable-documentation-option.patch} (100%)
 delete mode 100644 package/mutt/mutt-1.5.16-makedoc-hostcc.patch
 create mode 100644 package/mutt/mutt-1.5.17+20080114-nodoc.patch
 create mode 100644 package/ntp/ntp-4.2.4_p5-nano.patch
 create mode 100644 package/pango/S25pango

Thanks,
-- 
Thomas Petazzoni

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

* [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes
  2010-05-21  8:08 Thomas Petazzoni
@ 2010-05-21 18:38 ` Peter Korsgaard
  0 siblings, 0 replies; 23+ messages in thread
From: Peter Korsgaard @ 2010-05-21 18:38 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> Hello,
 Thomas> Here is a new pull request for the set of fixes I have for
 Thomas> 2010.05. This new pull request gathers changes from both of the
 Thomas> previous 2010-05-14-fixes and 2010-05-15-fixes pull requests.

Committed, thanks!

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2010-05-21 18:38 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-15 23:15 [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 01/12] iconv: disallow selection when BR2_ENABLE_LOCALE is enabled Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 02/12] lmbench: needs BR2_INET_RPC Thomas Petazzoni
2010-05-18 11:39   ` Peter Korsgaard
2010-05-15 23:15 ` [Buildroot] [PATCH 03/12] startup-notification: add dependency on xlib_libX11 Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 04/12] ntp: fix build failure Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 05/12] samba: only require libiconv if !BR2_ENABLE_LOCALE Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 06/12] pango: run pango-querymodules on the target instead of on the host Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 07/12] cairo: remove host variant Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 08/12] mutt: fix build and convert to autotools Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 09/12] lvm2: do not override CC and similar variables Thomas Petazzoni
2010-05-18 11:43   ` Peter Korsgaard
2010-05-19 11:24     ` Thomas Petazzoni
2010-05-19 11:51       ` Peter Korsgaard
2010-05-15 23:15 ` [Buildroot] [PATCH 10/12] uemacs: don't forget CFLAGS at link time Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 11/12] vpnc: pass " Thomas Petazzoni
2010-05-15 23:15 ` [Buildroot] [PATCH 12/12] netsnmp: disable build with external toolchain Thomas Petazzoni
2010-05-18 11:47   ` Peter Korsgaard
2010-05-18 11:56 ` [Buildroot] [pull request] Pull request for branch 2010-05-15-fixes Peter Korsgaard
2010-05-18 12:16   ` Thomas Petazzoni
2010-05-18 12:20     ` Peter Korsgaard
  -- strict thread matches above, loose matches on Subject: below --
2010-05-21  8:08 Thomas Petazzoni
2010-05-21 18:38 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox