public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH -rt] Disable lpptest on !Linux hosts
@ 2007-07-24 20:51 Deepak Saxena
  2007-07-24 21:03 ` Deepak Saxena
  2007-07-25  7:37 ` Ingo Molnar
  0 siblings, 2 replies; 3+ messages in thread
From: Deepak Saxena @ 2007-07-24 20:51 UTC (permalink / raw)
  To: Thomas Gleixner, Ingo Molnar; +Cc: linux-kernel


Sadly people keep wanting to build kernels on non-Linux hosts
(cygwin & solaris) and testlpp really doesn't like to build on 
those. I have a separate patch to testlpp.c that fixes this, 
but it really makes no sense to build the tool to run on your 
cygwin host as it's meant to be run on Linux with the testlpp
module loaded.

Even this patch isn't really the right solution b/c you really want 
to cross-build the may be cross-building for another architecture from 
Linux you want cross-compile, not host compile but there's no really easy
way to cross-compile a userland binary from the kernel build w/o some 
makefile uglyiness AFAICT.

Is there some sort of -rt userland package this could move to instead 
of being in the kernel itself...?

Signed-off-by: Deepak Saxena <dsaxena@mvista.com>

Index: linux-2.6/scripts/Makefile
===================================================================
--- linux-2.6.orig/scripts/Makefile
+++ linux-2.6/scripts/Makefile
@@ -13,9 +13,12 @@ hostprogs-$(CONFIG_LOGO)         += pnmt
 hostprogs-$(CONFIG_VT)           += conmakehash
 hostprogs-$(CONFIG_PROM_CONSOLE) += conmakehash
 hostprogs-$(CONFIG_IKCONFIG)     += bin2c
+HOST_OS := $(shell uname)
+ifeq ($(HOST_OS),Linux)
 ifdef CONFIG_LPPTEST
 hostprogs-y      += testlpp
 endif
+endif
 
 always		:= $(hostprogs-y) $(hostprogs-m)
 

-- 
Deepak Saxena - dsaxena@plexity.net - http://www.plexity.net 

"Americans think their danger is terrorists. They don't understand 
the terrorists cannot take away habeas corpus, the Bill of Rights, 
the Constitution. ... The terrorists are not anything like the threat 
that we face to the Bill of Rights and the Constitution from our own
government in the name of fighting terrorisim."
  - Paul Craig Roberts

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

* Re: [PATCH -rt] Disable lpptest on !Linux hosts
  2007-07-24 20:51 [PATCH -rt] Disable lpptest on !Linux hosts Deepak Saxena
@ 2007-07-24 21:03 ` Deepak Saxena
  2007-07-25  7:37 ` Ingo Molnar
  1 sibling, 0 replies; 3+ messages in thread
From: Deepak Saxena @ 2007-07-24 21:03 UTC (permalink / raw)
  To: Thomas Gleixner, Ingo Molnar; +Cc: linux-kernel

On Jul 24 2007, at 13:51, Deepak Saxena was caught saying:
> Even this patch isn't really the right solution b/c you really want 
> to cross-build the may be cross-building for another architecture from 
> Linux you want cross-compile, not host compile but there's no really easy

Sigh, not enough sleep. This should read:

Even this patch isn't really the right solution b/c may be cross-building 
for another architecture and you want to cross-compile this for that arch.

(Note that I "cross-compile" for x86 all the time to use a specific
 toolchain for a specific userland configuration).

~Deepak


-- 
Deepak Saxena - dsaxena@plexity.net - http://www.plexity.net 

"Americans think their danger is terrorists. They don't understand 
the terrorists cannot take away habeas corpus, the Bill of Rights, 
the Constitution. ... The terrorists are not anything like the threat 
that we face to the Bill of Rights and the Constitution from our own
government in the name of fighting terrorisim."
  - Paul Craig Roberts

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

* Re: [PATCH -rt] Disable lpptest on !Linux hosts
  2007-07-24 20:51 [PATCH -rt] Disable lpptest on !Linux hosts Deepak Saxena
  2007-07-24 21:03 ` Deepak Saxena
@ 2007-07-25  7:37 ` Ingo Molnar
  1 sibling, 0 replies; 3+ messages in thread
From: Ingo Molnar @ 2007-07-25  7:37 UTC (permalink / raw)
  To: Deepak Saxena; +Cc: Thomas Gleixner, linux-kernel


* Deepak Saxena <dsaxena@plexity.net> wrote:

>  hostprogs-$(CONFIG_IKCONFIG)     += bin2c
> +HOST_OS := $(shell uname)
> +ifeq ($(HOST_OS),Linux)
>  ifdef CONFIG_LPPTEST
>  hostprogs-y      += testlpp
>  endif
> +endif

ok, i've applied this.

	Ingo

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

end of thread, other threads:[~2007-07-25  7:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-24 20:51 [PATCH -rt] Disable lpptest on !Linux hosts Deepak Saxena
2007-07-24 21:03 ` Deepak Saxena
2007-07-25  7:37 ` Ingo Molnar

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