patchgrep - Patches: patch #4610, Consolidated documentation patch

 
 

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

patch #4610: Consolidated documentation patch

Submitter:  Charles Levert <charles_levert>
Submitted:  Fri 11 Nov 2005 05:07:45 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Open/Closed:  Closed

Jump to the original submission

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

Looks good -- applied.
Thanks.

Bernhard Rosenkraenzer <bero>
Group 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>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

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

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 9 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2006-08-18 bero StatusReady For Test Done
    Open/ClosedOpen Closed
2005-11-18 charles_levert Attached File- Added grep.texi.diff4, #5492
2005-11-18 charles_levert Attached File- Added grep.1, #5491
2005-11-18 charles_levert Attached File- Added grep.texi.diff3, #5490
2005-11-18 charles_levert Attached File- Added grep.1, #5489
2005-11-12 charles_levert Attached File- Added doc2.patch, #5467
2005-11-12 charles_levert StatusNone Ready For Test
2005-11-11 charles_levert Attached File- Added doc.patch, #5459

Back to the top

Powered by Savane 3.13-f8d8.
Corresponding source code