Bug 184527

Summary:	/releng/9.2/release/doc/en_US.ISO8859-1/relnotes/article.xml references freebsd45.dtd, not freebsd50.dtd
Product:	Documentation	Reporter:	slvr32
Component:	Books & Articles	Assignee:	Glen Barber <gjb>
Status:	Closed FIXED
Severity:	Affects Only Me
Priority:	Normal
Version:	Latest
Hardware:	Any
OS:	Any

Description slvr32 2013-12-06 01:40:00 UTC

When attempting to build a releng/9.2 release, errors referencing 'freebsd45.dtd' (should be freebsd50.dtd now?) cause docs build (and ultimate the rest of the release build) to fail, e.g.

# cd /usr/src/release
# sh generate-release.sh releng/9.2 /var/9.2-p2

eventually errors out with a dtd error related to not being to able to pull in freebsd45.dtd, which leads to undefined entities and other XML-related validation errors.

Fix: 

/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml should reference "../../../share/xml/freebsd50.dtd" (as in base/head now?)
How-To-Repeat: # cd /usr/src/release
# sh generate-release.sh releng/9.2 /var/9.2-p2

Comment 1 slvr32 2013-12-11 17:51:17 UTC

cd /usr/src/release/doc && make all install clean 'FORMATS=html txt'
INSTALL_COMPRESSED='' URLS_ABSOLUTE=YES
DOCDIR=/usr/obj/usr/src/release/rdoc
===> en_US.ISO8859-1 (all)
===> en_US.ISO8859-1/relnotes (all)
install /usr/doc/share/xml/catalog-cwd.xml
/usr/src/release/doc/en_US.ISO8859-1/relnotes/catalog-cwd.xml
echo '<!ENTITY base "../../../../..">' >>
/usr/src/release/doc/en_US.ISO8859-1/relnotes/autogen.ent
env XML_CATALOG_FILES="file:///usr/src/release/doc/en_US.ISO8859-1/relnotes/catalog-cwd.xml
 file:///usr/doc/en_US.ISO8859-1/share/xml/catalog.xml
file:///usr/doc/share/xml/catalog.xml
file:///usr/doc/share/xml/catalog-common.xml
file:///usr/local/share/xml/catalog
file:///usr/src/release/doc/en_US.ISO8859-1/relnotes/../../en_US.ISO8859-1/share/xml/catalog.xml
 file:///usr/src/release/doc/en_US.ISO8859-1/relnotes/../../share/xml/catalog.xml"
/usr/local/bin/xmllint --nonet --noent --valid --dropdtd --xinclude
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml >
article.parsed.xml.tmp
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:6: warning:
failed to load external entity
"/usr/src/release/share/xml/freebsd45.dtd"
]>
  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:10: element
title: validity error : No declaration for element title
    <title>&os; &release.current; Release Notes</title>
                                                       ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:12: element
corpauthor: validity error : No declaration for element corpauthor
    <corpauthor>The &os; Project</corpauthor>
                                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:14: element
pubdate: validity error : No declaration for element pubdate
/en_US.ISO8859-1/relnotes/article.xml 255888 2013-09-26 15:46:32Z gjb $</pubdate
                                                                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:17: element
year: validity error : No declaration for element year
      <year>2013</year>
                       ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:18: element
holder: validity error : No declaration for attribute role of element
holder
      <holder role="mailto:doc@FreeBSD.org">The &os; Documentation Project</hold
                                           ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:18: element
holder: validity error : No declaration for element holder
    <holder role="mailto:doc@FreeBSD.org">The &os; Documentation Project</holder
                                                                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:19: element
copyright: validity error : No declaration for element copyright
    </copyright>
                ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:21: element
legalnotice: validity error : No declaration for attribute id of
element legalnotice
    <legalnotice id="trademarks" role="trademarks">
                                                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:21: element
legalnotice: validity error : No declaration for attribute role of
element legalnotice
    <legalnotice id="trademarks" role="trademarks">
                                                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:22: parser
error : Entity 'tm-attrib.freebsd' not defined
      &tm-attrib.freebsd;
                         ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:23: parser
