Charles Lepple
2010-Nov-05 02:04 UTC
[Nut-upsdev] [nut-commits] svn commit r2669 - in branches/AsciiDoc: . m4
On Nov 4, 2010, at 6:21 PM, Arjen de Korte wrote:> Author: adkorte-guest > Date: Thu Nov 4 22:21:51 2010 > New Revision: 2669 > URL: http://trac.networkupstools.org/projects/nut/changeset/2669 > > Log: > Add checking minimum required version for AsciiDoc toolsI noticed that somewhere around this commit, the documentation stopped being built by default in BuildBot. Before I go digging too far, should I just add "--with-doc=auto" to the configure parameters, or should that eventually be included in the "--with-all=auto" settings? -- Charles Lepple
Arjen de Korte
2010-Nov-05 07:57 UTC
[Nut-upsdev] [nut-commits] svn commit r2669 - in branches/AsciiDoc: . m4
Citeren Charles Lepple <clepple op gmail.com>:>> Log: >> Add checking minimum required version for AsciiDoc tools > > I noticed that somewhere around this commit, the documentation > stopped being built by default in BuildBot. > > Before I go digging too far, should I just add "--with-doc=auto" to > the configure parameters, or should that eventually be included in > the "--with-all=auto" settings?We already have '--with-doc=auto' on the configure commandline, so this isn't needed. Howevere, there seems to be a problem interpreting the versions returned by the AsciiDoc tools (from the r2672 Debian-etch build, similar for the other buildbots):> checking for asciidoc... /usr/local/bin/asciidoc > checking for asciiDoc version (8.6.3 minimum required)... asciidoc > 8.6.3 foundHere it works as expected. Even though the version string we're comparing against is 'asciidoc 8.6.3', the AX_COMPARE_VERSION macro seems to understand it should skip the initial non-numerals.> checking for a2x... /usr/local/bin/a2x > checking for a2x version (8.6.1 minimum required)... a2x 8.6.1 is too oldOops. The above causes the 'nut_have_asciidoc' variable to be set to 'no', which means that the prerequisites for building the documentation are not met and it will not be build regardless of any configure setting.> checking for dblatex... /usr/bin/dblatex > checking for dblatex version... dblatex version 0.2-2 foundHere we only report the version (nothing more) so this isn't a problem.> checking whether to build and install documentation... noAnd finally, the decision is made not to build the documentation. Most likely we'll need to strip the initial part of the return from the 'a2x 8.6.1' version string, since that will probably be treated as '2.8.6.1' now (it works when I lower the requirement to 1.0.0). I'll see what I can do... Best regards, Arjen -- Please keep list traffic on the list (off-list replies will be rejected)