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

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Mon Nov 2 22:35:29 PST 2015


:-)

On Mon, Nov 2, 2015 at 10:30 PM, Zachary Turner <zturner at google.com> wrote:

>
>
> 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 :-/
>
>>


-- 
-Todd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151102/5e6ad190/attachment.html>


More information about the lldb-commits mailing list