> On 01/06/2010 11:20 AM, john dunning wrote:
> > On Wed, 2010-01-06 at 16:11 +0000, Gordon Sim wrote:
> >> On 01/06/2010 04:05 PM, Alan Conway wrote:
> >
> >>> Thats true, but I think it is still worth splitting out test sets 
> >>> for optional modules into separate files for the sake of 
> clarity and 
> >>> readability.
> >>
> >> Yes, thats a fair point.
> >
> > Thanks all.  For the immediate goal (getting the new features into 
> > qpid-config and adding tests for them) I'm going to go with 
> the least 
> > intrusive solution, which seems to be sensing the presence of the 
> > xml.so and driving the rest of the infrastructure accordingly.
> >
> > There likely does deserve to be some followon work to split up the 
> > test stuff in a more modular way.
> >
> 
> Shoud have mentioned this earlier but the cluster module is 
> an existing example 
> of this. The way it works is
>   - all cluster-specific build & test rules are in 
> src/tests/cluster.mk
>   - cluster.mk has the form "if HAVE_LIBCPG ... endif"
>   - cluster.mk is included in Makefile.am
> 
> The existing XML tests are handled similarly but they're not 
> broken out into a 
> separate .mk file as they're currently quite trivial. If you 
> add a lot of tests 
> then it might be worth breaking this out int xml.mk

The additions should also be in the cmake builds. We're supposed to
transition to cmake sometime post-0.6.

-Steve


---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscr...@qpid.apache.org

Reply via email to