<div class="gmail_quote">On Mon, Jan 11, 2010 at 9:35 AM, Gaetan Nadon <span dir="ltr"><<a href="mailto:memsize@videotron.ca">memsize@videotron.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div><div></div><div class="h5">
On Sun, 2010-01-10 at 14:06 -0800, Keith Packard wrote:
<blockquote type="CITE">
<pre>On Sun, 10 Jan 2010 16:06:51 -0500, Gaetan Nadon <<a href="mailto:memsize@videotron.ca" target="_blank">memsize@videotron.ca</a>> wrote:
> The dmx and scaled text files are generated from sgml files.
> The linuxdoc program is used to generated them.
We've generally been distributing the .txt files with the .tar.gz
package so that people don't have to figure out how to get linuxdoc or
doxygen to run. That will take some additional effort to get that to
happen when doing 'make distcheck' which I do for each release.
</pre>
</blockquote>
<br></div></div>
The .txt generated by linuxdoc isn't a problem so as long as it isn't built by default, which is the case now. We can ignore this patch.<br>
<br>
The html generated by doxygen are a problem. It breaks "make dist". By default, html files are not generated, so the tarball creation fails.<br>
<br>
<blockquote>
<pre>hw/dmx/doc
make[3]: *** No rule to make target `html/ChkNotMaskEv_8h_source.html', needed by `distdir'. Stop.
</pre>
</blockquote>
<br>
This is the subject of the other patch (which I need to reparent):<br>
<br>
<blockquote>
<table cellpadding="0">
<tbody><tr>
<th nowrap align="right" valign="top">
<b> </b>
</th>
<td>
[PATCH xserver] dmx-doc: do not distribute generated html files.
</td>
</tr>
</tbody></table>
</blockquote>
It won't stop a knowledgeable person from creating a tarball for a release, but it discourages developers from routinely running dist or distcheck as a good free test case. <br></div></blockquote><div><br></div><div>
Add</div><div><br></div><div> DISTCHECK_CONFIGURE_FLAGS = --enable-docs</div><div><br></div><div>(or whatever the ./configure option is) to the toplevel Makefile.am?</div><div><br></div><div>Kristian</div></div>