[151216] contrib/buildbot-test/master.cfg

[Mojca Miklavec]

On 11 August 2016 at 08:20, Lawrence Velázquez wrote:
>> On Aug 11, 2016, at 2:07 AM, Ryan Schmidt wrote:
>>> On Aug 10, 2016, at 11:56 PM, Mojca Miklavec wrote:
>>>
>>> I changed that in r151249 and 151258. I don't know any "official" way
>>> to construct PATH though and it wouldn't work on Windows anyway, so
>>> the whole point of changing this is a bit moot.
>>>
>>> Figuring out why
>>>   workdir=os.path.join(portwatcher_factory.workdir, "mpbb")
>>> didn't work and
>>>   workdir=portwatcher_factory.workdir+"/mpbb"
>>> was ok is left as an exercise to the reader.
>>
>> What happened when you tried to use it there?
>
> Mojca showed me this failure in the IRC channel:
>
>> exceptions.AttributeError: 'str' object has no attribute 'path'
>
> Which is unexpected to say the least. It's because "os" gets hijacked as a variable name, around line 440:
>
>     for plat in build_platforms:
>         os_match = extract_os.search(plat)
>         os = os_match.group(0) if os_match else plat
>
> This variable must be renamed.

Thank you, fixed.

Mojca