Boost C++ Libraries: Ticket #5709: boost.build docutils module not working

Vladimir Prus — Mon, 18 Jul 2011 05:54:00 GMT

Stefan,

as I've said on IRC, your first step should be to add

ECHO "Here I am" ;

as the first statement of docbook.init, and make sure it is executed. If it's not executed, it means you are modifying wrong docbook.init, or you don't have 'using docbook', or you have 'using docbook' in wrong user-config.jam, or something similar.

After then, you should add

RST2XXX = [ common.find-tool rst2html ] ;

as the last stateme of the "if" inside docbook.init.

Stefan Seefeld — Mon, 18 Jul 2011 11:13:06 GMT

Volodya,

I have modified tools/build/v2/tools/docutils.jam, and I can in fact see changes when I run 'bjam libs/python/doc', just not all, and just not with the effect I had hoped I would see.

In particular:

Into the 'init' rule, inside 'if ! $(.initialized)', I added:

RST2XXX = [ common.find-tool rst2html ] ;

ECHO "init: " $(RST2XXX) ;

but that didn't print anything.

Into the 'html' rule (after the RST2XXX definition) I added:

ECHO "rst2xxx: " $(RST2XXX) ;

which only prints 'rst2xxx:', i.e. no value.

Into the 'actions html' block, I added:

ECHO "$(RST2XXX)"

which generates an error '/bin/sh: line 1: ECHO: command not found', unless I invoke bjam with '-n', in which case the error disappears. (It still prints the '/usr/bin/python2.7' value that it seems to get from the (wrong) assignment in the html rule above.

Thanks for your help,

attachment set

Stefan Seefeld — Mon, 18 Jul 2011 12:25:56 GMT

attachment → patch

This fixes the issue.

Stefan Seefeld — Mon, 18 Jul 2011 12:26:59 GMT

Volodya,

the attached patch appears to be all that's needed to fix this, right ? Should I check it in ?

Thanks,

Vladimir Prus — Mon, 18 Jul 2011 12:35:09 GMT

Stefan,

this patch is fine, thanks.

You might want to include "Fixes #5709" in the commit message to have this issue closed by magic.

Thanks, Volodya

status changed; resolution set

Stefan Seefeld — Mon, 18 Jul 2011 12:37:08 GMT

status new → closed
resolution → fixed

(In [73215]) Fixes #5709