[Scons-dev] Likely bug - installing side effect files

[Ben Golding]

Thanks Gary.
http://scons.tigris.org/issues/show_bug.cgi?id=2982

I started to add some additional trace to Taskmaster.py, and compare the logs from sequential vs. parallel builds.

The bug seems to relate to how the node states are used for the side-effect nodes:
- In executed_with_callbacks(), the side-effect state is set to NODE_NO_STATE.
- In postprocess(), the logic to decrement the parent ref count is only active for a side-effect node with state NODE_EXECUTING.

Although this helped me to understand the bug a bit better, I'm not sure what is the correct fix.

Regards
Ben

-----Original Message-----
From: Scons-dev [mailto:scons-dev-bounces at scons.org] On Behalf Of Gary Oberbrunner
Sent: 30 October 2014 14:30
To: SCons developer list
Subject: Re: [Scons-dev] Likely bug - installing side effect files

On Thu, Oct 30, 2014 at 9:34 AM, Ben Golding <Ben.Golding at synopsys.com> wrote:
> tgt = Command('tgt', 'src', 'touch $TARGET sf0 sf1')
>   sf = SideEffect([ 'sf0', 'sf1' ], tgt)
>   Install('dir', tgt + sf)

I can reproduce this bug.  Please file a ticket!  The dependency tree
looks OK, and I can't easily see the cause of the bug.

Adding sf0 and sf1 to the targets list instead of as side effects makes it work.

-- 
Gary
_______________________________________________
Scons-dev mailing list
Scons-dev at scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev