[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