From: Keith Owens <kaos@ocs.com.au>
To: kbuild-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org
Subject: Re: [kbuild-devel] Re: Announce: Kernel Build for 2.5, release 3.0 is available
Date: Wed, 05 Jun 2002 12:38:18 +1000 [thread overview]
Message-ID: <20890.1023244698@kao2.melbourne.sgi.com> (raw)
In-Reply-To: Your message of "Tue, 04 Jun 2002 22:25:20 -0400." <200206050224.WAA00121@mail.reutershealth.com>
On Tue, 4 Jun 2002 22:25:20 -0400 (EDT),
John Cowan <jcowan@reutershealth.com> wrote:
>Keith Owens scripsit:
>
>> In order to do separate source and object correctly, kbuild 2.5
>> enforces the rule that #include "" comes from the local directory,
>> #include <> comes from the include path. include/linux/zlib.h
>> incorrectly does #include "zconf.h" instead of #include <linux/zconf.h>,
>> breaking the rules.
>
>This is not the standard gcc behavior, however; quoted-includes
>can come from the include path, although the current directory
>is searched first. The purpose of <>-includes is to suppress
>searching the current directory.
What gcc allows and what the kernel uses as a coding style are two
different things. Almost all of the kernel uses <> for global files
and "" for local files, this is the only sane way of coding for a large
project. However there are some exceptions where the wrong form has
been used.
The wrong form causes problems for separate source and object
directories. It also causes problems when you do not compile in the
same directory that does not contain the source, i.e. when you do a
global make instead of recursive make. kbuild 2.5 has identified all
of the problem includes. I avoided changing the source, instead I
added extra include paths as a temporary workaround, with FIXME
comments for later clean up. I noticed that some people have already
used the kbuild 2.5 FIXME comments to clean up their code.
next prev parent reply other threads:[~2002-06-05 2:38 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-06-03 2:35 Announce: Kernel Build for 2.5, release 3.0 is available Keith Owens
2002-06-03 5:31 ` Keith Owens
2002-06-03 19:22 ` [kbuild-devel] " Thomas Duffy
2002-06-03 19:31 ` Thunder from the hill
2002-06-03 19:58 ` Thomas Duffy
2002-06-03 20:06 ` Thunder from the hill
2002-06-03 21:12 ` Thomas Duffy
2002-06-04 0:19 ` Keith Owens
2002-06-04 0:29 ` Thomas Duffy
2002-06-04 4:53 ` Keith Owens
2002-06-04 9:16 ` Alex Riesen
2002-06-04 22:55 ` Keith Owens
2002-06-05 2:25 ` [kbuild-devel] " John Cowan
2002-06-05 2:38 ` Keith Owens [this message]
2002-06-05 18:13 ` Mark Mielke
2002-06-05 13:19 ` Alex Riesen
2002-06-04 13:59 ` [kbuild-devel] " Arnd Bergmann
2002-06-05 13:53 ` Keith Owens
2002-06-07 8:49 ` Keith Owens
2002-06-12 13:54 ` Keith Owens
2002-06-12 20:55 ` [kbuild-devel] " Thomas Duffy
2002-06-13 0:15 ` Keith Owens
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20890.1023244698@kao2.melbourne.sgi.com \
--to=kaos@ocs.com.au \
--cc=kbuild-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox