Обсуждение: Another small patch...

... this time, for maintenance.sgml.

Regards.


--
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com
Index: doc/src/sgml/maintenance.sgml
===================================================================
RCS file: /projects/cvsroot/pgsql/doc/src/sgml/maintenance.sgml,v
retrieving revision 1.82
diff -r1.82 maintenance.sgml
63c63
<    linkend="sql-vacuum"> command has to run on a regular basis for several
---
>    linkend="sql-vacuum">) command has to run on a regular basis for several

Guillaume Lelarge wrote:
> ... this time, for maintenance.sgml.

Applied, thanks.  I'm wondering whether this should be

    <para>
!    <productname>PostgreSQL</productname>'s
!    <xref linkend="sql-vacuum" endterm="sql-vacuum-title"> command has to run on
!    a regular basis for several reasons:

instead.  I didn't try to compile it so maybe the output is not OK.

--
Alvaro Herrera                               http://www.PlanetPostgreSQL.org/
"The only difference is that Saddam would kill you on private, where the
Americans will kill you in public" (Mohammad Saleh, 39, a building contractor)

Alvaro Herrera wrote:
> Guillaume Lelarge wrote:
> > ... this time, for maintenance.sgml.
>
> Applied, thanks.  I'm wondering whether this should be
>
>     <para>
> !    <productname>PostgreSQL</productname>'s
> !    <xref linkend="sql-vacuum" endterm="sql-vacuum-title"> command has to run on
> !    a regular basis for several reasons:
>
> instead.  I didn't try to compile it so maybe the output is not OK.

OK, I need opinions on this issue.  We currently have output that looks
like

"PostgreSQL's Vacuum (/Section 10.4.2/) command has to run ..."
(the part between // is a clickable link)

but with the patch above, it would show up as
"PostgreSQL's /Vacuum/ command has to run ..."

Is there a preferred way to link?  I notice there is plenty of usage of
both forms.

I wonder what does printed output look like?  Does it just just lose the
reference?

--
Alvaro Herrera                          Developer, http://www.PostgreSQL.org/
Al principio era UNIX, y UNIX habló y dijo: "Hello world\n".
No dijo "Hello New Jersey\n", ni "Hello USA\n".

Alvaro Herrera <alvherre@commandprompt.com> writes:
> OK, I need opinions on this issue.  We currently have output that looks
> like

> "PostgreSQL's Vacuum (/Section 10.4.2/) command has to run ..."
> (the part between // is a clickable link)

> but with the patch above, it would show up as
> "PostgreSQL's /Vacuum/ command has to run ..."

> Is there a preferred way to link?

I'd vote for the latter, at least in cases like this.  I think here
you're just providing a courtesy link that you don't expect people
to follow all that often, and so minimizing its visual impact is
a good thing.

            regards, tom lane