#general
- e
enough-analyst-54434
08/28/2015, 8:31 PMWe should make it not so - w
witty-crayon-22786
08/28/2015, 8:32 PM@rough-minister-58256: fwiw, it "just works" for our repo - e
enough-analyst-54434
08/28/2015, 8:32 PMit should be an easy part of our dev pipeline to do this - a
ancient-stone-50795
08/28/2015, 8:32 PMThey are split up into directories and this is a real C*F*CK. if you can read between the lines. The tests are not under src/test/java, they are under src/main/java and they don’t run them with mvn test they pack them up into a binary. Pants is working fine, but IntelliJ is very confused to find code in a non-test folder depending on code in a test folder and I’m trying to find a good way to makt ehis sane. - e
enough-analyst-54434
08/28/2015, 8:32 PMaha - r
rough-minister-58256
08/28/2015, 8:32 PM@witty-crayon-22786: until it doesn't - w
witty-crayon-22786
08/28/2015, 8:33 PM@ancient-stone-50795: why mark them as tests then? - w
witty-crayon-22786
08/28/2015, 8:33 PMer... at the pants level - w
witty-crayon-22786
08/28/2015, 8:33 PMa java_library can depend on junit, no problem - a
ancient-stone-50795
08/28/2015, 8:34 PMTL;DR above: its really an IntelliJ issue. I’m trying to give IntelliJ a clue that these are tests by defining
targets.junit_tests() - w
witty-crayon-22786
08/28/2015, 8:35 PMin general, the solution to maven problems seems to be "create another project" - w
witty-crayon-22786
08/28/2015, 8:35 PM(which is a problem in itself, but.) - w
witty-crayon-22786
08/28/2015, 8:35 PMin this case it would prevent a cycle - a
ancient-stone-50795
08/28/2015, 8:35 PMThe solution we are working on is (╯°□°)╯︵ ┻━┻ mvn I’ve got it expunged from about 10% of CI currently. - w
witty-crayon-22786
08/28/2015, 8:36 PMnice - w
witty-crayon-22786
08/28/2015, 8:37 PM@rough-minister-58256: so you would really block this? - a
ancient-stone-50795
08/28/2015, 8:37 PMbeing able to run tests with the right version of the JRE is allowing us to move more projects over. - r
rough-minister-58256
08/28/2015, 8:38 PMnot block, just mildly concerned - r
rough-minister-58256
08/28/2015, 8:39 PMetc - r
rough-minister-58256
08/28/2015, 8:39 PM"PyPy has alpha/beta-level support for the CPython C API, however, this feature is not yet complete. We strongly advise use of CFFI instead. CFFI come builtin with PyPy. Many libraries will require a bit of effort to work, but there are known success stories. Check out PyPy blog for updates, as well as the Compatibility Wiki. C extensions need to be recompiled for PyPy in order to work. Depending on your build system, it might work out of the box or will be slightly harder." - r
rough-minister-58256
08/28/2015, 8:40 PMintroduces compatibility concerns external to cpython - r
rough-minister-58256
08/28/2015, 8:40 PMbut lets say the reason pypy is faster is because of say the slots optimization it applies to all objects, but that affects us for one certain set of object types related to build-file parsing - we could extract that and implement ourselves and potentially achieve likewise speedups - w
witty-crayon-22786
08/28/2015, 8:40 PM@rough-minister-58256: sure, but it already works - r
rough-minister-58256
08/28/2015, 8:41 PMk - e
enough-analyst-54434
08/28/2015, 8:41 PMCertainly pants users can switch if they want to - e
enough-analyst-54434
08/28/2015, 8:41 PMAnd until we have CI we don;t support it - w
witty-crayon-22786
08/28/2015, 8:41 PMso it's more a question of whether we're planning to depend on hundreds more things - w
witty-crayon-22786
08/28/2015, 8:41 PMpants itself shouldn't be adding hundreds more deps, right? - r
rough-minister-58256
08/28/2015, 8:41 PMI think its an ongoing tradeoff - e
enough-analyst-54434
08/28/2015, 8:41 PMwell, its also a question of short term gain