Thread: [PATCH] fix tool list for FreeBSD

[PATCH] fix tool list for FreeBSD

From
Dag-Erling Smørgrav
Date:
The list of tools required to build the documentation on FreeBSD is
outdated.  The following patch fixes it:

diff --git a/doc/src/sgml/docguide.sgml b/doc/src/sgml/docguide.sgml
index 3cc1d60..9626592 100644
--- a/doc/src/sgml/docguide.sgml
+++ b/doc/src/sgml/docguide.sgml
@@ -219,10 +219,7 @@
     documentation on FreeBSD.
     <itemizedlist>
      <listitem>
-      <para><filename>textproc/sp</filename></para>
-     </listitem>
-     <listitem>
-      <para><filename>textproc/openjade</filename></para>
+      <para><filename>textproc/jade</filename></para>
      </listitem>
      <listitem>
       <para><filename>textproc/iso8879</filename></para>
@@ -231,7 +228,7 @@
       <para><filename>textproc/dsssl-docbook-modular</filename></para>
      </listitem>
      <listitem>
-      <para><filename>textproc/docbook-420</filename></para>
+      <para><filename>textproc/docbook-sgml</filename></para>
      </listitem>
     </itemizedlist>
    </para>

DES
--
Dag-Erling Smørgrav - des@des.no


Re: [PATCH] fix tool list for FreeBSD

From
Peter Eisentraut
Date:
On 10/16/14 3:12 PM, Dag-Erling Smørgrav wrote:
> The list of tools required to build the documentation on FreeBSD is
> outdated.  The following patch fixes it:
>
> diff --git a/doc/src/sgml/docguide.sgml b/doc/src/sgml/docguide.sgml
> index 3cc1d60..9626592 100644
> --- a/doc/src/sgml/docguide.sgml
> +++ b/doc/src/sgml/docguide.sgml
> @@ -219,10 +219,7 @@
>      documentation on FreeBSD.
>      <itemizedlist>
>       <listitem>
> -      <para><filename>textproc/sp</filename></para>
> -     </listitem>
> -     <listitem>
> -      <para><filename>textproc/openjade</filename></para>
> +      <para><filename>textproc/jade</filename></para>
>       </listitem>
>       <listitem>
>        <para><filename>textproc/iso8879</filename></para>

This replaces openjade by jade.  Why?  (Arguably, we should recommend
opensp if we're going to suggest openjade.)

> @@ -231,7 +228,7 @@
>        <para><filename>textproc/dsssl-docbook-modular</filename></para>
>       </listitem>
>       <listitem>
> -      <para><filename>textproc/docbook-420</filename></para>
> +      <para><filename>textproc/docbook-sgml</filename></para>
>       </listitem>
>      </itemizedlist>
>     </para>

That looks correct.  We should also remove iso8879, since it's pulled in
by docbook-sgml.

And we should add docbook-xml, docbook-xsl, and libxslt.

Also, the piece about the catalog file not being updated is surely outdated?

Ironically, following the link to the FreeBSD Handbook shows the same
problem:  They are linking to textproc/docbook-xml-450, which no longer
exists.



Re: [PATCH] fix tool list for FreeBSD

From
Dag-Erling Smørgrav
Date:
Peter Eisentraut <peter_e@gmx.net> writes:
> This replaces openjade by jade.  Why?  (Arguably, we should recommend
> opensp if we're going to suggest openjade.)

More or less by accident.  Neither was installed when I ran configure,
so the Makefile looked for jade, not openjade.  In either case, you can
remove textproc/sp; it is included in textproc/jade, while
textproc/openjade has a dependency on textproc/opensp.

> And we should add docbook-xml, docbook-xsl, and libxslt.

Do you have XML documentation?  I didn't look around to much, the files
I needed to edit were SGML.  I already had the XML tools, since I have
textproc/docproj installed.

> Also, the piece about the catalog file not being updated is surely
> outdated?

It worked fine, but I didn't want to remove it since I didn't know for
sure that it would work for everybody.

DES
--
Dag-Erling Smørgrav - des@des.no


Re: [PATCH] fix tool list for FreeBSD

From
Peter Eisentraut
Date:
On 10/20/14 3:39 PM, Dag-Erling Smørgrav wrote:
> Peter Eisentraut <peter_e@gmx.net> writes:
>> This replaces openjade by jade.  Why?  (Arguably, we should recommend
>> opensp if we're going to suggest openjade.)
>
> More or less by accident.  Neither was installed when I ran configure,
> so the Makefile looked for jade, not openjade.  In either case, you can
> remove textproc/sp; it is included in textproc/jade, while
> textproc/openjade has a dependency on textproc/opensp.

I have updated the documentation based on your information.

>> And we should add docbook-xml, docbook-xsl, and libxslt.
>
> Do you have XML documentation?  I didn't look around to much, the files
> I needed to edit were SGML.  I already had the XML tools, since I have
> textproc/docproj installed.

Yes, it's used for building man pages.

>> Also, the piece about the catalog file not being updated is surely
>> outdated?
>
> It worked fine, but I didn't want to remove it since I didn't know for
> sure that it would work for everybody.

I have removed this.  It was already a just-in-case at the time it was
written.