> On Aug 11, 2016, at 1:20 AM, Lawrence Velázquez <[email protected]> wrote:
> 
>> On Aug 11, 2016, at 2:07 AM, Ryan Schmidt <[email protected]> wrote:
>> 
>> 
>>> On Aug 10, 2016, at 11:56 PM, Mojca Miklavec <[email protected]> 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. :)

_______________________________________________
macports-dev mailing list
[email protected]
https://lists.macosforge.org/mailman/listinfo/macports-dev

Reply via email to