patchgrep - Patches: patch #4610, Consolidated documentation patch

 
 

You are not allowed to post comments on this tracker with your current authentification level.

patch #4610: Consolidated documentation patch

Submitted by:  Charles Levert <charles_levert>
Submitted on:  Fri 11 Nov 2005 05:07:45 AM UTC  
 
Category: NonePriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: NoneOpen/Closed: Closed

(Jump to the original submission Jump to the original submission)

Fri 18 Aug 2006 09:40:30 PM UTC, comment #7:

Looks good -- applied.
Thanks.

Bernhard Rosenkraenzer <bero>
Project Administrator
Thu 18 May 2006 09:56:59 PM UTC, comment #6:

This patch and file should go in before a release, in my opinion, as they describe grep's new colouring possibilities. I myself don't like the man page to be structured (preferring to see the options in strict alphabetical order), but let's wait what the users say. A few suggested changes to the wording of the texinfo patch:

or does whatever --> or gives whatever / or produces whatever

The general synopsis of a --> ...of the

and there can --> There can

summarizing these --> summarizing the

behaved like @sc{gnu} @command{grep}. --> ...@command{grep}'s.

minimum size field width --> minimum-size field width

no effect unless --> no effect unless the

Skip files @emph{and directories} --> Skip files

mentioned in the command line --> mentioned on...

the text terminal that is used --> your text terminal

Furthermore, instead of things like "@item mt=01;31", it should say "@item mt=" and add the default value between parentheses in
subsequent text: "... The default (01;31) is a bold red text..."

I hate the "(`-...' is specified by POSIX)" things.

Not in the patch, but in my opinion mistakenly described:

`[:' represents the open character class followed by a valid character class name. --> `[:' represents the open character class symbol, and should be followed by a valid character class name.

`:]' represents the close character class followed by a valid character class name. --> `:]' represents the close character class symbol.

There is something wrong with the Concept Index and the Index: the first wraps to section 2.1.5 (after context), the last to section 5.1 (Fundamental Structure). I would expect the first to wrap to the Index, and the last to wrap to the very start of manual, or to not wrap at all. They apparently jump to the first entry in their list when scrolling beyond the end. Maybe this is a generic info bug?

Benno Schulenberg <bens>
Fri 18 Nov 2005 06:44:40 PM UTC, comment #5:

TeXinfo manual, version 4, unidiff output.

Charles Levert <charles_levert>
Fri 18 Nov 2005 06:43:19 PM UTC, comment #4:

Man page, version 4, whole file.

Charles Levert <charles_levert>
Fri 18 Nov 2005 09:50:30 AM UTC, comment #3:

TeXinfo manual, version 3. I attached a unidiff output file.

Check out dvi and info.

Charles Levert <charles_levert>
Fri 18 Nov 2005 09:47:02 AM UTC, comment #2:

Man page, version 3. I attached the whole file because it's now smaller than diff output.

Check out "groff -Tascii" and "groff -Tps".

Charles Levert <charles_levert>
Sat 12 Nov 2005 12:19:58 PM UTC, comment #1:

Version 2. See ChangeLog. Again, check out dvi and info.

Charles Levert <charles_levert>
Fri 11 Nov 2005 05:07:45 AM UTC, original submission:

Here is what I was working on, as promised.

ChangeLog included.

Try to view both the dvi and info output of grep.texi to get a good idea of the structuring work that was done, and of the extra explanations that this enabled.

Charles Levert <charles_levert>

 

Attached Files
file #5799:  grep.texi.diff4 added by charles_levert (75kB - application/octet-stream - TeXinfo manual, unidiff output, version 4)
file #5824:  grep.1 added by charles_levert (29kB - text/x-troff-man - grep.1, whole file, version 4)
file #5841:  grep.texi.diff3 added by charles_levert (74kB - application/octet-stream - TeXinfo manual, unidiff output, version 3)
file #5840:  grep.1 added by charles_levert (29kB - text/x-troff-man - grep.1, whole file, version 3)
file #5969:  doc2.patch added by charles_levert (60kB - text/x-patch - "cvs diff -u" patch version 2)
file #6323:  doc.patch added by charles_levert (36kB - text/x-patch - "cvs diff -u" patch version 1)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 9 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Fri 18 Aug 2006 09:40:30 PM UTCberoStatusReady For Test=>Done
  Open/ClosedOpen=>Closed
Fri 18 Nov 2005 06:44:40 PM UTCcharles_levertAttached File-=>Added grep.texi.diff4, #5492
Fri 18 Nov 2005 06:43:19 PM UTCcharles_levertAttached File-=>Added grep.1, #5491
Fri 18 Nov 2005 09:50:30 AM UTCcharles_levertAttached File-=>Added grep.texi.diff3, #5490
Fri 18 Nov 2005 09:47:02 AM UTCcharles_levertAttached File-=>Added grep.1, #5489
Sat 12 Nov 2005 12:19:59 PM UTCcharles_levertAttached File-=>Added doc2.patch, #5467
Sat 12 Nov 2005 12:19:58 PM UTCcharles_levertStatusNone=>Ready For Test
Fri 11 Nov 2005 05:07:45 AM UTCcharles_levertAttached File-=>Added doc.patch, #5459

Back to the top


Powered by Savane 3.1-cleanup