From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 36741 invoked by alias); 20 May 2017 06:05:13 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 15028 invoked by uid 89); 20 May 2017 06:03:55 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-7.0 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_1,RP_MATCHES_RCVD,SPF_PASS autolearn=unavailable version=3.3.2 spammy=documents X-HELO: smtp.pacific.net Subject: Re: [PATCH v4 3/5] manual: Convert errno @comments to new @errno macro. To: Zack Weinberg , libc-alpha@sourceware.org References: <20170519093353.6158-1-ricaljasan@pacific.net> <20170519093353.6158-4-ricaljasan@pacific.net> Cc: Joseph Myers , Carlos O'Donell , Michael Kerrisk From: Rical Jasan Message-ID: <21b6b782-2069-10a0-b7a7-463378f2cfd2@pacific.net> Date: Sat, 20 May 2017 06:05:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2017-05/txt/msg00618.txt.bz2 On 05/19/2017 02:03 PM, Zack Weinberg wrote: > On 05/19/2017 05:33 AM, Rical Jasan wrote: >> errno.texi documents error macros, their values, and error strings in >> Texinfo @comments, some of which are also used for @standards. The >> purpose of this commit is to separate the standards from the error >> strings so that both the @standards conversion script picks up clean >> @standards and the errno documentation framework is improved. >> >> The error names, values, and messages are consolidated in a new custom >> macro, @errno. It is not clear that scripts within the sources rely >> on the special Texinfo @comment-based format to generate files used >> throughout the library, so the definition of @errno in macros.texi now >> provides a comment indicating the dependency. The dependent scripts >> are updated to use @errno, which also simplifies them a bit. The >> files those scripts generate were verified to be unchanged. >> >> The @errno macro is not visibly rendered in any way at this time, but >> it does use an @cindex command to add the error string to the Concept >> Index, to facilitate searching on error messages. >> >> * manual/errno.texi: Convert @comment-based errno >> documentation to @errno. >> * manual/macros.texi (@errno): New macro. Consolidate errors, >> their values, and messages, adding the error string to the >> Concept Index. Provide a warning in the comment about >> external (to the manual) dependencies. >> * sysdeps/gnu/errlist.awk: Use @errno instead of @comments. >> * sysdeps/mach/hurd/errnos.awk: Likewise. > > This is also OK. Please go ahead and commit. I will adapt my > outstanding patches to match. Committed. I pushed this before @standards, so the macros.texi patch is slightly altered, to account for it not being present. Thank you, Rical