See below On Wed, Feb 17, 2010 at 4:46 PM, FUJITA Tomonori <fujita.tomonori at lab.ntt.co.jp> wrote: > On Wed, 17 Feb 2010 11:16:43 +1100 > ronnie sahlberg <ronniesahlberg at gmail.com> wrote: ... >> --- >> doc/Makefile | 17 ++++++- >> doc/htmlpages/tgtd.8.html | 40 +++++++++++++++ >> doc/manpages/tgtd.8 | 81 +++++++++++++++++++++++++++++ > > Do we need to add tgtd.8.html and tgtd.8? I thought that xsltproc > generates them from tgtd.8.xml? They are needed for the case when the user does not have xsltproc installed and configured. So for those users, they can still "make" and "make install" and these files are installed. These users just get the pregenerated ones that came from the git tree instead of updating/creating these files. Otherwise these users wouldn't get the docs, and forcing them to install xsltproc would be "rude". This way you only need to install xsltproc if you want to update and rebuild the docs. ... >> +manpages/tgtd.8: tgtd.8.xml >> + -test -z "$(XSLTPROC)" || $(XSLTPROC) -o $@ http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl $< >> + >> +htmlpages/tgtd.8.html: tgtd.8.xml >> + -test -z "$(XSLTPROC)" || $(XSLTPROC) -o $@ http://docbook.sourceforge.net/release/xsl/current/html/docbook.xsl $< > > It means that it tries to connect to docbook.sf.net? If so, can we use > the local files instead? Good point. I have updated the makefile to use the local files. Attached is a new patch. regards ronnei sahlberg -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-Add-documentation-about-the-tgtd-daemon-in-XSLT-form.patch.gz Type: application/x-gzip Size: 3482 bytes Desc: not available URL: <http://lists.wpkg.org/pipermail/stgt/attachments/20100217/d08d9341/attachment.bin> -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-Add-documentation-about-the-tgtd-daemon-in-XSLT-form.patch Type: text/x-diff Size: 11327 bytes Desc: not available URL: <http://lists.wpkg.org/pipermail/stgt/attachments/20100217/d08d9341/attachment.patch> |