[Pacemaker] patch for rsc_order loops.

[Frank DiMeo]

And I see it's in the repo already :)

-Frank

> -----Original Message-----
> From: Andrew Beekhof [mailto:andrew at beekhof.net]
> Sent: Thursday, December 10, 2009 6:30 AM
> To: Frank DiMeo
> Cc: pacemaker at oss.clusterlabs.org
> Subject: Re: patch for rsc_order loops.
> 
> I found a simpler fix :-)
> 
> diff -r 4582140c2f1b pengine/constraints.c
> --- a/pengine/constraints.c	Thu Dec 10 10:54:03 2009 +0100
> +++ b/pengine/constraints.c	Thu Dec 10 12:29:05 2009 +0100
> @@ -739,6 +739,7 @@ unpack_order_set(xmlNode *set, int score
>  	goto done;
>      }
> 
> +    last = NULL;
>      local_score *= -1;
>      action = invert_action(action);
> 
> 
> 
> On Thu, Dec 10, 2009 at 12:19 PM, Andrew Beekhof <andrew at beekhof.net>
> wrote:
> > On Tue, Dec 8, 2009 at 2:28 PM, Frank DiMeo
> <Frank.DiMeo at bigbandnet.com> wrote:
> >> I'm not sure what criteria you usually use for your test cases.
> >
> > The bz2 input files.
> > I'll go extract them from the other mail.
> >
> >> I ran my 2 and 4 resource case through this code and they both
> worked, and I generated the flow diagrams using ptest and it shows no
> loops.
> >>
> >> -Frank
> >>
> >>> -----Original Message-----
> >>> From: Andrew Beekhof [mailto:andrew at beekhof.net]
> >>> Sent: Tuesday, December 08, 2009 3:10 AM
> >>> To: Frank DiMeo
> >>> Cc: pacemaker at oss.clusterlabs.org
> >>> Subject: Re: patch for rsc_order loops.
> >>>
> >>> On Mon, Dec 7, 2009 at 8:19 PM, Frank DiMeo
> >>> <Frank.DiMeo at bigbandnet.com> wrote:
> >>> > I've enclosed at mercurial patch for what I believe is a fix for
> the
> >>> > resource ordering loop creation that I've been chasing.  Please
> look
> >>> it over
> >>> > and if you approve, I can submit it.
> >>>
> >>> Do you have a test case for this?
> >>
> >