* [patch] console_codes.4: ffix @ 2022-01-14 6:48 nick black 2022-01-17 17:30 ` Alejandro Colomar (man-pages) [not found] ` <CAN4uE+p-uKzHNYry2YhCMfEFBQ2jUqpDAGx=+eha01w-L4fAjg@mail.gmail.com> 0 siblings, 2 replies; 12+ messages in thread From: nick black @ 2022-01-14 6:48 UTC (permalink / raw) To: linux-man; +Cc: Alejandro Colomar, Michael Kerrisk, linux-kernel Fix up the busted OSC command list (reset palette and set palette). Remove CSI prefix from the list of non-CSI escapes. End all items of said list with periods, matching other sections of the page. Signed-off-by: nick black <nickblack@linux.com> --- man4/console_codes.4 | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git man4/console_codes.4 man4/console_codes.4 index d549b76a9..001de1955 100644 --- man4/console_codes.4 +++ man4/console_codes.4 @@ -139,29 +139,28 @@ T} ESC 8 DECRC T{ Restore state most recently saved by ESC 7. T} -ESC [ CSI Control sequence introducer ESC % Start sequence selecting character set ESC % @ \0\0\0Select default (ISO 646 / ISO 8859-1) ESC % G \0\0\0Select UTF-8 ESC % 8 \0\0\0Select UTF-8 (obsolete) ESC # 8 DECALN T{ -DEC screen alignment test \- fill screen with E's +DEC screen alignment test \- fill screen with E's. T} ESC ( T{ Start sequence defining G0 character set (followed by one of B, 0, U, K, as below) T} ESC ( B T{ -Select default (ISO 8859-1 mapping) +Select default (ISO 8859-1 mapping). T} ESC ( 0 T{ -Select VT100 graphics mapping +Select VT100 graphics mapping. T} ESC ( U T{ -Select null mapping \- straight to character ROM +Select null mapping \- straight to character ROM. T} ESC ( K T{ -Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8) +Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8). T} ESC ) T{ Start sequence defining G1 (followed by one of B, 0, U, K, as above). @@ -169,12 +168,13 @@ T} ESC > DECPNM Set numeric keypad mode ESC = DECPAM Set application keypad mode ESC ] OSC T{ -(Should be: Operating system command) -ESC ] P \fInrrggbb\fP: set palette, with parameter -given in 7 hexadecimal digits after the final P :-(. -Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates +Operating System Command prefix. +T} +ESC ] R Reset palette. +ESC ] P T{ +Set palette, with parameter given in 7 hexadecimal digits \fInrrggbb\fP after +the final P. Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates the red/green/blue values (0\(en255). -ESC ] R: reset palette T} .TE .ad -- 2.34.1 -- nick black -=- https://www.nick-black.com to make an apple pie from scratch, you need first invent a universe. ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-01-14 6:48 [patch] console_codes.4: ffix nick black @ 2022-01-17 17:30 ` Alejandro Colomar (man-pages) 2022-01-19 17:05 ` nick black [not found] ` <CAN4uE+p-uKzHNYry2YhCMfEFBQ2jUqpDAGx=+eha01w-L4fAjg@mail.gmail.com> 1 sibling, 1 reply; 12+ messages in thread From: Alejandro Colomar (man-pages) @ 2022-01-17 17:30 UTC (permalink / raw) To: nick black, linux-man; +Cc: Michael Kerrisk, linux-kernel Hello nick, On 1/14/22 07:48, nick black wrote: > Fix up the busted OSC command list (reset palette and > set palette). Remove CSI prefix from the list of non-CSI > escapes. End all items of said list with periods, > matching other sections of the page. > > Signed-off-by: nick black <nickblack@linux.com> > --- > man4/console_codes.4 | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git man4/console_codes.4 man4/console_codes.4 > index d549b76a9..001de1955 100644 > --- man4/console_codes.4 > +++ man4/console_codes.4 > @@ -139,29 +139,28 @@ T} > ESC 8 DECRC T{ > Restore state most recently saved by ESC 7. > T} > -ESC [ CSI Control sequence introducer As I understand it, it's there because CSI is not a CSI-prefixed sequence (i.e., you have to first document CSI itself, and then in a different list you can document sequences prefixed by CSI). So I'd say it belongs there. > ESC % Start sequence selecting character set > ESC % @ \0\0\0Select default (ISO 646 / ISO 8859-1) > ESC % G \0\0\0Select UTF-8 > ESC % 8 \0\0\0Select UTF-8 (obsolete) > ESC # 8 DECALN T{ > -DEC screen alignment test \- fill screen with E's > +DEC screen alignment test \- fill screen with E's. > T} > ESC ( T{ > Start sequence defining G0 character set > (followed by one of B, 0, U, K, as below) > T} > ESC ( B T{ > -Select default (ISO 8859-1 mapping) > +Select default (ISO 8859-1 mapping). > T} > ESC ( 0 T{ > -Select VT100 graphics mapping > +Select VT100 graphics mapping. > T} > ESC ( U T{ > -Select null mapping \- straight to character ROM > +Select null mapping \- straight to character ROM. > T} > ESC ( K T{ > -Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8) > +Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8). > T} > ESC ) T{ > Start sequence defining G1 (followed by one of B, 0, U, K, as above). > @@ -169,12 +168,13 @@ T} > ESC > DECPNM Set numeric keypad mode > ESC = DECPAM Set application keypad mode > ESC ] OSC T{ > -(Should be: Operating system command) > -ESC ] P \fInrrggbb\fP: set palette, with parameter > -given in 7 hexadecimal digits after the final P :-(. > -Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates > +Operating System Command prefix. > +T} > +ESC ] R Reset palette. > +ESC ] P T{ > +Set palette, with parameter given in 7 hexadecimal digits \fInrrggbb\fP after > +the final P. Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates > the red/green/blue values (0\(en255). > -ESC ] R: reset palette Regarding this, I don't know what the intention of the original author was. The pre-git code was (man-pages-1.70): ``` ESC ( U \0\0\0Select null mapping \- straight to character ROM ESC ( K \0\0\0Select user mapping \- the map that is loaded by \0\0\0the utility \fBmapscrn\fP(8). ESC ) Start sequence defining G1 (followed by one of B, 0, U, K, as above). ESC > DECPNM Set numeric keypad mode ESC = DECPAM Set application keypad mode ESC ] OSC (Should be: Operating system command) ESC ] P \fInrrggbb\fP: set palette, with parameter given in 7 hexadecimal digits after the final P :-(. Here \fIn\fP is the color (0-15), and \fIrrggbb\fP indicates the red/green/blue values (0-255). ESC ] R: reset palette .TE .SS "ECMA-48 CSI sequences" ``` Maybe it should go into a separate subsection called "Operating system commands"? Thanks, Alex -- Alejandro Colomar Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ http://www.alejandro-colomar.es/ ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-01-17 17:30 ` Alejandro Colomar (man-pages) @ 2022-01-19 17:05 ` nick black 2022-01-25 17:35 ` nick black 0 siblings, 1 reply; 12+ messages in thread From: nick black @ 2022-01-19 17:05 UTC (permalink / raw) To: Alejandro Colomar (man-pages); +Cc: linux-man, Michael Kerrisk, linux-kernel Alejandro Colomar (man-pages) left as an exercise for the reader: > As I understand it, it's there because CSI is not a CSI-prefixed sequence > (i.e., you have to first document CSI itself, and then in a different list > you can document sequences prefixed by CSI). > So I'd say it belongs there. except this is literally in the table named "ESC- but not CSI-sequences". and it's not, by itself, a sequence. and it's detailed in the "ECMA-48 CSI Sequences" section below. so it is documented, near where it's relevant. IMHO. if we're going to keep it, we ought add OSC to this section by the same reasoning. i can do that and send a fresh patch, or you can do it to my patch, or we can do with what i have. > Maybe it should go into a separate subsection called "Operating system > commands"? well, there are only these few, and they are "ESC- but not CSI sequences". i don't honestly think the average reader cares whether something is a CSI or an OSC or linux-specific control sequence, especially since there's really no user-relevant reason as to why one is in any given group. but i can go ahead and break this section out if you'd like. put another way, some people might read the man page wanting to know "how do i change a color". i can't imagine anyone ever wanting to know "what are the various OSC-prefixed commands?" that said, i'm happy to introduce the substructure if it gets the formatting fixed =]. -- nick black -=- https://www.nick-black.com to make an apple pie from scratch, you need first invent a universe. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-01-19 17:05 ` nick black @ 2022-01-25 17:35 ` nick black 0 siblings, 0 replies; 12+ messages in thread From: nick black @ 2022-01-25 17:35 UTC (permalink / raw) To: Alejandro Colomar (man-pages); +Cc: linux-man, Michael Kerrisk, linux-kernel nick black left as an exercise for the reader: > if we're going to keep it, we ought add OSC to this section by > the same reasoning. i can do that and send a fresh patch, or you > can do it to my patch, or we can do with what i have. ... > sequence, especially since there's really no user-relevant > reason as to why one is in any given group. but i can go ahead > and break this section out if you'd like. ... > put another way, some people might read the man page wanting to > know "how do i change a color". i can't imagine anyone ever > wanting to know "what are the various OSC-prefixed commands?" > that said, i'm happy to introduce the substructure if it gets > the formatting fixed =]. Alejandro, would you like me to make some/all of the mentioned changes, or is this patch fine as it is, or will you be making the changes? Right now the console_codes(4) page has nasty formatting errors (addressed herein); even if we make no content changes, they should be fixed IMHO. -- nick black -=- https://www.nick-black.com to make an apple pie from scratch, you need first invent a universe. ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <CAN4uE+p-uKzHNYry2YhCMfEFBQ2jUqpDAGx=+eha01w-L4fAjg@mail.gmail.com>]
* [patch] console_codes.4: ffix [not found] ` <CAN4uE+p-uKzHNYry2YhCMfEFBQ2jUqpDAGx=+eha01w-L4fAjg@mail.gmail.com> @ 2022-03-20 16:02 ` G. Branden Robinson 2022-03-20 18:15 ` Alejandro Colomar (man-pages) 2022-03-21 8:33 ` nick black 0 siblings, 2 replies; 12+ messages in thread From: G. Branden Robinson @ 2022-03-20 16:02 UTC (permalink / raw) To: linux-man; +Cc: nick black, Alejandro Colomar, Michael Kerrisk, linux-kernel [-- Attachment #1: Type: text/plain, Size: 6704 bytes --] Hi Nick, Alex has been waiting on me for a review of this patch. I took longer than I should have because I thought he was expecting a technical evaluation of the accuracy of the sequences documented.[1] Now I see it was just a matter of man(7) and tbl(1) syntactical and style review. Easy bits first. > Remove CSI prefix from the list of non-CSI escapes. +1 > End all items of said list with periods, matching other sections of > the page. +1 > Fix up the busted OSC command list (reset palette and set palette). > ESC ] OSC T{ > -(Should be: Operating system command) > -ESC ] P \fInrrggbb\fP: set palette, with parameter > -given in 7 hexadecimal digits after the final P :-(. > -Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates > +Operating System Command prefix. > +T} > +ESC ] R Reset palette. > +ESC ] P T{ > +Set palette, with parameter given in 7 hexadecimal digits \fInrrggbb\fP after > +the final P. Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates > the red/green/blue values (0\(en255). > -ESC ] R: reset palette > T} Yes, this was majorly hosed up, syntactically. What you have is good repair to obvious damage. +1. I have some suggestions for further improvement, but please don't gate the patch application on these. 1. "Operating System Command prefix." does not need to be in a text block. It can be set as an ordinary table entry just like "Reset palette." 2. The description of ESC ] P _is_ in a text block. That's good, because guess what? Since it is _text_, you don't need those pesky font selection escape sequences. You can use man(7) font macros instead. You can furthermore apply all of the style rules that attach to ordinary man page text. And I would tighten the wording, too. So I would write... ESC ] P T{ Set palette, with a parameter of 7 hexadecimal digits .I nrrggbb after .BR P . .IR n\~ is the color, and .I rrggbb its 8-bit red/green/blue channel values. T} Observe the use of a non-breaking space escape sequence \~ to act as a "tie", preventing a line break between "n" and the very short word "is". This is not essential; it is a recommended typographical practice. I also saw someone ask how to do this on StackExchange recently[2]. I dropped mentions of range because I feel that people using hexadecimal can be expected to know how to count in that base. > .ad l > .TS > .TE > .ad 3. Consider dumping these adjustment requests bracketing the tables. They arise from a misconception that adjustment cannot be manipulated from within tbl(1) text blocks. It can. I would check the output to see if leaving adjustment as-is results in ugly table entries, and if does, add 'na' requests to the beginnings of the affected text blocks on a case by case basis. We try to discourage the use of *roff requests in man page text; moving them into tbl(1) content is a slight improvement. I rewrote the groff tbl(1) man page for 1.23 (forthcoming) because I found the existing one difficult to understand. Even Lesk's original paper (CSTR #49) left me wondering in places. (I have a thick skull.) Here is the subsection on text blocks. Font style changes are lost in plain text email. More context is available at the source[3]. [[ Text blocks An ordinary table entry’s contents can make a column, and therefore the table, excessively wide; the table then exceeds the line length of the page, and becomes ugly or is exposed to truncation by the output device. When a table entry requires more conventional typesetting, breaking across more than one output line (and thereby increasing the height of its row), it can be placed within a text block. tbl interprets a table entry of “T{” at the end of an input line not as table data, but as a token starting a text block. Similarly, “T}” at the start of an input line ends a text block. Text block tokens can share an input line with other table data (preceding T{ and following T}). Input lines between these tokens are formatted in a diversion by troff. Text blocks cannot be nested. Multiple text blocks can occur in a table row. Like other table entries, text blocks are formatted as was the text prior to the table, modified by applicable column descriptors. Specifically, the classifiers A, C, L, N, R, and S determine a text block’s alignment within its cell, but not its adjustment. You can add na or ad requests to the beginning of a text block to alter its adjustment distinctly from other text in the document. As with other table entries, when a text block ends, any alterations to its formatting are discarded. They do not affect subsequent table entries, not even other text blocks. If w or x modifiers are not specified for all columns of a text block’s span, the default length of the text block (more precisely, the line length used to process the text block diversion) is computed as L×C/(N+1), where L is the current line length, C the number of columns spanned by the text block, and N the number of columns in the table. If necessary, you can also control a text block’s width by including an ll (line length) request in it prior to any text to be formatted. Because a diversion is used to format the text block, its width is subsequently available in the register dl. ]] Regards, Branden [1] I did that for the ones most recast, ESC ] P and ESC ] R, but unfortunately the console driver on my system suffers from serious redraw problems on scroll; if you reset the palette from the last line on the screen, many characters on the screen do not get repainted in the new color. Keep changing the palette, and color errors accumulate, so the appearance of color 7 (the default foreground) can appear in many shades simultaneously even though the driver, presumably, thinks they're all the same. If the redrawing errors are deterministic, can I assume that this will never be rectified because someone out there must have a 24-bit color ASCII art image viewer for the console, and fixing the bug would ruin it? [2] https://unix.stackexchange.com/questions/694622/how-can-i-prevent-a-line-break-between-option-and-parameter-using-rb-and-ir/694765#694765 [3] https://git.savannah.gnu.org/cgit/groff.git/tree/src/preproc/tbl/tbl.1.man [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-03-20 16:02 ` G. Branden Robinson @ 2022-03-20 18:15 ` Alejandro Colomar (man-pages) 2022-03-21 8:30 ` nick black 2022-03-21 8:33 ` nick black 1 sibling, 1 reply; 12+ messages in thread From: Alejandro Colomar (man-pages) @ 2022-03-20 18:15 UTC (permalink / raw) To: G. Branden Robinson; +Cc: nick black, Michael Kerrisk, linux-kernel, linux-man Hi, nick, and Branden! On 3/20/22 17:02, G. Branden Robinson wrote: > Hi Nick, > > Alex has been waiting on me for a review of this patch. I took longer > than I should have because I thought he was expecting a technical > evaluation of the accuracy of the sequences documented.[1] Now I see it > was just a matter of man(7) and tbl(1) syntactical and style review. No, you were originally right. I did mean both, but especially "a technical evaluation of the accuracy of the sequences documented". You did it anyway, so thanks! :) > > Easy bits first. > >> Remove CSI prefix from the list of non-CSI escapes. > > +1 > >> End all items of said list with periods, matching other sections of >> the page. > > +1 > >> Fix up the busted OSC command list (reset palette and set palette). > >> ESC ] OSC T{ >> -(Should be: Operating system command) >> -ESC ] P \fInrrggbb\fP: set palette, with parameter >> -given in 7 hexadecimal digits after the final P :-(. >> -Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates >> +Operating System Command prefix. >> +T} >> +ESC ] R Reset palette. >> +ESC ] P T{ >> +Set palette, with parameter given in 7 hexadecimal digits \fInrrggbb\fP after >> +the final P. Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates >> the red/green/blue values (0\(en255). >> -ESC ] R: reset palette >> T} > > Yes, this was majorly hosed up, syntactically. What you have is good > repair to obvious damage. +1. > > I have some suggestions for further improvement, but please don't gate > the patch application on these. [...] Okay, thanks! nick, can you please resend the patch? I've lost the original email. Cheers, Alex -- Alejandro Colomar Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/ http://www.alejandro-colomar.es/ ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-03-20 18:15 ` Alejandro Colomar (man-pages) @ 2022-03-21 8:30 ` nick black 2022-03-22 12:27 ` Alejandro Colomar (man-pages) 0 siblings, 1 reply; 12+ messages in thread From: nick black @ 2022-03-21 8:30 UTC (permalink / raw) To: Alejandro Colomar (man-pages) Cc: G. Branden Robinson, Michael Kerrisk, linux-kernel, linux-man Alejandro Colomar (man-pages) left as an exercise for the reader: > nick, can you please resend the patch? I've lost the original email. Fix up the busted OSC command list (reset palette and set palette). Remove CSI prefix from the list of non-CSI escapes. End all items of said list with periods, matching other sections of the page. Signed-off-by: nick black <nickblack@linux.com> --- man4/console_codes.4 | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git man4/console_codes.4 man4/console_codes.4 index d549b76a9..001de1955 100644 --- man4/console_codes.4 +++ man4/console_codes.4 @@ -139,29 +139,28 @@ T} ESC 8 DECRC T{ Restore state most recently saved by ESC 7. T} -ESC [ CSI Control sequence introducer ESC % Start sequence selecting character set ESC % @ \0\0\0Select default (ISO 646 / ISO 8859-1) ESC % G \0\0\0Select UTF-8 ESC % 8 \0\0\0Select UTF-8 (obsolete) ESC # 8 DECALN T{ -DEC screen alignment test \- fill screen with E's +DEC screen alignment test \- fill screen with E's. T} ESC ( T{ Start sequence defining G0 character set (followed by one of B, 0, U, K, as below) T} ESC ( B T{ -Select default (ISO 8859-1 mapping) +Select default (ISO 8859-1 mapping). T} ESC ( 0 T{ -Select VT100 graphics mapping +Select VT100 graphics mapping. T} ESC ( U T{ -Select null mapping \- straight to character ROM +Select null mapping \- straight to character ROM. T} ESC ( K T{ -Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8) +Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8). T} ESC ) T{ Start sequence defining G1 (followed by one of B, 0, U, K, as above). @@ -169,12 +168,13 @@ T} ESC > DECPNM Set numeric keypad mode ESC = DECPAM Set application keypad mode ESC ] OSC T{ -(Should be: Operating system command) -ESC ] P \fInrrggbb\fP: set palette, with parameter -given in 7 hexadecimal digits after the final P :-(. -Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates +Operating System Command prefix. +T} +ESC ] R Reset palette. +ESC ] P T{ +Set palette, with parameter given in 7 hexadecimal digits \fInrrggbb\fP after +the final P. Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates the red/green/blue values (0\(en255). -ESC ] R: reset palette T} .TE .ad -- 2.34.1 -- nick black -=- https://www.nick-black.com to make an apple pie from scratch, you need first invent a universe. ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-03-21 8:30 ` nick black @ 2022-03-22 12:27 ` Alejandro Colomar (man-pages) 2022-03-22 12:56 ` nick black 0 siblings, 1 reply; 12+ messages in thread From: Alejandro Colomar (man-pages) @ 2022-03-22 12:27 UTC (permalink / raw) To: nick black; +Cc: G. Branden Robinson, Michael Kerrisk, linux-kernel, linux-man Hi nick, On 3/21/22 09:30, nick black wrote: > Alejandro Colomar (man-pages) left as an exercise for the reader: >> nick, can you please resend the patch? I've lost the original email. > > Fix up the busted OSC command list (reset palette and > set palette). Remove CSI prefix from the list of non-CSI > escapes. End all items of said list with periods, > matching other sections of the page. > > Signed-off-by: nick black <nickblack@linux.com> Patch applied. However, it's weird: I had to apply the following to your patch before applying it with `git am`: /^diff --git/s, man4, a/man4, /^diff --git/s, man4, b/man4, /^--- man4/s, man4, a/man4, /^+++ man4/s, man4, b/man4, I'm curious, how did you generate the patch? Cheers, Alex > --- > man4/console_codes.4 | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git man4/console_codes.4 man4/console_codes.4 > index d549b76a9..001de1955 100644 > --- man4/console_codes.4 > +++ man4/console_codes.4 > @@ -139,29 +139,28 @@ T} > ESC 8 DECRC T{ > Restore state most recently saved by ESC 7. > T} > -ESC [ CSI Control sequence introducer > ESC % Start sequence selecting character set > ESC % @ \0\0\0Select default (ISO 646 / ISO 8859-1) > ESC % G \0\0\0Select UTF-8 > ESC % 8 \0\0\0Select UTF-8 (obsolete) > ESC # 8 DECALN T{ > -DEC screen alignment test \- fill screen with E's > +DEC screen alignment test \- fill screen with E's. > T} > ESC ( T{ > Start sequence defining G0 character set > (followed by one of B, 0, U, K, as below) > T} > ESC ( B T{ > -Select default (ISO 8859-1 mapping) > +Select default (ISO 8859-1 mapping). > T} > ESC ( 0 T{ > -Select VT100 graphics mapping > +Select VT100 graphics mapping. > T} > ESC ( U T{ > -Select null mapping \- straight to character ROM > +Select null mapping \- straight to character ROM. > T} > ESC ( K T{ > -Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8) > +Select user mapping \- the map that is loaded by the utility \fBmapscrn\fP(8). > T} > ESC ) T{ > Start sequence defining G1 (followed by one of B, 0, U, K, as above). > @@ -169,12 +168,13 @@ T} > ESC > DECPNM Set numeric keypad mode > ESC = DECPAM Set application keypad mode > ESC ] OSC T{ > -(Should be: Operating system command) > -ESC ] P \fInrrggbb\fP: set palette, with parameter > -given in 7 hexadecimal digits after the final P :-(. > -Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates > +Operating System Command prefix. > +T} > +ESC ] R Reset palette. > +ESC ] P T{ > +Set palette, with parameter given in 7 hexadecimal digits \fInrrggbb\fP after > +the final P. Here \fIn\fP is the color (0\(en15), and \fIrrggbb\fP indicates > the red/green/blue values (0\(en255). > -ESC ] R: reset palette > T} > .TE > .ad -- Alejandro Colomar Linux man-pages comaintainer; http://www.kernel.org/doc/man-pages/ http://www.alejandro-colomar.es/ ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-03-22 12:27 ` Alejandro Colomar (man-pages) @ 2022-03-22 12:56 ` nick black 2022-03-22 12:59 ` Alejandro Colomar (man-pages) 0 siblings, 1 reply; 12+ messages in thread From: nick black @ 2022-03-22 12:56 UTC (permalink / raw) To: Alejandro Colomar (man-pages) Cc: G. Branden Robinson, Michael Kerrisk, linux-kernel, linux-man Alejandro Colomar (man-pages) left as an exercise for the reader: > Patch applied. > However, it's weird: I had to apply the following to your patch before > applying it with `git am`: > > /^diff --git/s, man4, a/man4, > /^diff --git/s, man4, b/man4, > /^--- man4/s, man4, a/man4, > /^+++ man4/s, man4, b/man4, > > I'm curious, how did you generate the patch? i bounced this out from ~/Mail/sent, and have no idea how i originally created it, sorry =\. i'm assuming git email-send? -- nick black -=- https://www.nick-black.com to make an apple pie from scratch, you need first invent a universe. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-03-22 12:56 ` nick black @ 2022-03-22 12:59 ` Alejandro Colomar (man-pages) 2022-03-22 13:01 ` nick black 0 siblings, 1 reply; 12+ messages in thread From: Alejandro Colomar (man-pages) @ 2022-03-22 12:59 UTC (permalink / raw) To: nick black; +Cc: G. Branden Robinson, Michael Kerrisk, linux-kernel, linux-man On 3/22/22 13:56, nick black wrote: > Alejandro Colomar (man-pages) left as an exercise for the reader: >> Patch applied. >> However, it's weird: I had to apply the following to your patch before >> applying it with `git am`: >> >> /^diff --git/s, man4, a/man4, >> /^diff --git/s, man4, b/man4, >> /^--- man4/s, man4, a/man4, >> /^+++ man4/s, man4, b/man4, >> >> I'm curious, how did you generate the patch? > > i bounced this out from ~/Mail/sent, and have no idea how i > originally created it, sorry =\. i'm assuming git email-send? > Yeah, I guessed that you used git format-patch && git send-email, especially since there's a git version at the end of the patch. The weird thing is that git always (AFAIK) writes (and needs) those a/ and b/ prefixes, so... I don't know. If it repeats, I'll try to investigate the reason. Thanks! Alex -- Alejandro Colomar Linux man-pages comaintainer; http://www.kernel.org/doc/man-pages/ http://www.alejandro-colomar.es/ ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-03-22 12:59 ` Alejandro Colomar (man-pages) @ 2022-03-22 13:01 ` nick black 0 siblings, 0 replies; 12+ messages in thread From: nick black @ 2022-03-22 13:01 UTC (permalink / raw) To: Alejandro Colomar (man-pages) Cc: G. Branden Robinson, Michael Kerrisk, linux-kernel, linux-man Alejandro Colomar (man-pages) left as an exercise for the reader: > If it repeats, I'll try to investigate the reason. or i can just make sure i'm sending the expected format--it ought not be beyond my powers =]. thanks for bringing this to my attention. -- nick black -=- https://www.nick-black.com to make an apple pie from scratch, you need first invent a universe. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [patch] console_codes.4: ffix 2022-03-20 16:02 ` G. Branden Robinson 2022-03-20 18:15 ` Alejandro Colomar (man-pages) @ 2022-03-21 8:33 ` nick black 1 sibling, 0 replies; 12+ messages in thread From: nick black @ 2022-03-21 8:33 UTC (permalink / raw) To: G. Branden Robinson Cc: linux-man, Alejandro Colomar, Michael Kerrisk, linux-kernel [-- Attachment #1: Type: text/plain, Size: 719 bytes --] G. Branden Robinson left as an exercise for the reader: > I have some suggestions for further improvement, but please don't gate > the patch application on these. always good to hear from you, GBR. i replied to alejandro's request with the original, not wanting to mess with (and then test) groff right at the moment, but i've saved your animadversions. i've got a patch outstanding for the linux console right now, and once it lands, i'll need come back and modify this page again (indeed, that's why i was looking at it in the first place). i'll effect your suggestions then, thanks! -- nick black -=- https://www.nick-black.com to make an apple pie from scratch, you need first invent a universe. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2022-03-22 13:01 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-01-14 6:48 [patch] console_codes.4: ffix nick black 2022-01-17 17:30 ` Alejandro Colomar (man-pages) 2022-01-19 17:05 ` nick black 2022-01-25 17:35 ` nick black [not found] ` <CAN4uE+p-uKzHNYry2YhCMfEFBQ2jUqpDAGx=+eha01w-L4fAjg@mail.gmail.com> 2022-03-20 16:02 ` G. Branden Robinson 2022-03-20 18:15 ` Alejandro Colomar (man-pages) 2022-03-21 8:30 ` nick black 2022-03-22 12:27 ` Alejandro Colomar (man-pages) 2022-03-22 12:56 ` nick black 2022-03-22 12:59 ` Alejandro Colomar (man-pages) 2022-03-22 13:01 ` nick black 2022-03-21 8:33 ` nick black
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).