Boost C++ Libraries: Ticket #4872: Docs don't build corectly

Frank Mori Hess — Tue, 23 Nov 2010 04:07:45 GMT

(In [66694]) Patch from John Maddock: http://lists.boost.org/Archives/boost/2010/11/173288.php

owner, component, severity changed

Frank Mori Hess — Tue, 23 Nov 2010 04:19:27 GMT

owner changed from Frank Mori Hess to Vladimir Prus
component signals2 → build
severity Showstopper → Problem

I applied the patch posted by John Maddock which works around the bug for the signals2 doc: http://lists.boost.org/Archives/boost/2010/11/173288.php. However, according to the report the bug is in Boost.Build when compiling on msvc in debug mode. In the following line from libs/signals2/doc/snippet-extractor.jam:

$(EXTRACTOR-COMMAND) "$(<:D)" $(>)

the ":D" doesn't result in a directory as it should on msvc in debug mode.

Vladimir Prus — Tue, 23 Nov 2010 06:55:42 GMT

It seems to me that :D modifier is working as expected, and the problem is that $(EXTRACTOR-COMMAND) expands to two paths -- path to exe and path to pdb. Does $(EXTRACTOR-COMMAND[1]) fix the problem?

Frank Mori Hess — Tue, 23 Nov 2010 15:36:47 GMT

I changed EXTRACTOR_COMMAND to EXTRACTOR_COMMAND[1] locally and verified that signals2 doc build still works correctly under Linux after the change. I don't have msvc to verify it resolves the msvc/debug problem there though. Is EXTRACTOR-COMMAND[1] what I was supposed to be doing (that is, should I commit the change to svn)?

anonymous — Tue, 23 Nov 2010 16:54:41 GMT

I just tested that with msvc-10 on Windows, and yes it's working there too:

actions extract-snippets bind EXTRACTOR-COMMAND
{
    $(EXTRACTOR-COMMAND[1]) "$(<:D)" $(>)
}

HTH, John.

Steven Watanabe — Wed, 06 Apr 2011 23:52:27 GMT

(In [71058]) Make sure that the command works even in debug mode. Refs #4872.

status changed; resolution set

Steven Watanabe — Wed, 06 Apr 2011 23:56:24 GMT

status new → closed
resolution → fixed

This is a pretty simple thing, so I checked it with msvc and gcc using both debug and release and committed it. (I temporarily reverted John's other patch to test debug).