pg.ddx.io pgsql-docs@postgresql.org mailing list archive
help / color / mirror / Atom feed[PATCH] fix tool list for FreeBSD
4+ messages / 2 participants
[nested] [flat]
* [PATCH] fix tool list for FreeBSD
@ 2014-10-16 19:12 Dag-Erling Smørgrav <des@des.no>
0 siblings, 1 reply; 4+ messages in thread
From: Dag-Erling Smørgrav @ 2014-10-16 19:12 UTC (permalink / raw)
To: pgsql-docs
The list of tools required to build the documentation on FreeBSD is
outdated. The following patch fixes it:
inline patch 1: doc/src/sgml/docguide.sgml (+2/-5 lines) download inline-1.patch
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
--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: [PATCH] fix tool list for FreeBSD
@ 2014-10-20 17:59 Peter Eisentraut <peter_e@gmx.net>
parent: Dag-Erling Smørgrav <des@des.no>
0 siblings, 1 reply; 4+ messages in thread
From: Peter Eisentraut @ 2014-10-20 17:59 UTC (permalink / raw)
To: Dag-Erling Smørgrav <des@des.no>; pgsql-docs
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.
--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: [PATCH] fix tool list for FreeBSD
@ 2014-10-20 19:39 Dag-Erling Smørgrav <des@des.no>
parent: Peter Eisentraut <peter_e@gmx.net>
0 siblings, 1 reply; 4+ messages in thread
From: Dag-Erling Smørgrav @ 2014-10-20 19:39 UTC (permalink / raw)
To: Peter Eisentraut <peter_e@gmx.net>; +Cc: pgsql-docs
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
--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
^ permalink raw reply [nested|flat] 4+ messages in thread
* Re: [PATCH] fix tool list for FreeBSD
@ 2014-10-21 14:48 Peter Eisentraut <peter_e@gmx.net>
parent: Dag-Erling Smørgrav <des@des.no>
0 siblings, 0 replies; 4+ messages in thread
From: Peter Eisentraut @ 2014-10-21 14:48 UTC (permalink / raw)
To: Dag-Erling Smørgrav <des@des.no>; +Cc: pgsql-docs
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.
--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs
^ permalink raw reply [nested|flat] 4+ messages in thread
end of thread, other threads:[~2014-10-21 14:48 UTC | newest]
Thread overview: 4+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2014-10-16 19:12 [PATCH] fix tool list for FreeBSD Dag-Erling Smørgrav <des@des.no>
2014-10-20 17:59 ` Peter Eisentraut <peter_e@gmx.net>
2014-10-20 19:39 ` Dag-Erling Smørgrav <des@des.no>
2014-10-21 14:48 ` Peter Eisentraut <peter_e@gmx.net>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox