Re: [blfs-book] [blfs] branch trunk updated: Tags for SGML and XML. Add a patch for paps.

On Tue, 2026-02-24 at 21:46 -0600, Douglas R. Reno wrote:
> On 2/24/26 9:40 PM, Git Owner ([email protected] via blfs-book 
> Mailing List) wrote:
> > This is an automated email from the git hooks/post-receive script.
> > 
> > git pushed a commit to branch trunk
> > in repository blfs.
> > 
> > The following commit(s) were added to refs/heads/trunk by this push:
> >       new c2e01df6df Tags for SGML and XML.  Add a patch for paps.
> >       new ae0f1c364e Merge branch 'trunk' of
> > git.linuxfromscratch.org:blfs into trunk
> > c2e01df6df is described below
> > 
> > commit c2e01df6df9454170590288cebff640b512844f9
> > Author: Bruce Dubbs <[email protected]>
> > AuthorDate: Tue Feb 24 21:38:43 2026 -0600
> > 
> >      Tags for SGML and XML.  Add a patch for paps.
> > ---
> >   networking/connect/dhcpcd.xml       |  2 +-
> >   networking/netprogs/bridgeutils.xml |  2 +-
> >   pst/ps/enscript.xml                 |  2 +-
> >   pst/ps/epdfview.xml                 |  2 +-
> >   pst/ps/paps.xml                     | 16 ++++++++++++++--
> >   pst/sgml/docbook-3.1-dtd.xml        |  2 +-
> >   pst/sgml/docbook-4.5-dtd.xml        |  2 +-
> >   pst/sgml/docbook-dsssl.xml          |  2 +-
> >   pst/sgml/docbook-utils.xml          |  2 +-
> >   pst/sgml/openjade.xml               |  2 +-
> >   pst/xml/docbook-xsl.xml             |  4 ++--
> >   pst/xml/docbook5.xml                |  2 +-
> >   12 files changed, 26 insertions(+), 14 deletions(-)
> > 
> > diff --git a/networking/connect/dhcpcd.xml
> > b/networking/connect/dhcpcd.xml
> > index 27c017ae94..d9328755f5 100644
> > --- a/networking/connect/dhcpcd.xml
> > +++ b/networking/connect/dhcpcd.xml
> > @@ -41,7 +41,7 @@
> >         </para>
> >       </note>
> >   
> > -    &lfs124_checked;
> > +    &lfs130_checked;
> >   
> >       <bridgehead renderas="sect3">Package Information</bridgehead>
> >       <itemizedlist spacing="compact">
> > diff --git a/networking/netprogs/bridgeutils.xml
> > b/networking/netprogs/bridgeutils.xml
> > index 9239cad7bb..64ea48b2d9 100644
> > --- a/networking/netprogs/bridgeutils.xml
> > +++ b/networking/netprogs/bridgeutils.xml
> > @@ -31,7 +31,7 @@
> >         networks for a hosted virtual machine (VM).
> >       </para>
> >   
> > -    &lfs124_checked;
> > +    &lfs130_checked;
> >   
> >       <bridgehead renderas="sect3">Package Information</bridgehead>
> >       <itemizedlist spacing="compact">
> > diff --git a/pst/ps/enscript.xml b/pst/ps/enscript.xml
> > index 61e7e16369..efb653defa 100644
> > --- a/pst/ps/enscript.xml
> > +++ b/pst/ps/enscript.xml
> > @@ -34,7 +34,7 @@
> >         to PostScript, HTML, RTF, ANSI and overstrikes.
> >       </para>
> >   
> > -    &lfs124_checked;
> > +    &lfs130_checked;
> >   
> >       <caution>
> >         <para>
> > diff --git a/pst/ps/epdfview.xml b/pst/ps/epdfview.xml
> > index 7860d93cc0..91b842866d 100644
> > --- a/pst/ps/epdfview.xml
> > +++ b/pst/ps/epdfview.xml
> > @@ -35,7 +35,7 @@
> >         <application>MuPDF</application>.
> >       </para>
> >   
> > -    &lfs124_checked;
> > +    &lfs130_checked;
> >   
> >       <bridgehead renderas="sect3">Package Information</bridgehead>
> >       <itemizedlist spacing="compact">
> > diff --git a/pst/ps/paps.xml b/pst/ps/paps.xml
> > index 27c1071f89..45465e1977 100644
> > --- a/pst/ps/paps.xml
> > +++ b/pst/ps/paps.xml
> > @@ -33,7 +33,7 @@
> >         <application>Pango</application>.
> >       </para>
> >   
> > -    &lfs124_checked;
> > +    &lfs130_checked;
> >   
> >       <bridgehead renderas="sect3">Package Information</bridgehead>
> >       <itemizedlist spacing="compact">
> > @@ -69,6 +69,16 @@
> >         </listitem>
> >       </itemizedlist>
> >   
> > +    <bridgehead renderas="sect3">Additional Downloads</bridgehead>
> > +    <itemizedlist spacing='compact'>
> > +      <listitem>
> > +        <para>
> > +          Required patch: <ulink url=
> > +            "https://github.com/dov/paps/pull/77.patch"/>
> > +        </para>
> > +      </listitem>
> > +    </itemizedlist>
> > +
> >       <bridgehead renderas="sect3">paps Dependencies</bridgehead>
> >       <bridgehead renderas="sect4">Required</bridgehead>
> >       <para role="required">
> > @@ -92,7 +102,9 @@
> >         First fix a problem with glib-2.81.0 or later:
> >       </para>
> >   
> > -<screen><userinput>sed -i -r 's/g_utf8_(next|offset)/(char*)
> > &amp;/' src/paps.cc</userinput></screen>
> > +<screen><userinput>sed -i -r 's/g_utf8_(next|offset)/(char*)
> > &amp;/' src/paps.cc &amp;&amp;
> > +patch -Np1 -i ../77.patch
> > +</userinput></screen>
> >   
> >       <para>
> >         Install <application>paps</application> by running the
> > following
> 
> [snip]
> 
> I'm confused here, why are we pulling a patch from GitHub rather than 
> adding it to our repository and following our standard?

Even if we could change our standard, the name "77.patch" is just stupid
and we shouldn't allow it ever.

-- 
Xi Ruoyao <[email protected]>

-- 
http://lists.linuxfromscratch.org/sympa/info/blfs-dev
Unsubscribe: See the above information page