[Xorp-hackers] Installing tests

Ben Greear greearb at candelatech.com
Fri Dec 4 07:29:29 PST 2009


On 12/04/2009 04:48 AM, Bruce Simpson wrote:
> Ben Greear wrote:
>> I routinely build on one system and test on others. It would be nice
>> to have
>> an option to install the tests. Think you could add an scons argument
>> to allow
>> that?
>
> I can see that it would be a good feature to have in some situations,
> but I'm afraid I can't do this just at the moment.
>
> I will most likely fix the tests in libxipc so I can baseline the
> upcoming Thrift code, I can make no promises above and beyond that.

Those are the ones I care about too.  I had them working at one point,
but when merging I broke SConstruct, at least.

> The work I did on porting the regression tests to the SCons build, was
> just intended to be 'good enough' to get going. XORP's regression tests
> are actually pretty tied to the 'gmake check' way of doing things, and
> it's hardly a portable or particularly great test framework. Python's
> UnitTest beats the pants off it, and CxxTest is the closest I've seen in
> the C++ world.

I don't even care if the tests are automated, I was just hoping the binaries
would exist somewhere, probably in the normal xorp binary install directory.

> In the meantime, you could implement this by adding
> env.Alias('install-check', [ ... install actions ... ]) in the
> SConscripts for the tests. However, the RPATH is not guaranteed to be
> correct for a shared build. It might be an idea to add a separate
> PathVariable() in the top-level SConstruct to tell it where to put the
> tests.

Yeah, that is basically what I removed when dealing with merging in upstream.
I'll re-add something when I get a chance.

Thanks,
Ben


>
> Hope this helps.
>
> cheers,
> BMS


-- 
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc  http://www.candelatech.com



More information about the Xorp-hackers mailing list