On Tue, Feb 18, 2014 at 7:49 AM, Michael Kerrisk (man-pages)
<mtk.manpages@xxxxxxxxx> wrote:
> Hello Denis, Peter,
>
> On Mon, Feb 17, 2014 at 11:39 PM, D. Barbier <bouzim@xxxxxxxxx> wrote:
>> On 2014-02-17 16:44 GMT+01:00 Michael Kerrisk (man-pages) wrote:
>>> On Mon, Feb 17, 2014 at 4:33 PM, Simon Paillard <spaillard@xxxxxxxxxx> wrote:
>>>> On Mon, Feb 17, 2014 at 02:15:51PM +0100, Michael Kerrisk (man-pages) wrote:
>>>>> On 02/16/2014 02:48 PM, Simon Paillard wrote:
>>>>> > Introduced in 33972b0b231b9b0ee708f27fe42461dbca6e2968
>>>>>
>>>>> Bother! I still didn't get it right.
>>>>>
>>>>> Thanks for the patch! Applied.
>>>>
>>>> BTW, I check the rendering of the iso 8859 -11 and -13 manpages before/after
>>>> the change and I think there is something to be fixed too.
>>>
>>> Oh -- that was automated with some scripts by Peter Schiffer, and the
>>> results looked okay to me, but maybe I did not look closely enough.
>>> What problems do you see?
>>
>> Hello,
>>
>> Simon is right, Peter's script handles iso_8859-{11,13}.7 in a
>> different way, I do not understand why.
>> You can fix them with:
>> for f in iso_8859-11 iso_8859-13; do
>> cp man7/$f.7 $f
>> iconv -f utf8 -t latin1 $f | iconv -f iso-${f#iso_} -t utf8 > man7/$f.7
>> done
>
> Denis, thanks for the scripts.
>
> Peter, perhaps we've missed something. Was there a reason to treat
> iso_8859-{11,13}.7 differently?
Got it. iconv(1) does not understand the some encoding names that
start "iso_", but does understand the corresponding forms "iso-".
Thus, iso_8859-11.7 and iso_8859-13.7 weren't acceptable, and Peter's
script (mis-)worked around this. The correct approach, AFAICT is to
use the forms iso-8859-11.7 and iso-8859-13.7. I committed a change to
the script to fix that.
Denis, I've applied your change.
Simon, thanks for catching this.
Peter, sorry I didn't better review the original change.
Cheers,
Michael
--
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
