From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 92D433858CDA for ; Tue, 21 Feb 2023 09:15:39 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 92D433858CDA Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1676970939; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:in-reply-to:in-reply-to: references:references; bh=VuSYWuwcyxJuWko6D5Ot42akp1wngNu+dDjZP5bOxuc=; b=UkD/3fEI3OO70CesgWoQmZ4EPfAosjVBFhHDJ5hdNy7JIpogIlGlodss8D1CK+RF9jvt9a MCgsw0TBN2aOkJu5/UxBOqbdpZa0h68+ifmnRJ45OLeNdpx2O5wy5QBx7fafzRCG5WN/x4 kEme6R513sQ6epBLFmYLrIg5xBvIuJk= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-323-k7bLa6HIMIWZpje30neHhw-1; Tue, 21 Feb 2023 04:15:36 -0500 X-MC-Unique: k7bLa6HIMIWZpje30neHhw-1 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.rdu2.redhat.com [10.11.54.5]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 9C399802D35; Tue, 21 Feb 2023 09:15:36 +0000 (UTC) Received: from calimero.vinschen.de (unknown [10.39.192.118]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 7A4E418EC2; Tue, 21 Feb 2023 09:15:36 +0000 (UTC) Received: by calimero.vinschen.de (Postfix, from userid 500) id B2039A8079F; Tue, 21 Feb 2023 10:15:34 +0100 (CET) Date: Tue, 21 Feb 2023 10:15:34 +0100 From: Corinna Vinschen To: newlib@sourceware.org Cc: Jon TURNEY Subject: Re: [PATCH v3 0/2] newlib/libc/time/strftime: fix multi-page table format issues Message-ID: Reply-To: newlib@sourceware.org Mail-Followup-To: newlib@sourceware.org, Jon TURNEY References: <20230217204902.3735-1-Brian.Inglis@Shaw.ca> <20230221041801.51970-1-Brian.Inglis@Shaw.ca> MIME-Version: 1.0 In-Reply-To: <20230221041801.51970-1-Brian.Inglis@Shaw.ca> X-Scanned-By: MIMEDefang 3.1 on 10.11.54.5 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Disposition: inline X-Spam-Status: No, score=-4.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Jon, I'd like your GTG on the patchset before merging it. Thanks, Corinna On Feb 20 21:17, Brian Inglis wrote: > Discussion about why newlib man generation by docbook2man is > incompatible with how man is incompatible with groff/tbl/grohtml: > > https://lists.gnu.org/archive/html/bug-groff/2023-02/msg00118.html > > There does not appear to be good way to deal in docbook2man processing > with generation of tables > "page" size, or that may not adversely affect > other [newlib] doc man page tables, as the problem occurs solely on the > strftime.3 man page! > > The imminent groff/tbl release fixes a number of tbl issues, so may > affect man pages with tables differently. > > The following groff/grohtml release plans to change grohtml, from > generating tables as PNG graphics, which don't work reliably on some > "devices"/file formats, and are not searchable, to generating tables in > searchable text form on all "devices"/file formats, and fix other > related issues, so may also affect man pages with tables differently. > > So for the current release, localize the changes to the man page chew > input embedded in the strftime.c source comments, and the generated > strftime.3 man page table formatting. > Be prepared to tweak formatting if doc generation needs it, and > eventually eliminate custom processing. > > newlib/libc/time/strftime.c: split chew table of conversion format > specifiers as man/tbl/groff can not handle large tables on all output > devices/file formats > > newlib/libc/Makefile.inc: sed fix strftime.3 tbl/groff format issues: > remove multiple "^l l$" tbl line formats at tops of tables; > change remaining "^l l\.$" tbl line formats at tops of tables so second > column is lx and extends to margin; > remove "^.PP$" breaks before each format description as it misaligns > text below format specifier; > remove blank line between adjacent tables > > Brian Inglis (2): > newlib/libc/time/strftime.c: split chew table of conversion format > specifiers > newlib/libc/Makefile.inc: sed fix strftime.3 tbl/groff format issues > > newlib/libc/time/strftime.c | 3 ++- > newlib/libc/Makefile.inc | 1 + > 2 files changed, 3 insertions(+), 1 deletion(-) > > -- > 2.39.0