<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 25, 2019 at 8:44 PM Zachary Turner via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">The idea behind seven is that it’s a place to put stuff that we need for py2/py3 interoperability that doesn’t already exist in six.  Yes, maybe there’s only one thing there now, but there could be more over time.<br><br>At least that was the thinking when I created it. </blockquote><div><br></div><div>When I was looking at this with Davide yesterday, we were under the impression that it was an external package (<a href="https://pypi.org/project/seven">https://pypi.org/project/seven</a>) or part thereof. Compatibility with Python 2.5 didn't sound particularly important which is why I wanted to remove it. Anyway, now that I know its purpose, I'm okay with fixing the newline issue there. </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">It seems like there’s two separate issues here: 1) you need to fix a bug, and 2) you want to propose removing seven.  Is it worth doing those two things separately?<br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 25, 2019 at 6:00 PM Jonas Devlieghere via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">JDevlieghere created this revision.<br>
JDevlieghere added reviewers: davide, zturner, stella.stamenova, labath.<br>
JDevlieghere added a project: LLDB.<br>
Herald added a reviewer: serge-sans-paille.<br>
<br>
When running the test suite on macOS with Python 3 we noticed a difference in behavior between Python 2 and Python 3 for `seven.get_command_output`. The output contained a newline with Python 3, but not for Python 2. This resulted in an invalid SDK path passed to the compiler.<br>
<br>
There were only two actual usages left of this module so I propose to remove it and have a simple, local implementation for `get_command_output`.<br>
<br>
<br>
Repository:<br>
  rLLDB LLDB<br>
<br>
<a href="https://reviews.llvm.org/D57275" rel="noreferrer" target="_blank">https://reviews.llvm.org/D57275</a><br>
<br>
Files:<br>
  packages/Python/lldbsuite/support/seven.py<br>
  packages/Python/lldbsuite/test/dosep.py<br>
  packages/Python/lldbsuite/test/dotest.py<br>
  packages/Python/lldbsuite/test/functionalities/exec/TestExec.py<br>
  packages/Python/lldbsuite/test/functionalities/fat_archives/TestFatArchives.py<br>
  packages/Python/lldbsuite/test/lang/objc/ivar-IMP/TestObjCiVarIMP.py<br>
<br>
</blockquote></div>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
</blockquote></div></div></div>