error : Entity 'tm-attrib.ibm' not defined
      &tm-attrib.ibm;
                     ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:24: parser
error : Entity 'tm-attrib.ieee' not defined
      &tm-attrib.ieee;
                      ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:25: parser
error : Entity 'tm-attrib.intel' not defined
      &tm-attrib.intel;
                       ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:26: parser
error : Entity 'tm-attrib.sparc' not defined
      &tm-attrib.sparc;
                       ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:27: parser
error : Entity 'tm-attrib.general' not defined
      &tm-attrib.general;
                         ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:28: element
legalnotice: validity error : No declaration for element legalnotice
    </legalnotice>
                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:37: element
para: validity error : No declaration for element para
    Some brief remarks on upgrading are also presented.</para>
                                                              ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:38: element
abstract: validity error : No declaration for element abstract
    </abstract>
               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:39: element
articleinfo: validity error : No declaration for element articleinfo
  </articleinfo>
                ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:41: element
sect1: validity error : No declaration for attribute id of element
sect1
  <sect1 id="intro">
                   ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:42: element
title: validity error : No declaration for element title
    <title>Introduction</title>
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:47: element
para: validity error : No declaration for element para
      upgrading from previous versions of &os;.</para>
                                                      ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:51: element
ulink: validity error : No declaration for attribute url of element
ulink
      url="&release.url;"></ulink> or any of its mirrors.  More
                         ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:51: element
ulink: validity error : No declaration for element ulink
      url="&release.url;"></ulink> or any of its mirrors.  More
                                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:54: parser
error : Entity 'url.books.handbook' not defined
      url="&url.books.handbook;/mirrors.html"><quote>Obtaining
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:54: element
ulink: validity error : No declaration for attribute url of element
ulink
      url="&url.books.handbook;/mirrors.html"><quote>Obtaining
                                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:55: element
quote: validity error : No declaration for element quote
      &os;</quote> appendix</ulink> to the <ulink
                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:55: element
ulink: validity error : No declaration for element ulink
      &os;</quote> appendix</ulink> to the <ulink
                                   ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:56: parser
error : Entity 'url.books.handbook' not defined
      url="&url.books.handbook;/">&os; Handbook</ulink>.</para>
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:56: element
ulink: validity error : No declaration for attribute url of element
ulink
      url="&url.books.handbook;/">&os; Handbook</ulink>.</para>
                                 ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:56: element
ulink: validity error : No declaration for element ulink
      url="&url.books.handbook;/">&os; Handbook</ulink>.</para>
                                                       ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:56: element
para: validity error : No declaration for element para
      url="&url.books.handbook;/">&os; Handbook</ulink>.</para>
                                                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:60: element
quote: validity error : No declaration for element quote
      <quote>late-breaking</quote> information discovered late in the
                                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:64: element
para: validity error : No declaration for element para
      &release.current; can be found on the &os; Web site.</para>
                                                                 ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:65: element
sect1: validity error : No declaration for element sect1
  </sect1>
          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:67: element
sect1: validity error : No declaration for attribute id of element
sect1
  <sect1 id="new">
                 ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:68: element
title: validity error : No declaration for element title
    <title>What's New</title>
                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:71: element
para: validity error : No declaration for element para
      features in &os; since &release.prev;.</para>
                                                   ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:81: element
para: validity error : No declaration for element para
      improvements.</para>
                          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:85: element
ulink: validity error : No declaration for attribute url of element
ulink
    url="http://www.freebsd.org/releases/9.2R/relnotes-detailed.html">here</ulink>.
                                                                     ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:85: element
ulink: validity error : No declaration for element ulink
    url="http://www.freebsd.org/releases/9.2R/relnotes-detailed.html">here</ulink>.

           ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:85: element
para: validity error : No declaration for element para
http://www.freebsd.org/releases/9.2R/relnotes-detailed.html">here</ulink>.</para
                                                                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:87: element
sect2: validity error : No declaration for attribute id of element
sect2
    <sect2 id="security">
                        ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:88: element
