#general
- r
rough-minister-58256
08/28/2015, 8:41 PMthat we now have to accept - w
witty-crayon-22786
08/28/2015, 8:41 PMso being more restrictive on deps in core seems beneficial even - e
enough-analyst-54434
08/28/2015, 8:42 PMIf the daemon holding the graph in mem moot the speedup - e
enough-analyst-54434
08/28/2015, 8:42 PMit moots the speedup - w
witty-crayon-22786
08/28/2015, 8:42 PMnope - w
witty-crayon-22786
08/28/2015, 8:42 PM...unless we move products into the graph soonish - e
enough-analyst-54434
08/28/2015, 8:43 PMthats more the point - w
witty-crayon-22786
08/28/2015, 8:43 PMi know we should - e
enough-analyst-54434
08/28/2015, 8:43 PMif the optimizations of the next year moot the pypy - e
enough-analyst-54434
08/28/2015, 8:43 PMthey moot the pypy - w
witty-crayon-22786
08/28/2015, 8:43 PMperfect enemy of the good - e
enough-analyst-54434
08/28/2015, 8:43 PMNO! - e
enough-analyst-54434
08/28/2015, 8:43 PMI';m just saying we don;'t support pypy unitl we have CI - w
witty-crayon-22786
08/28/2015, 8:43 PMah, k - w
witty-crayon-22786
08/28/2015, 8:43 PMheh - e
enough-analyst-54434
08/28/2015, 8:43 PMonce you have CI you can add any dep you want - e
enough-analyst-54434
08/28/2015, 8:43 PMand we know if it fails under pypy - w
witty-crayon-22786
08/28/2015, 8:44 PMgotcha. yes. - r
rough-minister-58256
08/28/2015, 8:44 PMall I’m saying is this: pypy is not cpython and has its own set of baggage that comes with formal adoption. we should understand what those tradeoffs are if we decide to restrict pants to a pypy-compatible future and furthermore determine if they can be achieved without it such that we could otherwise continue down a vanilla cpython support track. - r
rough-minister-58256
08/28/2015, 8:45 PMbut if the data is there to support formal support for pypy, by all means - e
enough-analyst-54434
08/28/2015, 8:45 PMI'm guessing a CI and dual support will be enough to bridge whatever gaps and avoid making decisions like that - e
enough-analyst-54434
08/28/2015, 8:45 PMwe officially support CPython 2.7, we also are green under pypy - e
enough-analyst-54434
08/28/2015, 8:46 PMso go ahead and use it if you're on a project where it matters - r
rough-minister-58256
08/28/2015, 8:47 PMthen the next time you add a feature to pants that passes cpython CI but doesnt with pypy - the onus is on you as the developer to make it work with both? - e
enough-analyst-54434
08/28/2015, 8:47 PMor start the hard conversation - yes - e
enough-analyst-54434
08/28/2015, 8:47 PMbut defer the hard thing - e
enough-analyst-54434
08/28/2015, 8:47 PMit may not happen - e
enough-analyst-54434
08/28/2015, 8:47 PMit likely wont happen - e
enough-analyst-54434
08/28/2015, 8:47 PMand we'lllikely be able to ditch pypy in favor of our fundamental perf improvements - e
enough-analyst-54434
08/28/2015, 8:47 PMlater