linux-kbuild.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] kbuild: remove old information in headers_install.txt document
@ 2014-12-02  4:28 Masahiro Yamada
  2014-12-02 13:39 ` Jonathan Corbet
  2014-12-05 14:23 ` Yann Droneaud
  0 siblings, 2 replies; 4+ messages in thread
From: Masahiro Yamada @ 2014-12-02  4:28 UTC (permalink / raw)
  To: linux-kbuild
  Cc: Masahiro Yamada, Michal Marek, Jonathan Corbet, linux-doc, linux-kernel

The arch header directories "include/asm-*" were moved long before.

Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
---

 Documentation/kbuild/headers_install.txt | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
index 951eb9f..0d8fe5e 100644
--- a/Documentation/kbuild/headers_install.txt
+++ b/Documentation/kbuild/headers_install.txt
@@ -28,10 +28,7 @@ optional arguments:
 
 ARCH indicates which architecture to produce headers for, and defaults to the
 current architecture.  The linux/asm directory of the exported kernel headers
-is platform-specific, to see a complete list of supported architectures use
-the command:
-
-  ls -d include/asm-* | sed 's/.*-//'
+is platform-specific.
 
 INSTALL_HDR_PATH indicates where to install the headers.  It defaults to
 "./usr/include".
-- 
1.9.1


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

* Re: [PATCH] kbuild: remove old information in headers_install.txt document
  2014-12-02  4:28 [PATCH] kbuild: remove old information in headers_install.txt document Masahiro Yamada
@ 2014-12-02 13:39 ` Jonathan Corbet
  2014-12-05 14:23 ` Yann Droneaud
  1 sibling, 0 replies; 4+ messages in thread
From: Jonathan Corbet @ 2014-12-02 13:39 UTC (permalink / raw)
  To: Masahiro Yamada; +Cc: linux-kbuild, Michal Marek, linux-doc, linux-kernel

On Tue,  2 Dec 2014 13:28:53 +0900
Masahiro Yamada <yamada.m@jp.panasonic.com> wrote:

> diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
> index 951eb9f..0d8fe5e 100644
> --- a/Documentation/kbuild/headers_install.txt
> +++ b/Documentation/kbuild/headers_install.txt
> @@ -28,10 +28,7 @@ optional arguments:
>  
>  ARCH indicates which architecture to produce headers for, and defaults to the
>  current architecture.  The linux/asm directory of the exported kernel headers
> -is platform-specific, to see a complete list of supported architectures use
> -the command:
> -
> -  ls -d include/asm-* | sed 's/.*-//'
> +is platform-specific.
>  

This change makes sense as far as it goes, but might it not be better to
tell readers the way to get the list of supported architectures in the
new scheme of things...?

Thanks,

jon

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

* Re: [PATCH] kbuild: remove old information in headers_install.txt document
  2014-12-02  4:28 [PATCH] kbuild: remove old information in headers_install.txt document Masahiro Yamada
  2014-12-02 13:39 ` Jonathan Corbet
@ 2014-12-05 14:23 ` Yann Droneaud
  2014-12-11  1:41   ` Masahiro Yamada
  1 sibling, 1 reply; 4+ messages in thread
From: Yann Droneaud @ 2014-12-05 14:23 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: linux-kbuild, Michal Marek, Jonathan Corbet, linux-doc,
	linux-kernel, Sam Ravnborg, Yann Droneaud

Hi,

Le mardi 02 décembre 2014 à 13:28 +0900, Masahiro Yamada a écrit :
> The arch header directories "include/asm-*" were moved long before.
> 
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> ---
> 
>  Documentation/kbuild/headers_install.txt | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
> index 951eb9f..0d8fe5e 100644
> --- a/Documentation/kbuild/headers_install.txt
> +++ b/Documentation/kbuild/headers_install.txt
> @@ -28,10 +28,7 @@ optional arguments:
>  
>  ARCH indicates which architecture to produce headers for, and defaults to the
>  current architecture.  The linux/asm directory of the exported kernel headers
> -is platform-specific, to see a complete list of supported architectures use
> -the command:
> -
> -  ls -d include/asm-* | sed 's/.*-//'
> +is platform-specific.
>  
>  INSTALL_HDR_PATH indicates where to install the headers.  It defaults to
>  "./usr/include".

Back in september, I've already sent a patch[1] to address this,
following a discussion with Sam Ravnborg[2]. See the mail thread[3].

Regards.

[1] http://lkml.kernel.org/r/b5181cf7604baa454c11f7aa92d07dd05349ce46.1410712841.git.ydroneaud@opteya.com
[2] http://lkml.kernel.org/r/20140713212608.GB16496@ravnborg.org
[3] http://lkml.kernel.org/g/cover.1410712841.git.ydroneaud@opteya.com

-- 
Yann Droneaud
OPTEYA




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

* Re: [PATCH] kbuild: remove old information in headers_install.txt document
  2014-12-05 14:23 ` Yann Droneaud
@ 2014-12-11  1:41   ` Masahiro Yamada
  0 siblings, 0 replies; 4+ messages in thread
From: Masahiro Yamada @ 2014-12-11  1:41 UTC (permalink / raw)
  To: Yann Droneaud
  Cc: linux-kbuild, Michal Marek, Jonathan Corbet, linux-doc,
	linux-kernel, Sam Ravnborg

Hi Yann,






On Fri, 05 Dec 2014 15:23:48 +0100
Yann Droneaud <ydroneaud@opteya.com> wrote:

> Hi,
> 
> Le mardi 02 decembre 2014 a 13:28 +0900, Masahiro Yamada a ecrit :
> > The arch header directories "include/asm-*" were moved long before.
> > 
> > Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > ---
> > 
> >  Documentation/kbuild/headers_install.txt | 5 +----
> >  1 file changed, 1 insertion(+), 4 deletions(-)
> > 
> > diff --git a/Documentation/kbuild/headers_install.txt b/Documentation/kbuild/headers_install.txt
> > index 951eb9f..0d8fe5e 100644
> > --- a/Documentation/kbuild/headers_install.txt
> > +++ b/Documentation/kbuild/headers_install.txt
> > @@ -28,10 +28,7 @@ optional arguments:
> >  
> >  ARCH indicates which architecture to produce headers for, and defaults to the
> >  current architecture.  The linux/asm directory of the exported kernel headers
> > -is platform-specific, to see a complete list of supported architectures use
> > -the command:
> > -
> > -  ls -d include/asm-* | sed 's/.*-//'
> > +is platform-specific.
> >  
> >  INSTALL_HDR_PATH indicates where to install the headers.  It defaults to
> >  "./usr/include".
> 
> Back in september, I've already sent a patch[1] to address this,
> following a discussion with Sam Ravnborg[2]. See the mail thread[3].
> 
> Regards.
> 
> [1] http://lkml.kernel.org/r/b5181cf7604baa454c11f7aa92d07dd05349ce46.1410712841.git.ydroneaud@opteya.com
> [2] http://lkml.kernel.org/r/20140713212608.GB16496@ravnborg.org
> [3] http://lkml.kernel.org/g/cover.1410712841.git.ydroneaud@opteya.com
> 

Sorry, I missed your patch.

You are the first, so I agree yours should go in.



Best Regards
Masahiro Yamada


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

end of thread, other threads:[~2014-12-11  1:43 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-12-02  4:28 [PATCH] kbuild: remove old information in headers_install.txt document Masahiro Yamada
2014-12-02 13:39 ` Jonathan Corbet
2014-12-05 14:23 ` Yann Droneaud
2014-12-11  1:41   ` Masahiro Yamada

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).