https://pantsbuild.org/ logo
Powered by
#development
  • a

    aloof-angle-91616

    12/07/2018, 3:02 AM
    and then we don't even need to but we could just start thinking in terms of tuples like 
    (Console, ListOptions, Specs) -> IO
    and it all becomes utter nonsense especially because the rule graph isn't this absurd nonsense that a compiler engineer pretended was ok, no, it's a production-ready shared library. it's nonsense
  • w

    witty-crayon-22786

    12/07/2018, 3:03 AM
    oof. i can't even get into Effects right now.
  • a

    aloof-angle-91616

    12/07/2018, 3:03 AM
    (exactly)
  • w

    witty-crayon-22786

    12/07/2018, 3:03 AM
    but if this were starlark***... yea
    a
    • 2
    • 3
  • a

    aloof-angle-91616

    12/07/2018, 3:05 AM
    a type system that's defined by the rule graph and is intertwined with dependency injection is a type system nobody's seen before, or at least that's how i wanna think about it (meaning: i've never used haskell and i've always found it incredibly annoying)
  • a

    aloof-angle-91616

    12/07/2018, 3:07 AM
    thinking about either banging out some rust support with 
    python_dist()
    to build the native engine, or implementing BSP, or both. gotta pick something for tonight
  • w

    witty-crayon-22786

    12/07/2018, 3:07 AM
    mmmm
  • w

    witty-crayon-22786

    12/07/2018, 3:07 AM
    i'm going to log off. have a good night!
  • a

    aloof-angle-91616

    12/07/2018, 3:07 AM
    peace!
  • a

    aloof-angle-91616

    12/07/2018, 3:07 AM
    thanks a ton for the PRs today they're very very very very cool
  • w

    witty-crayon-22786

    12/07/2018, 3:08 AM
    your stuff has been awesome too. excellent week.
    👨‍🎓 2
  • w

    witty-crayon-22786

    12/07/2018, 3:08 AM
    ciao
  • a

    aloof-angle-91616

    12/07/2018, 3:16 AM
    the BSP route is something i'm gonna go do with shane i think because he expressed immense interest and winter is the best time to do anything like that
  • a

    aloof-angle-91616

    12/07/2018, 3:16 AM
    i hope everyone likes using rust because i'm about to support the heck out of it
  • a

    aloof-angle-91616

    12/07/2018, 3:16 AM
    might have to base off of #6880 because it's too good
  • a

    aloof-angle-91616

    12/07/2018, 3:26 AM
    idk if this was already known but 
    loc
    is a fantastic tool which is just 
    cloc
    but in rust and it indexed our monorepo in about the same time that ripgrep takes to search it (very fast), and we might want to consider using that instead of the 
    cloc
    script
  • h

    happy-kitchen-89482

    12/07/2018, 3:37 AM
    Apropos of nothing in particular, I notice that the order of 
    __init__()
    args is different than the declared order of ~equivalent 
    datatype
    fields here: https://github.com/benjyw/pants/blob/49146eff49d5116e58739e3ff201a50311f0e46b/src/python/pants/engine/rules.py#L150
    a
    w
    • 3
    • 4
  • h

    happy-kitchen-89482

    12/07/2018, 3:37 AM
    Is that by design? It’s mildly, although not terribly, confusing
  • h

    happy-kitchen-89482

    12/07/2018, 3:44 AM
    Well, https://github.com/pantsbuild/pants/pull/6880 is just very very cool indeed
    🎉 1
    ☝️ 1
  • a

    aloof-angle-91616

    12/07/2018, 9:32 AM
    merged this to unblock myself, or something: https://github.com/pantsbuild/pants/pull/6869
    🎉 1
    h
    • 2
    • 19
  • w

    witty-crayon-22786

    12/07/2018, 6:17 PM
    this part already exists in master, but: running 
    ./pants --no-v1 --v2 --loop --enable-pantsd list ::
    in pantsbuild pants, and then touching `src/python/pants/init/engine_initializer.py`:
    Copy code
    I1207 10:15:21.370506 93875 engine_initializer.py:210] running @console_rule('list')
I1207 10:15:22.700844 93875 engine_initializer.py:212] completed @console_rule('list')
I1207 10:15:26.532598 93875 scheduler_service.py:88] enqueuing 1 changes for subscription all_files
W1207 10:15:26.546345 93875 native.py:411] invalidation: cleared 1 and dirtied 6 nodes for: {"src/python/pants/init", "src/python/pants/init/engine_initializer.py"}
I1207 10:15:26.560921 93875 engine_initializer.py:210] running @console_rule('list')
I1207 10:15:26.615602 93875 engine_initializer.py:212] completed @console_rule('list')
    🔥 1
  • w

    witty-crayon-22786

    12/07/2018, 6:18 PM
    55ms
  • w

    witty-crayon-22786

    12/07/2018, 6:32 PM
    or 
    83ms
    between receiving the file event and finishing re-running
  • w

    witty-crayon-22786

    12/07/2018, 6:57 PM
  • w

    witty-crayon-22786

    12/07/2018, 9:30 PM
    and some more in the next comment
  • a

    aloof-angle-91616

    12/08/2018, 1:44 AM
    i'm gonna start rebasing off of https://github.com/pantsbuild/pants/pull/6880 for this attempt to replace most of the shell scripts in 
    build-support/bin/native/
    with rules, but don't let that stop you from pushing breaking changes
  • w

    witty-crayon-22786

    12/08/2018, 1:56 AM
    there is a change inbound in about 15 mins i hope
    👍 1
  • a

    aloof-angle-91616

    12/08/2018, 2:34 AM
    just had a great talk with greg from techdocs and if we need another excuse to rewrite tons of pants code then site generation might be it
  • w

    witty-crayon-22786

    12/08/2018, 2:35 AM
    ok, sorry. had to bisect something. just pushed
  • a

    aloof-angle-91616

    12/08/2018, 2:35 AM
    omg no worries thanks
1...597598599...906Latest