[chef] Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: COOK-3084 - wrong python on RHEL5

[Mike < >]

My vote is the patch-level change on this branch:
https://github.com/akiernan/python/tree/handle-rhel5.
Compare here: 
https://github.com/akiernan/python/compare/opscode-cookbooks:master...handle-rhel5

My question would be regarding the test here:
https://github.com/akiernan/python/compare/opscode-cookbooks:master...handle-rhel5#L7R29

Should/can this default to a node attribute, so if the attribute was
modified to "/usr/bin/python27" it could pass as well? Or do we
refrain from adding attributes to tests?

Anyways, this is awesome work Alex - very thorough!
-M


On Mon, Jun 10, 2013 at 5:15 PM, Alex Kiernan 
<
 >
 wrote:
>
> On 10 Jun 2013 21:54, "Bryan McLellan" 
> <
 >
>  wrote:
>>
>> On Sun, Jun 9, 2013 at 1:31 PM, Alex Kiernan 
>> <
 >
>> wrote:
>> > Just waiting on the test-suite run to complete before pushing new
>> > branches which include minitest assertions.
>>
>> Is this pushed?
>
> It is.
>
>> Are the branches all set?
>>
>
> Should be...
>
>> Do we need to vote or did logic win out here somewhere? Who still
>> needs some convincing?
>>
>
> If we can get consensus on which approach I'll feel the test run across that
> branch as it's roughly 90 minutes per test run per branch.