On Mon, Nov 2, 2015 at 10:12 PM, Zachary Turner <ztur...@google.com> wrote:
> On Mon, Nov 2, 2015 at 10:02 PM Todd Fiala <todd.fi...@gmail.com> wrote: > >> Do you have a script you used to tweak these? >> >> (We have a ton of stuff broken internally on this that I need to fix up). >> >> We're starting to need to do quite a bit of extra typing for the package >> structure :-/ >> > > No, I just used a recursive regex replacement. I had to find a few > different patterns: > > from X import foo # from lldbsuite.test.X import foo > import X # import lldbsuite.test.X as X > import foo, X #import foo\nimport lldbsuite.test.X as X > > and I handled each one on a case-by-case basis. > > Okay, that sounds reasonable, thanks. > 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... > How much of a problem is this going to be on your side? Is there an > equally "good" solution that doesn't require so much fixup on your side? > > No big deal at all. I just need to fix bits that don't adhere. It took me more work to figure out what changed than it probably will to fix it. > the only fixup I encountered was in individual test cases and test suites > (i.e. files that are loaded by unittest2). Does this account for all the > test failures on your side as well, or is there something else going on I > missed? > Nah I think this is just it. Thanks! -- -Todd
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits