public inbox for linux-kbuild@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] Make it possible to override INSTALL_HDR_PATH
@ 2011-05-03 21:46 Arun Sharma
  2011-05-03 22:00 ` Yann E. MORIN
  0 siblings, 1 reply; 3+ messages in thread
From: Arun Sharma @ 2011-05-03 21:46 UTC (permalink / raw)
  To: linux-kbuild

diff --git a/Makefile b/Makefile
index cb715df..ec68886 100644
--- a/Makefile
+++ b/Makefile
@@ -1009,7 +1009,7 @@ firmware_install: FORCE
 # Kernel headers
 
 #Default location for installed headers
-export INSTALL_HDR_PATH = $(objtree)/usr
+export INSTALL_HDR_PATH ?= $(objtree)/usr
 
 hdr-inst := -rR -f $(srctree)/scripts/Makefile.headersinst obj
 
-- 
1.7.4

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

* Re: [PATCH] Make it possible to override INSTALL_HDR_PATH
  2011-05-03 21:46 [PATCH] Make it possible to override INSTALL_HDR_PATH Arun Sharma
@ 2011-05-03 22:00 ` Yann E. MORIN
  2011-05-03 22:34   ` Arun Sharma
  0 siblings, 1 reply; 3+ messages in thread
From: Yann E. MORIN @ 2011-05-03 22:00 UTC (permalink / raw)
  To: linux-kbuild; +Cc: Arun Sharma

Arun, All,

On Tuesday 03 May 2011 23:46:12 Arun Sharma wrote:
> diff --git a/Makefile b/Makefile
> index cb715df..ec68886 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1009,7 +1009,7 @@ firmware_install: FORCE
>  # Kernel headers
>  
>  #Default location for installed headers
> -export INSTALL_HDR_PATH = $(objtree)/usr
> +export INSTALL_HDR_PATH ?= $(objtree)/usr
>  
>  hdr-inst := -rR -f $(srctree)/scripts/Makefile.headersinst obj
>  

Already works for me here in 2.6.38.5:
  $ head -n 5 Makefile
  VERSION = 2
  PATCHLEVEL = 6
  SUBLEVEL = 38
  EXTRAVERSION = .5
  NAME = Flesh-Eating Bats with Fangs
  $ rm -rf FOO
  $ mkdir FOO
  $ make INSTALL_HDR_PATH=$(pwd)/FOO ARCH=x86 headers_install
  $ ls -l FOO
  total 4
  drwxr-xr-x 12 ymorin users 4096 2011-05-03 23:57 include
  $ cat FOO/include/linux/version.h
  #define LINUX_VERSION_CODE 132646
  #define KERNEL_VERSION(a,b,c) (((a) << 16) + ((b) << 8) + (c))
  $ make --version | head -n 1
  GNU Make 3.81

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

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

* Re: [PATCH] Make it possible to override INSTALL_HDR_PATH
  2011-05-03 22:00 ` Yann E. MORIN
@ 2011-05-03 22:34   ` Arun Sharma
  0 siblings, 0 replies; 3+ messages in thread
From: Arun Sharma @ 2011-05-03 22:34 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: linux-kbuild, Arun Sharma

On Wed, May 04, 2011 at 12:00:29AM +0200, Yann E. MORIN wrote:
>   $ make INSTALL_HDR_PATH=$(pwd)/FOO ARCH=x86 headers_install
>   $ ls -l FOO

Works for me too :) I was trying:

echo 'INSTALL_HDR_PATH=$RPM_BUILD_ROOT/usr make %{?_smp_mflags} headers_install'

along the lines of:

scripts/packaging/mkspec:

echo 'INSTALL_MOD_PATH=$RPM_BUILD_ROOT make %{?_smp_mflags} KBUILD_SRC=modules_install'

While we're on the topic, could we enhance mkspec to spit out -devel and
-headers rpms too? I can post patches if there is sufficient interest.

It'd be nice to use the same mechanism to define/override:

INSTALL_PATH
INSTALL_MOD_PATH
INSTALL_HDR_PATH

 -Arun

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

end of thread, other threads:[~2011-05-03 22:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-05-03 21:46 [PATCH] Make it possible to override INSTALL_HDR_PATH Arun Sharma
2011-05-03 22:00 ` Yann E. MORIN
2011-05-03 22:34   ` Arun Sharma

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