[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check
From: |
Bruno Haible |
Subject: |
Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this |
Date: |
Tue, 5 Jul 2011 11:41:10 +0200 |
User-agent: |
KMail/1.9.9 |
Hi Jim,
Jim Meyering wrote:
> It is trivial to avoid leading TABs in a script
> that emits a Makefile snippet with leading TABs.
> Here's one way that is perhaps minimally invasive:
> The change replaces each explicit leading TAB with 8 leading
> spaces and converts each "cat" to a sed command that performs
> the desired 8-sp-to-TAB mapping.
Thanks for the suggestion. It fixes the "make check" complaint.
But the major problem with tabs in .m4 files is that it can get
broken in the future by someone who accidentally untabifies the entire
file. If we're going to add complexity here, I would much prefer that
it gets rid of this problem.
In other words, do you know how to put a tab into a shell variable,
without writing a tab in the source file? For newline, we write
nl='
'
tab=`printf '\t'`
appears to work. Is it portable?
Bruno
- [PATCH] maint: indent with spaces, not TABs, and add a rule to check this, Jim Meyering, 2011/07/04
- Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this, Bruno Haible, 2011/07/04
- Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this, Jim Meyering, 2011/07/05
- Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this,
Bruno Haible <=
- Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this, Jim Meyering, 2011/07/05
- Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this, Bruno Haible, 2011/07/07
- Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this, Jim Meyering, 2011/07/07
Re: [PATCH] maint: indent with spaces, not TABs, and add a rule to check this, Jim Meyering, 2011/07/05