title: validity error : No declaration for element title
      <title>Security Advisories</title>
                                        ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:93: element
ulink: validity error : No declaration for attribute url of element
ulink
    <ulink url="http://security.FreeBSD.org/"></ulink>.</para>
                                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:93: element
ulink: validity error : No declaration for element ulink
    <ulink url="http://security.FreeBSD.org/"></ulink>.</para>
                                                      ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:93: element
para: validity error : No declaration for element para
    <ulink url="http://security.FreeBSD.org/"></ulink>.</para>
                                                              ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:95: element
informaltable: validity error : No declaration for attribute frame of
element informaltable
      <informaltable frame="none" pgwide="1">
                                            ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:95: element
informaltable: validity error : No declaration for attribute pgwide of
element informaltable
      <informaltable frame="none" pgwide="1">
                                            ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:96: element
tgroup: validity error : No declaration for attribute cols of element
tgroup
    <tgroup cols="3">
                    ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:97: element
colspec: validity error : No declaration for attribute colwidth of
element colspec
      <colspec colwidth="1*" />
                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:97: element
colspec: validity error : No declaration for element colspec
      <colspec colwidth="1*" />
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:98: element
colspec: validity error : No declaration for attribute colwidth of
element colspec
      <colspec colwidth="1*" />
                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:98: element
colspec: validity error : No declaration for element colspec
      <colspec colwidth="1*" />
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:99: element
colspec: validity error : No declaration for attribute colwidth of
element colspec
      <colspec colwidth="3*" />
                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:99: element
colspec: validity error : No declaration for element colspec
      <colspec colwidth="3*" />
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:102: element
entry: validity error : No declaration for element entry
          <entry>Advisory</entry>
                                 ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:103: element
entry: validity error : No declaration for element entry
          <entry>Date</entry>
                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:104: element
entry: validity error : No declaration for element entry
          <entry>Topic</entry>
                              ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:105: element
row: validity error : No declaration for element row
        </row>
              ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:106: element
thead: validity error : No declaration for element thead
      </thead>
              ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:110: element
entry: validity error : No declaration for element entry
          <entry>No security advisories to display.</entry>
                                                           ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:111: parser
error : Entity 'nbsp' not defined
          <entry>&nbsp;</entry>
                       ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:111: element
entry: validity error : No declaration for element entry
          <entry>&nbsp;</entry>
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:112: parser
error : Entity 'nbsp' not defined
          <entry>&nbsp;</entry>
                       ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:112: element
entry: validity error : No declaration for element entry
          <entry>&nbsp;</entry>
                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:113: element
row: validity error : No declaration for element row
        </row>
              ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:114: element
tbody: validity error : No declaration for element tbody
      </tbody>
              ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:115: element
tgroup: validity error : No declaration for element tgroup
    </tgroup>
             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:116: element
informaltable: validity error : No declaration for element
informaltable
      </informaltable>
                      ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:117: element
sect2: validity error : No declaration for element sect2
    </sect2>
            ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:118: element
sect1: validity error : No declaration for element sect1
  </sect1>
          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:120: element
sect1: validity error : No declaration for attribute id of element
sect1
  <sect1 id="upgrade">
                     ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:121: element
title: validity error : No declaration for element title
    <title>Upgrading from previous releases of &os;</title>
                                                           ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:124: parser
error : Entity 'man.freebsd-update.8' not defined
      <title>Upgrading using &man.freebsd-update.8; or a source-based
                                                   ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:125: element
title: validity error : No declaration for element title
    procedure</title>
                     ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:127: element
para: validity error : No declaration for attribute arch of element
para
      <para arch="amd64,i386">Beginning with &os; 6.2-RELEASE, binary
                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:130: parser
error : Entity 'man.freebsd-update.8' not defined
    &man.freebsd-update.8; utility.  The binary upgrade procedure
                          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:132: element
filename: validity error : No declaration for element filename
    unmodified <filename>GENERIC</filename> kernel, distributed as
                                           ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:134: parser
error : Entity 'man.freebsd-update.8' not defined
    &man.freebsd-update.8; utility requires that the host being
                          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:135: element
