Re: [ptxdist] [PATCH] udev: Use ptxdist kernel headers

2011-08-17 Thread Bernhard Walle
Am 16.08.2011 15:56, schrieb Michael Olbrich:
 On Mon, Aug 15, 2011 at 07:34:58PM +0200, bernh...@bwalle.de wrote:
 From: Bernhard Walle wa...@corscience.de

 When a toolchain without kernel headers (such as the Lite version from
 Codesourcery) or with very old kernel header is used, udev doesn't build
 because videodev.h is missing.

 This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of
 udev like it's done for busybox or barebox.

 Tested on ARM with kernel headers 2.6.36.4 (the version
 OSELAS.Toolchain() is currently using).
 
 You need to select KERNEL_HEADERS as well otherwise udev might be build
 before the kernel headers are available.

But how to express the fact that we don't want to force the kernel
headers to be built. But if they are built, they should be built before
udev.

Does select KERNEL_HEADERS if KERNEL_HEADERS work?


Regards,
Bernhard

-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH] udev: Use ptxdist kernel headers

2011-08-17 Thread Michael Olbrich
On Wed, Aug 17, 2011 at 09:17:02AM +0200, Bernhard Walle wrote:
 Am 16.08.2011 15:56, schrieb Michael Olbrich:
  On Mon, Aug 15, 2011 at 07:34:58PM +0200, bernh...@bwalle.de wrote:
  From: Bernhard Walle wa...@corscience.de
 
  When a toolchain without kernel headers (such as the Lite version from
  Codesourcery) or with very old kernel header is used, udev doesn't build
  because videodev.h is missing.
 
  This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of
  udev like it's done for busybox or barebox.
 
  Tested on ARM with kernel headers 2.6.36.4 (the version
  OSELAS.Toolchain() is currently using).
  
  You need to select KERNEL_HEADERS as well otherwise udev might be build
  before the kernel headers are available.
 
 But how to express the fact that we don't want to force the kernel
 headers to be built. But if they are built, they should be built before
 udev.
 
 Does select KERNEL_HEADERS if KERNEL_HEADERS work?

hmmm, different idea: make kernel-headers a 'base' package. That way it's
built before any non-base package.

menuconfig KERNEL_HEADER
tristate
select BASE
[...]

Michael

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH] udev: Use ptxdist kernel headers

2011-08-16 Thread Michael Olbrich
On Mon, Aug 15, 2011 at 07:34:58PM +0200, bernh...@bwalle.de wrote:
 From: Bernhard Walle wa...@corscience.de
 
 When a toolchain without kernel headers (such as the Lite version from
 Codesourcery) or with very old kernel header is used, udev doesn't build
 because videodev.h is missing.
 
 This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of
 udev like it's done for busybox or barebox.
 
 Tested on ARM with kernel headers 2.6.36.4 (the version
 OSELAS.Toolchain() is currently using).

You need to select KERNEL_HEADERS as well otherwise udev might be build
before the kernel headers are available.

Michael

 Signed-off-by: Bernhard Walle wa...@corscience.de
 ---
  rules/udev.make |5 +
  1 files changed, 5 insertions(+), 0 deletions(-)
 
 diff --git a/rules/udev.make b/rules/udev.make
 index ac2f0d4..b57bd01 100644
 --- a/rules/udev.make
 +++ b/rules/udev.make
 @@ -47,6 +47,11 @@ $(UDEV_SOURCE):
  
  #
  # autoconf
 +
 +UDEV_CONF_ENV := \
 + $(CROSS_ENV) \
 + CPPFLAGS=-I$(KERNEL_HEADERS_INCLUDE_DIR) $(CROSS_CPPFLAGS)
 +
  #
  # Note: explicit sbindir to avoid ${exec_prefix}/sbin/ on the target.
  UDEV_AUTOCONF := \
 -- 
 1.7.6
 
 
 -- 
 ptxdist mailing list
 ptxdist@pengutronix.de
 

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

-- 
ptxdist mailing list
ptxdist@pengutronix.de


[ptxdist] [PATCH] udev: Use ptxdist kernel headers

2011-08-15 Thread bernhard
From: Bernhard Walle wa...@corscience.de

When a toolchain without kernel headers (such as the Lite version from
Codesourcery) or with very old kernel header is used, udev doesn't build
because videodev.h is missing.

This patch adds the $(KERNEL_HEADERS_INCLUDE_DIR) to the include path of
udev like it's done for busybox or barebox.

Tested on ARM with kernel headers 2.6.36.4 (the version
OSELAS.Toolchain() is currently using).

Signed-off-by: Bernhard Walle wa...@corscience.de
---
 rules/udev.make |5 +
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/rules/udev.make b/rules/udev.make
index ac2f0d4..b57bd01 100644
--- a/rules/udev.make
+++ b/rules/udev.make
@@ -47,6 +47,11 @@ $(UDEV_SOURCE):
 
 #
 # autoconf
+
+UDEV_CONF_ENV := \
+   $(CROSS_ENV) \
+   CPPFLAGS=-I$(KERNEL_HEADERS_INCLUDE_DIR) $(CROSS_CPPFLAGS)
+
 #
 # Note: explicit sbindir to avoid ${exec_prefix}/sbin/ on the target.
 UDEV_AUTOCONF := \
-- 
1.7.6


-- 
ptxdist mailing list
ptxdist@pengutronix.de