All of lore.kernel.org
 help / color / mirror / Atom feed
* Set INSTALL_HDR_PATH on command line
@ 2010-04-22  7:23 Jonas Bonn
  2010-04-22 16:45 ` Sam Ravnborg
  0 siblings, 1 reply; 2+ messages in thread
From: Jonas Bonn @ 2010-04-22  7:23 UTC (permalink / raw)
  To: linux-kbuild

[-- Attachment #1: Type: text/plain, Size: 460 bytes --]

In the toplevel Linux Makefile, INSTALL_HDR_PATH is hardcoded as
follows:

# Kernel headers

#Default location for installed headers
export INSTALL_HDR_PATH = $(objtree)/usr

Should this not be done with ?= so that the install path can be
overridden on the command line?

export INSTALL_HDR_PATH ?= $(objtree)/usr

This would make installation of headers for cross-build environments
simpler.  Or am I missing something else?

Regards,
Jonas

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

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

* Re: Set INSTALL_HDR_PATH on command line
  2010-04-22  7:23 Set INSTALL_HDR_PATH on command line Jonas Bonn
@ 2010-04-22 16:45 ` Sam Ravnborg
  0 siblings, 0 replies; 2+ messages in thread
From: Sam Ravnborg @ 2010-04-22 16:45 UTC (permalink / raw)
  To: Jonas Bonn; +Cc: linux-kbuild

On Thu, Apr 22, 2010 at 09:23:28AM +0200, Jonas Bonn wrote:
> In the toplevel Linux Makefile, INSTALL_HDR_PATH is hardcoded as
> follows:
> 
> # Kernel headers
> 
> #Default location for installed headers
> export INSTALL_HDR_PATH = $(objtree)/usr
> 
> Should this not be done with ?= so that the install path can be
> overridden on the command line?
> 
> export INSTALL_HDR_PATH ?= $(objtree)/usr

What you can do today is:

    make INSTALL_HDR_PATH=/my/dir


Difference between "=" and "?=" is that the latter
only assign a new value if the variables was not defined.

	Sam

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

end of thread, other threads:[~2010-04-22 17:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-04-22  7:23 Set INSTALL_HDR_PATH on command line Jonas Bonn
2010-04-22 16:45 ` Sam Ravnborg

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.