linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Bootconfig ..too many deep level of header file
@ 2021-03-21  3:08 Bhaskar Chowdhury
  2021-03-21 13:17 ` Masami Hiramatsu
  0 siblings, 1 reply; 2+ messages in thread
From: Bhaskar Chowdhury @ 2021-03-21  3:08 UTC (permalink / raw)
  To: mhiramat; +Cc: LinuxKernel

 Hey, Masami

 I was wondering why so many level of "deep nesting" of a particualr header
 file like this one :

 ✔ ~/git-linux/linux/tools/bootconfig/include/linux [patch L|✔]
08:34 $ cat bootconfig.h
/* SPDX-License-Identifier: GPL-2.0 */
#ifndef _BOOTCONFIG_LINUX_BOOTCONFIG_H
#define _BOOTCONFIG_LINUX_BOOTCONFIG_H

#include "../../../../include/linux/bootconfig.h"

#endif

I am surely missing some basic stuff...but...anyway, I have taken a peek on other files in the same dir and none have the nesting like that(the include ../ ...line)

Any specific reason? OR did you thought of keeping all the header file in one paplace(wildly guessing) ...


~Bhaskar

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

* Re: Bootconfig ..too many deep level of header file
  2021-03-21  3:08 Bootconfig ..too many deep level of header file Bhaskar Chowdhury
@ 2021-03-21 13:17 ` Masami Hiramatsu
  0 siblings, 0 replies; 2+ messages in thread
From: Masami Hiramatsu @ 2021-03-21 13:17 UTC (permalink / raw)
  To: Bhaskar Chowdhury; +Cc: LinuxKernel

Hi Bhaskar,

On Sun, 21 Mar 2021 08:38:56 +0530
Bhaskar Chowdhury <unixbhaskar@gmail.com> wrote:

>  Hey, Masami
> 
>  I was wondering why so many level of "deep nesting" of a particualr header
>  file like this one :
> 
>  ✔ ~/git-linux/linux/tools/bootconfig/include/linux [patch L|✔]
> 08:34 $ cat bootconfig.h
> /* SPDX-License-Identifier: GPL-2.0 */
> #ifndef _BOOTCONFIG_LINUX_BOOTCONFIG_H
> #define _BOOTCONFIG_LINUX_BOOTCONFIG_H
> 
> #include "../../../../include/linux/bootconfig.h"
> 
> #endif
> 
> I am surely missing some basic stuff...but...anyway, I have taken a peek on other files in the same dir and none have the nesting like that(the include ../ ...line)
> 
> Any specific reason? OR did you thought of keeping all the header file in one paplace(wildly guessing) ...

If you carefully check the number of ../, you can find that it refers the
 <topdir>/include/linux/bootconfig.h.
Since the bootconfig library code needs bootconfig.h but I don't like to make
a copy of the bootconfig.h under the tools directory, I made a dummy header
file which links to the kernel's bootconfig.h.

Thank you,

-- 
Masami Hiramatsu <mhiramat@kernel.org>

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

end of thread, other threads:[~2021-03-21 13:18 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-21  3:08 Bootconfig ..too many deep level of header file Bhaskar Chowdhury
2021-03-21 13:17 ` Masami Hiramatsu

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).