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

Ryan Schmidt ryandesign at macports.org
Wed Aug 10 23:22:36 PDT 2016


> On Aug 11, 2016, at 1:20 AM, Lawrence Velázquez <larryv at macports.org> wrote:
> 
>> On Aug 11, 2016, at 2:07 AM, Ryan Schmidt <ryandesign at macports.org> wrote:
>> 
>> 
>>> On Aug 10, 2016, at 11:56 PM, Mojca Miklavec <mojca at macports.org> 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.

See, that would be my fault for adding that os variable. I don't know why you people let me anywhere near Python code when I don't know how to Python. :)



More information about the macports-dev mailing list