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

Lawrence Velázquez larryv at macports.org
Wed Aug 10 23:20:56 PDT 2016


> 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.

vq


More information about the macports-dev mailing list