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

    witty-crayon-22786

    11/18/2016, 7:02 PM
    but also, would minimize behaviour changes in favor of landing something
  • f

    fancy-queen-20734

    11/18/2016, 7:03 PM
    fair
  • f

    fancy-queen-20734

    11/18/2016, 7:20 PM
    Got the temp file change and per root traces, now checking on further tests to unannotate.
  • f

    fancy-queen-20734

    11/18/2016, 7:21 PM
    I’d like to move some state checking into rust, but I think having a trace per root is better than no traces
  • w

    witty-crayon-22786

    11/18/2016, 7:21 PM
    heh. sure
  • r

    rough-minister-58256

    11/18/2016, 11:05 PM
    @fancy-queen-20734 what’s up with the 
    =
    prefix on the types in the trace?
  • r

    rough-minister-58256

    11/18/2016, 11:05 PM
    e.g. 
    Computing =Collection.of(HydratedTarget) for SiblingAddresses(directory=u’')
  • r

    rough-minister-58256

    11/18/2016, 11:05 PM
    =Collection.of
  • r

    rough-minister-58256

    11/18/2016, 11:05 PM
    is that intentional?
  • f

    fancy-queen-20734

    11/18/2016, 11:05 PM
    @rough-minister-58256 it’s because they are TypeConstraints
  • r

    rough-minister-58256

    11/18/2016, 11:05 PM
    ic
  • r

    rough-minister-58256

    11/18/2016, 11:05 PM
    wonder if that notation would be confusing to end users
  • w

    witty-crayon-22786

    11/18/2016, 11:07 PM
    yea. i'd still like to switch it back to just a straight union
  • r

    rough-minister-58256

    11/18/2016, 11:19 PM
    looks like we’re still a bit high on memory utilization for list:: in source even with the native engine:
  • r

    rough-minister-58256

    11/18/2016, 11:19 PM
    @User uploaded a file:

    Pasted image at 2016-11-18, 3:19 PM▾

  • w

    witty-crayon-22786

    11/18/2016, 11:22 PM
    heh
  • r

    rough-minister-58256

    11/18/2016, 11:22 PM
    @User uploaded a file:

    Pasted image at 2016-11-18, 3:22 PM▾

  • r

    rough-minister-58256

    11/18/2016, 11:22 PM
    and climbing
  • r

    rough-minister-58256

    11/18/2016, 11:32 PM
    peaked @ 7.10GB
  • r

    rough-minister-58256

    11/18/2016, 11:43 PM
    slower than expected as well 😞
  • r

    rough-minister-58256

    11/19/2016, 12:04 AM
    how would folks feel about adopting https://github.com/rust-lang-nursery/rustfmt for the native code? seems like it could help kill lots of style related things right off the bat.
  • w

    witty-crayon-22786

    11/19/2016, 12:20 AM
    @rough-minister-58256 : i think that it might be a debug build...?
  • w

    witty-crayon-22786

    11/19/2016, 12:20 AM
    don't know about that though.
  • r

    rough-minister-58256

    11/19/2016, 12:20 AM
    possible?
  • r

    rough-minister-58256

    11/19/2016, 12:21 AM
    how would I tell
  • w

    witty-crayon-22786

    11/19/2016, 12:21 AM
    i should have checked before saying that though. don't want to get any hopes up
  • r

    rough-minister-58256

    11/19/2016, 12:22 AM
    it did build the bin locally using john’s thing - I had to run pants —enable-v2-engine in pants’ repo first to get that in place before I could run with it in source (or else it looked for a binutil path)
  • w

    witty-crayon-22786

    11/19/2016, 12:22 AM
    have to look at what john did with the release process
  • w

    witty-crayon-22786

    11/19/2016, 12:22 AM
    aha. yea, that's probably a debug build then.
  • r

    rough-minister-58256

    11/19/2016, 12:22 AM
    k
1...194195196...906Latest