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]
Date:	Thu, 2 Nov 2006 21:16:50 +0100
From:	Sam Ravnborg <sam@...nborg.org>
To:	LKML <linux-kernel@...r.kernel.org>,
	Oleg Verych <olecom@...wer.upol.cz>,
	Jesper Juhl <jesper.juhl@...il.com>, trivial@...nel.org
Subject: Re: [patch] make the Makefile mostly stay within col 80

On Thu, Nov 02, 2006 at 07:16:12AM +0000, Oleg Verych wrote:
> 
> On 2006-11-01, Jesper Juhl wrote:
> > Trivial little thing really. 
> > Try to make most of the Makefile obey the 80 column width rule.
> 
> I'm already working on it. I did a lot more stuff, but currently i'm
> stuck with very first patch, i've tried to push to mister Andrew:
> <http://marc.theaimsgroup.com/?l=linux-mm-commits&m=116198944205036&w=2>
> 
> As i'm using emacs, i cann't revert this open/save/close patch every
> time. If someone with RH-based distro is willing to help, i'll be glad.
> Version of make is Red Hat make-3.80-10.2.
> 
> Also, i want Sam Ravnborg to comment on that effort (e-mail added). Thanks.

Most of the time I spent on Linux development is in a 80xsomething so
I support the effort to make it fit into 80 coloumn.
But only if done sensible and not as a hard rule. Some stuff really
is less readable if it is adjusted to fit into a 80 coloumn.

I do not support tabifying the Makefiles. In a makefile <tab> has
a special interpretation and the rule of thumb is:

1) Use tab to indent commands as make requires it
2) Commands spanning more than one line may be indented with tabs.

Avoid tabs in all other places.

This is not the same ruleset as used in the .c source but the difference
here is that an assignment is not turned into a command in .c code
just because it is prefixed by a tab.

At present I'm fed up with day time job that is almost around the clock
time job. It will take a month before I will be active in kbuild area
again so please be patient. If something really urgent shows up I
can act - but just not much time to do so in.

	Sam

-
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/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