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

    witty-crayon-22786

    07/12/2018, 6:53 PM
    i'm curious whether we're missing something there though.
  • w

    witty-crayon-22786

    07/12/2018, 6:55 PM
    asking in the tokio gitter.
  • w

    witty-crayon-22786

    07/12/2018, 6:55 PM
    (fine publishing it though.)
  • w

    witty-crayon-22786

    07/12/2018, 8:56 PM
    thank you john.
  • a

    average-vr-56795

    07/12/2018, 10:14 PM
    Am I ok to merge #6106 or should I hold off until CI works again?
  • w

    witty-crayon-22786

    07/12/2018, 10:23 PM
    ...fine with me.
  • w

    witty-crayon-22786

    07/12/2018, 10:24 PM
    (i merged something which we had green CI for. if you have confidence that that one will not break anything further...)
  • a

    average-vr-56795

    07/13/2018, 7:45 AM
    @witty-crayon-22786 had a brief play with async/await last night. Definitely promising, but have a ways to go. I'm particular, async functions currently aren't allowed to take any references as params, including &self, which means we can't escape all of that bonus cloning (though we can probably do it closer to the relevant call sites), and I ended up having to pull out separate static functions everywhere
  • a

    average-vr-56795

    07/13/2018, 7:46 AM
    Async blocks are super not ready. Haven't yet gotten a non-trivial one working.
  • a

    average-vr-56795

    07/13/2018, 7:47 AM
    I had assumed stuff mostly worked and they were quibbling syntax, because that's how the RFC threads are looking, but it's definitely a lot less mature than I thought
    🤣 1
  • a

    average-vr-56795

    07/13/2018, 9:55 AM
    I published https://crates.io/crates/boxfuture by the way - I, and anyone who can receive email on pantsbuild@gmail.com, are the only owners, very happy to add more owners if anyone wants to be added
  • e

    enough-analyst-54434

    07/13/2018, 1:17 PM
    Alright, CI should be going green with https://github.com/pantsbuild/pants/pull/6104 I have some cleanup to do replacing XXX's with issue pointers over to pex. AFK for a few hours then back at it.
    🎉 1
  • a

    average-vr-56795

    07/13/2018, 2:53 PM
    @witty-crayon-22786 I’m trying to run pants in instruments, and it seems to be working, but symbols don’t have names unless they’re systemy things like 
    _pthread_*
    , 
    start
    , 
    kevent
    , 
    readdir
    , … - am I meant to symbolise somehow?
  • a

    average-vr-56795

    07/13/2018, 2:56 PM
    I have the dSYM, but the process instruments thinks it’s profiling is bash, and it won’t let me add the dSYM for native engine anywhere…
  • a

    average-vr-56795

    07/13/2018, 3:08 PM
    Tried running a pex, and then it thinks the root process is 
    env
    and refuses to add my dSYM
  • w

    witty-crayon-22786

    07/13/2018, 3:58 PM
    @average-vr-56795: sounds like you're not connected to the right process...?
  • w

    witty-crayon-22786

    07/13/2018, 3:58 PM
    you will see rust symbols, even in 
    --release
    , because we have them enabled
  • w

    witty-crayon-22786

    07/13/2018, 3:59 PM
    looking in 
    top
    for the python process with high cpu usage
  • w

    witty-crayon-22786

    07/13/2018, 3:59 PM
    ...helps
  • w

    witty-crayon-22786

    07/13/2018, 4:00 PM
    i don't do any manual dSYM-y steps... just what i mentioned there
  • w

    witty-crayon-22786

    07/13/2018, 4:00 PM
    ...oh. don't "run pants in instruments"
  • w

    witty-crayon-22786

    07/13/2018, 4:00 PM
    attach to a running process.
  • w

    witty-crayon-22786

    07/13/2018, 4:00 PM
    @average-vr-56795: ^
  • a

    average-vr-56795

    07/13/2018, 4:19 PM
    aha!
  • a

    average-vr-56795

    07/13/2018, 4:21 PM
    @witty-crayon-22786 Hurrah! I Have symbols! Thanks! 😄
  • w

    witty-crayon-22786

    07/13/2018, 4:22 PM
    👍
  • w

    witty-crayon-22786

    07/13/2018, 4:22 PM
    another strategy to catch "everything interesting" is to launch pantsd with 
    ./pants help
    or something, and then find and attach to that process while it's idling
  • a

    average-vr-56795

    07/13/2018, 4:23 PM
    I like it 🙂
  • a

    average-vr-56795

    07/13/2018, 4:23 PM
    Wow, panic backtracing adds a lot of noise 😞 Oh well, new tech stack for me to get an intuition for the feel of profiles in 🙂
  • w

    witty-crayon-22786

    07/13/2018, 4:23 PM
    @average-vr-56795: recommend doing the "top functions" thing, and then ignoring the top few
1...559560561...906Latest