Re: [jamboost] tools/build/tools/boostbook.jam (rev 1.5) broken?

From: Vladimir Prus (ghost_at_[hidden])
Date: 2003-07-24 01:18:51

Hi Eric,

Eric Friedman wrote:
> In the past few days I've been getting the error output (below) when
> trying to build BoostBook using bjam 3.1.6. Searching through the
> Boost CVS, I found that tools/build/tools/boostbook.jam was the only
> BoostBook-related change since my last update. Indeed, using rev 1.4
> of the file causes the errors to disappear and everything once again
> functions properly.
>
> I know very little about Boost.Build and bjam, however, so I'm not
> sure what might be wrong with rev 1.5. If anyone on this list more
> who uses BoostBook could look into this, I'd greatly appreciate it.

> xslt-xsltproc bin\gcc\debug\boost.docbook
> warning: failed to load external entity "docbook.xsl"
> cannot parse docbook.xsl

I get similiar error now. And rev 1.4 works. The reason is that XML catalog is 
never created. The rev 1.5 marked catalog as TEMPORARY, and that did not 
work. I'm actually at loss what effect TEMPORARY has in V2 (or in jam, 
actually), so we probably should not use it at all. 

And, BTW, there's another bug in rev 1.6. I've just comitted the attached 
patch. Please try again.

- Volodya

 --Boundary-00=_Lp3H/7vqpHQjjY1 Content-Type: text/x-diff;
charset="iso-8859-1";
name="boostbook.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="boostbook.diff"

Index: boostbook.jam
===================================================================
RCS file: /cvsroot/boost/boost/tools/build/tools/boostbook.jam,v
retrieving revision 1.6
diff -u -r1.6 boostbook.jam
--- boostbook.jam	17 Jul 2003 15:34:28 -0000	1.6
+++ boostbook.jam	24 Jul 2003 06:17:08 -0000
@@ -94,9 +94,14 @@
your boost installation. Searched in:
: $(search-dirs:J="
") ;
- 
- }
- }
+ } 
+ # Add trailing slash since some other code 
+ # uses :B modifier to add last element, and fails
+ # without trailing slash. The code really should be fixed,
+ # but not now.
+ .boostbook-xsl-dir = $(.boostbook-xsl-dir)/ ;
+ .boostbook-dtd-dir = $(.boostbook-dtd-dir)/ ; 
+ } 
}

rule xsl-dir
@@ -220,10 +225,7 @@
text += "</catalog>" ;

print.output $(actual) ;
- print.text $(text) : true ;
-
-# ALWAYS $(actual) ;
- TEMPORARY $(actual) ;
+ print.text $(text) : true ; 
}
}
}
 --Boundary-00=_Lp3H/7vqpHQjjY1--