All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alejandro Colomar (man-pages)" <alx.manpages@gmail.com>
To: "Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>
Cc: linux-man <linux-man@vger.kernel.org>
Subject: Re: make html: Errors
Date: Tue, 16 Feb 2021 11:46:49 +0100	[thread overview]
Message-ID: <28981140-bfc6-d7f7-697c-2479af764311@gmail.com> (raw)
In-Reply-To: <3dd7707d-6ea6-c68b-8f53-15e743f273c3@gmail.com>

Hi Michael,

On 2/16/21 11:23 AM, Michael Kerrisk (man-pages) wrote:
> Hi Alex,
> 
> On 2/15/21 10:11 PM, Alejandro Colomar (man-pages) wrote:
>> Hi Michael,
>>
>> I tried to render HTML pages with the Makefile.
>> I had some errors, even though the operation succeed (I have the
>> resulting HTML files, and the contents look good in the browser.
>> Has it happened to you before?  Do you know how to solve it?
>> I copied here the full error messages, in case it is useful.
> 
> There's a bug in the script, it seems. Not all of the pages
> were being correctly processed. In particular, link pages
> (".so dir/page.n") were not being correctly handled. I think
> the patch below fixes things. Can you test and confirm?

Yes, it fixes the error about link pages.  Now I have:

$ make HTDIR=.html/ html
man2html: Unknown operator .
man2html: Unknown operator .
man2html: Unknown operator .
man2html: Unknown operator .
man2html: Unknown operator .
man2html: Unknown operator .
man2html: Unknown operator .
man2html: Unknown operator .
man2html: Unknown operator .
$

This seems to be an error while parsing <elf.5>.

> 
>> I guess this is what you use for generating the online pages, isn't it?
> 
> No, which is why I never noticed the bug. I suspect hardly anyone
> tried "make html" for a very long time.
> 
> Cheers,
> 
> Michael
> 
> 
> diff --git a/Makefile b/Makefile
> index 328418e8e..ea31eab80 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -17,7 +17,7 @@ html:
>         for i in man?; do \
>                 [ -d $(HTDIR)/"$$i" ] || mkdir -p $(HTDIR)/"$$i"; \
>                 find "$$i/" -type f | while read f; do \
> -                       (cd "$$i"; man2html $(HTOPTS) `basename $$f`) | \
> +                       man2html $(HTOPTS) $$f | \
>                         sed -e '1,2d' > $(HTDIR)/"$$i"/`basename $$f`.html; \
>                 done; \
>         done; fi
> 

git am failed to apply it.  I had to apply it manually.

Thanks,

Alex


-- 
Alejandro Colomar
Linux man-pages comaintainer; https://www.kernel.org/doc/man-pages/
http://www.alejandro-colomar.es/

  reply	other threads:[~2021-02-16 10:47 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-15 21:11 make html: Errors Alejandro Colomar (man-pages)
2021-02-16 10:23 ` Michael Kerrisk (man-pages)
2021-02-16 10:46   ` Alejandro Colomar (man-pages) [this message]
2021-02-16 11:20     ` Michael Kerrisk (man-pages)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=28981140-bfc6-d7f7-697c-2479af764311@gmail.com \
    --to=alx.manpages@gmail.com \
    --cc=linux-man@vger.kernel.org \
    --cc=mtk.manpages@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.