Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Paul Eggert
On 01/23/2012 09:28 AM, Eric Blake wrote: > we _still_ want INSTALL.UTF-8 output, with the nicer > balanced ‘\1’ quoting. Yes, that should be part of any improvement to makeinfo. To be honest, I thought makeinfo already did that, if one used the proper combinations of @documentlanguage, @documente

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Eric Blake
On 01/23/2012 10:23 AM, Paul Eggert wrote: > On 01/23/2012 09:07 AM, Eric Blake wrote: >> given our recent push to use '' quoting everywhere, >> should we be ditching INSTALL.ISO and make INSTALL use '' >> quoting? > > Yes indeed. I've been meaning to suggest the same thing. > >> Should we be us

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Paul Eggert
On 01/23/2012 09:07 AM, Eric Blake wrote: > given our recent push to use '' quoting everywhere, > should we be ditching INSTALL.ISO and make INSTALL use '' > quoting? Yes indeed. I've been meaning to suggest the same thing. > Should we be using '@set txicodequotebacktick' or similar > markup in

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-23 Thread Eric Blake
[adding autoconf] On 01/21/2012 09:03 AM, Jim Meyering wrote: > Bruno Haible wrote: >> >>> I too would prefer to use Paul's patch and thus to remove the >>> doc/INSTALL* exemptions. >>> >>> Bruno, I know it's not your preference, but would that be ok with you. >> >> My objections were mostly esthe

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Jim Meyering
Bruno Haible wrote: > >> I too would prefer to use Paul's patch and thus to remove the >> doc/INSTALL* exemptions. >> >> Bruno, I know it's not your preference, but would that be ok with you. > > My objections were mostly esthetic, hence minor. You can decide to > ignore my esthetic preferences. T

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Bruno Haible
Hi Jim, > I too would prefer to use Paul's patch and thus to remove the > doc/INSTALL* exemptions. > > Bruno, I know it's not your preference, but would that be ok with you. My objections were mostly esthetic, hence minor. You can decide to ignore my esthetic preferences. Bruno

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Jim Meyering
Jim Meyering wrote: > Eric Blake wrote: >> On 01/17/2012 11:44 AM, Jim Meyering wrote: >> > * doc/INSTALL: Likewise. > * doc/INSTALL.ISO: Likewise. > * doc/INSTALL.UTF-8: Likewise. These files are generated through doc/Makefile. Also, on text files that consist of multip

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Jim Meyering
Eric Blake wrote: > On 01/17/2012 11:44 AM, Jim Meyering wrote: > * doc/INSTALL: Likewise. * doc/INSTALL.ISO: Likewise. * doc/INSTALL.UTF-8: Likewise. >>> >>> These files are generated through doc/Makefile. Also, on text files >>> that consist of multiple paragraphs, I find it good (

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-21 Thread Eric Blake
On 01/17/2012 11:44 AM, Jim Meyering wrote: >>> * doc/INSTALL: Likewise. >>> * doc/INSTALL.ISO: Likewise. >>> * doc/INSTALL.UTF-8: Likewise. >> >> These files are generated through doc/Makefile. Also, on text files >> that consist of multiple paragraphs, I find it good (esthetic) to have >> a blan

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-18 Thread Jim Meyering
Karl Berry wrote: > However, I am optimistic that I will be able to make matching > changes upstream. > > Sorry, but for myself, I think it is a terrible waste of time to be > thinking about this for fdl*.texi. The blank lines don't hurt anything Hi Karl, I know this hurts your sensibili

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-18 Thread Jim Meyering
Bruno Haible wrote: > Paul Eggert wrote: >> The "cat INSTALL" argument is not that persuasive > > The other argument is that when viewing a file with "vi" and scrolling > through it, the cursor interrupts the reading flow when it is placed > on the last line and that line is not a blank one. > > Bu

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Bruno Haible
Paul Eggert wrote: > The "cat INSTALL" argument is not that persuasive The other argument is that when viewing a file with "vi" and scrolling through it, the cursor interrupts the reading flow when it is placed on the last line and that line is not a blank one. But of course, these arguments are

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Paul Eggert
On 01/17/12 10:44, Jim Meyering wrote: > Exempting three more is no problem. It's no big deal but I'd rather omit the trailing empty line. The "cat INSTALL" argument is not that persuasive, as the sort of person who types "cat INSTALL" or "more INSTALL" is already used to files that don't end in

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Karl Berry
However, I am optimistic that I will be able to make matching changes upstream. Sorry, but for myself, I think it is a terrible waste of time to be thinking about this for fdl*.texi. The blank lines don't hurt anything and you're not supposed to be modifying those files, so can't we just

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Bruno Haible
Jim Meyering wrote: > >> * doc/INSTALL: Likewise. > >> * doc/INSTALL.ISO: Likewise. > >> * doc/INSTALL.UTF-8: Likewise. > Exempting three more is no problem. Thanks! Bruno

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Jim Meyering
Bruno Haible wrote: > Hi Jim, > >> * doc/regexprops-generic.texi: Add a newline at EOF. There was none. > > Good point. It was more of a problem with 'cvs' than with 'git'. But I > like this effort anyway. > >> * doc/Copyright/assign.translation.manual: Remove empty lines at EOF. >> * doc/Copyrig

Re: [PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Bruno Haible
Hi Jim, > * doc/regexprops-generic.texi: Add a newline at EOF. There was none. Good point. It was more of a problem with 'cvs' than with 'git'. But I like this effort anyway. > * doc/Copyright/assign.translation.manual: Remove empty lines at EOF. > * doc/Copyright/request-assign.future: Likewis

[PATCH 2/8] maint: remove empty lines at EOF, but excluding modules/*

2012-01-17 Thread Jim Meyering
From: Jim Meyering Apply syntax rules at home as well as abroad. Most changes were induced by running this: make srcdir=. _build-aux=build-aux -f top/maint.mk \ sc_prohibit_empty_lines_at_EOF | grep -v modules/ \ | xargs perl -pi -0777 -e 's/\n\n+$/\n/' * cfg.mk (local-checks-to-skip):