From: Sam Ravnborg <sam@ravnborg.org>
To: LKML <linux-kernel@vger.kernel.org>,
Oleg Verych <olecom@flower.upol.cz>,
Jesper Juhl <jesper.juhl@gmail.com>,
trivial@kernel.org
Subject: Re: [patch] make the Makefile mostly stay within col 80
Date: Thu, 2 Nov 2006 21:16:50 +0100 [thread overview]
Message-ID: <20061102201650.GA9237@uranus.ravnborg.org> (raw)
In-Reply-To: <slrnekj6ps.2in.olecom@flower.upol.cz>
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
next prev parent reply other threads:[~2006-11-02 20:16 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-01 23:47 [patch] make the Makefile mostly stay within col 80 Jesper Juhl
2006-11-02 7:09 ` Oleg Verych
2006-11-02 20:16 ` Sam Ravnborg [this message]
2006-11-03 16:35 ` Oleg Verych
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=20061102201650.GA9237@uranus.ravnborg.org \
--to=sam@ravnborg.org \
--cc=jesper.juhl@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=olecom@flower.upol.cz \
--cc=trivial@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