para: validity error : No declaration for element para
    upgraded have Internet connectivity.</para>
                                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:140: element
filename: validity error : No declaration for element filename
    <filename>/usr/src/UPDATING</filename>.</para>
                                          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:140: element
para: validity error : No declaration for element para
    <filename>/usr/src/UPDATING</filename>.</para>
                                                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:144: element
ulink: validity error : No declaration for attribute url of element
ulink
    url="http://www.FreeBSD.org/releases/9.2R/installation.html"></ulink>.</para>
                                                                ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:144: element
ulink: validity error : No declaration for element ulink
    url="http://www.FreeBSD.org/releases/9.2R/installation.html"></ulink>.</para>
                                                                         ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:144: element
para: validity error : No declaration for element para
    url="http://www.FreeBSD.org/releases/9.2R/installation.html"></ulink>.</para>

          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:148: element
emphasis: validity error : No declaration for element emphasis
      after backing up <emphasis>all</emphasis> data and
                                               ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:149: element
para: validity error : No declaration for element para
      configuration files.</para>
                                 ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:150: element
important: validity error : No declaration for element important
      </important>
                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:151: element
sect2: validity error : No declaration for element sect2
    </sect2>
            ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:154: element
title: validity error : No declaration for attribute id of element
title
      <title id="upgrade-pitfalls">User-visible incompatibilities</title>
                                  ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:154: element
title: validity error : No declaration for element title
      <title id="upgrade-pitfalls">User-visible incompatibilities</title>
                                                                         ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:160: element
ulink: validity error : No declaration for attribute url of element
ulink
    url="http://www.freebsd.org/releases/9.0R/relnotes-detailed.html#UPGRADE">Upgra
                                                                             ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:161: element
ulink: validity error : No declaration for element ulink
    Section in 9.0-RELEASE Release Notes</ulink> carefully before
                                                ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:163: element
emphasis: validity error : No declaration for element emphasis
    FreeBSD mailing lists.</emphasis></para>
                                     ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:163: element
para: validity error : No declaration for element para
    FreeBSD mailing lists.</emphasis></para>
                                            ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:164: element
sect2: validity error : No declaration for element sect2
    </sect2>
            ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:165: element
sect1: validity error : No declaration for element sect1
  </sect1>
          ^
/usr/src/release/doc/en_US.ISO8859-1/relnotes/article.xml:166: element
article: validity error : No declaration for element article
</article>
          ^
*** [article.parsed.xml] Error code 4

Stop in /usr/src/release/doc/en_US.ISO8859-1/relnotes.
*** [all] Error code 1

Stop in /usr/src/release/doc/en_US.ISO8859-1.
*** [all] Error code 1

Stop in /usr/src/release/doc.
*** [reldoc] Error code 1

Stop in /usr/src/release.
*** [release] Error code 1

Stop in /usr/src/release.

Comment 2 Glen Barber freebsd_committer

2013-12-11 19:06:37 UTC

State Changed
From-To: open->closed

We will not fix this, as it is not a security issue. 

Using the ports tree at the time of the 9.2-RELEASE will work 
(ports/branches/RELENG_9_2_0). 

This is fixed in stable/9 and will be a non-issue for 9.3-RELEASE.

Comment 3 Glen Barber freebsd_committer

2013-12-11 19:06:37 UTC

Responsible Changed
From-To: freebsd-doc->gjb

We will not fix this, as it is not a security issue. 

Using the ports tree at the time of the 9.2-RELEASE will work 
(ports/branches/RELENG_9_2_0). 

This is fixed in stable/9 and will be a non-issue for 9.3-RELEASE.

Comment 4 slvr32 2013-12-11 21:26:51 UTC

Well, it's disappointing that this won't be fixed.

It's extremely useful to build releases (and produce the related
iso's) at an updated patch level, so that all 'new installs' of
FreeBSD don't (have to) start at -RELEASE.

I've personally been building FreeBSD releases this way for several
years (since before the 'generate-release.sh' script was introduced),
so I'd call this issue a regression, whether or not the broken release
build in this case is a security issue.

Thanks,
- Jason