[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #61315] Several groff source files don't include <config.h> first

From: Paul Eggert
Subject: [bug #61315] Several groff source files don't include <config.h> first
Date: Sat, 9 Oct 2021 03:51:07 -0400 (EDT)
User-agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:93.0) Gecko/20100101 Firefox/93.0


                 Summary: Several groff source files don't include <config.h>
                 Project: GNU troff
            Submitted by: eggert
            Submitted on: Sat 09 Oct 2021 12:51:06 AM PDT
                Category: None
                Severity: 3 - Normal
              Item Group: Build/Installation
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: None



Gnulib-using apps are supposed to include <config.h> first, if they use any
feature that Gnulib might affect. Since Gnulib redefines standard headers,
this means pretty much every source file must include <config.h> first. The
Groff source code forgets to do this in some places. Although this is OK with
the current groff master branch (commit
2ab0dacb95863a2e347d06cf970676c74c784ce2), it won't work once you update Groff
to current Gnulib, as reported by Bjarni Ingi Gislason here:


The obvious fix is to include <config.h> first in every source file that isn't
doing that already. Proposed patch attached.


File Attachments:

Date: Sat 09 Oct 2021 12:51:06 AM PDT  Name:
0001-gnulib-Include-config.h-first.patch  Size: 9KiB   By: eggert
Modify Groff to include &lt;config.h&gt; first.


Reply to this item at:


  Message sent via Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]