[Lldb-commits] [lldb] r251886 - Tighten up sys.path, and use absolute imports everywhere.

[Zachary Turner via lldb-commits]

On Mon, Nov 2, 2015 at 10:27 PM Todd Fiala <todd.fiala at gmail.com> wrote:

> On Mon, Nov 2, 2015 at 10:12 PM, Zachary Turner <zturner at google.com>
> wrote:
>
>>
>> Luckily the extra typing only comes at the import site.  If you use the
>> syntax I suggested in #2 and #3 above (import lldbsuite.test.X as X) then
>> the syntax at any sites where you use a definition from inside the module
>> are the same.
>>
>> I was a little bit annoyed at the extra typing myself, but when I look
>> into it, it's considered a "feature".  See PEP 0328 - Imports:
>> Multi-Line and Absolute/Relative
>> <https://www.python.org/dev/peps/pep-0328/> and as far as I can tell the
>> way we were doing things before was considered "wrong"  :(
>>
>>
> Heh.  Well, coming out of the gate with broken backward compatibility from
> 2.x to 3.x was considered "right" ( :-P ), so I'm not sure how much weight
> I'd put in the rulings from on high...
>
But.. but.. as long as you get to 3, we'll never make this mistake again,
we promise!  Anyway, welcome to my hell for the past 3 weeks :-/

>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151103/ac6a93e7/attachment.html>