linux-kernel - Re: [PATCH] Include <asm/termbits.h> from <linux/tty

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Pine.LNX.4.64.0706081211280.1504@localhost.localdomain>
Date:	Fri, 8 Jun 2007 12:15:56 -0400 (EDT)
From:	"Robert P. J. Day" <rpjday@...dspring.com>
To:	Alan Cox <alan@...rguk.ukuu.org.uk>
cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Andrew Morton <akpm@...l.org>
Subject: Re: [PATCH] Include <asm/termbits.h> from <linux/tty_driver.h>.

On Fri, 8 Jun 2007, Alan Cox wrote:

> > if a header file needs a definition/declaration that exists in another
> > header file, it should include that other header file.  any other
> > approach is wrong.
>
> And if you try this in the real world for a project as large as the
> kernel you will die in a mass of unresolvable dependancy loops. The
> current behaviour is quite intentional

what utter nonsense.  if i include tty.h, then i will be including
header files like workqueue.h and mutex.h and perhaps other rubbish i
have absolutely no interest in and don't want to waste my time
parsing.

if i choose to invest an extra minute or two and be more selective and
include only those headers which represent content i actually *need*,
then i have every right to be a bit put out if my build suddenly
fails.

if you don't agree with that patch, fine.  but suggesting that it's
going to result in "a mass of unresolvable dependancy loops" is sheer
laziness.

rday
-- 
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://fsdev.net/wiki/index.php?title=Main_Page
========================================================================
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/