[Scons-users] internal error: no cycle found

[Kenny, Jason L]

 I will make a note once I get to this. I would guess this function changed names as all node objects have a side_effect item.

Jason

-----Original Message-----
From: scons-users-bounces at scons.org [mailto:scons-users-bounces at scons.org] On Behalf Of Gary Oberbrunner
Sent: Thursday, June 14, 2012 11:46 AM
To: SCons users mailing list
Subject: Re: [Scons-users] internal error: no cycle found

On Thu, Jun 14, 2012 at 12:25 PM, TOM TANNER (BLOOMBERG/ LONDON) <ttanner2 at bloomberg.net> wrote:

>> I believe this is issue 2777,

>> http://scons.tigris.org/issues/show_bug.cgi?id=2777.  One of the devs 

>> is working on integrating the fix, it's expected to land soon.  

>> You're welcome to try the patch in the issue page and let us know if it fixes it for you.

>

...

> So I deleted one of the targets under question, and built just that. I got this:

>

> AttributeError: 'File' object has no attribute 'get_side_effects':


OK, looks like you applied the patch correctly. :-)  I see what you mean; nothing defines the get_side_effects method on a Node.  It looks to me like the patch is a bit damaged.  The first patch (id=859) adds the Node methods properly.  The second one (which is supposed to be an updated version of the first) is missing those changes.  I haven't reviewed in detail but it looks like there's a bit more work to do on that ticket.  I'll make a note there.  If you can figure it out and submit a better patch, please feel free!  (Jason, looks like you will have to try to figure this out too.)

--
Gary
_______________________________________________
Scons-users mailing list
Scons-users at scons.org
http://four.pairlist.net/mailman/listinfo/scons-users