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

    wide-energy-11069

    03/30/2016, 9:46 PM
    i dont think so
  • w

    witty-crayon-22786

    03/30/2016, 9:46 PM
    @bored-art-40741: i don't expect he'll want to, but just in case...
  • w

    wide-energy-11069

    03/30/2016, 9:47 PM
    oh it might. let me try it
  • b

    bored-art-40741

    03/30/2016, 9:47 PM
    There's also the question of why 
    XDG_CACHE_HOME
    isn't being respected by the tool bootstrapping
  • w

    witty-crayon-22786

    03/30/2016, 9:50 PM
    @wide-energy-11069: either way, the idea behind the concurrent creation context manager is multi-fold
  • w

    witty-crayon-22786

    03/30/2016, 9:50 PM
    if the operation to create the jar fails halfway through, you'll have a half created jar in the final output location
  • w

    witty-crayon-22786

    03/30/2016, 9:50 PM
    not a huge deal in this case, but
  • w

    wide-energy-11069

    03/30/2016, 9:52 PM
    i see
  • f

    fast-megabyte-33579

    03/30/2016, 10:28 PM
    did anyone have any thoughts about my including-thrift-files-in-published-jars question earlier (about 12 hours ago)?
  • w

    witty-crayon-22786

    03/30/2016, 10:31 PM
    @fast-megabyte-33579: iirc, it does include the thrift
  • w

    witty-crayon-22786

    03/30/2016, 10:32 PM
    generically putting it in an 
    idl
    classified jar would also be fine.
  • f

    fast-megabyte-33579

    03/30/2016, 10:32 PM
    using the publish extras thingummy?
  • w

    witty-crayon-22786

    03/30/2016, 10:34 PM
    i think either of those would be fine as built-in
  • w

    witty-crayon-22786

    03/30/2016, 11:25 PM
    ...blargh.
  • w

    witty-crayon-22786

    03/30/2016, 11:25 PM
    PSA: 
    memoized_method
    does not do the right thing for generators
  • m

    mammoth-pharmacist-54215

    03/30/2016, 11:26 PM
    what does it do incorrectly exactly?
  • w

    witty-crayon-22786

    03/30/2016, 11:26 PM
    memoizes exactly the generator object
  • m

    mammoth-pharmacist-54215

    03/30/2016, 11:26 PM
    oh that is not ideal
  • w

    witty-crayon-22786

    03/30/2016, 11:27 PM
    yea. i guess it would change the type in some cases. maybe some other explicit decorator... 
    memoized_generator
    or something
  • m

    mammoth-pharmacist-54215

    03/30/2016, 11:29 PM
    yeah, I would imagine you could store the original generator and tee off of it when you hit the cache
  • m

    mammoth-pharmacist-54215

    03/30/2016, 11:31 PM
    quick look at the code it looks like toolz has the same problem… its an interesting use case. http://toolz.readthedocs.org/en/latest/_modules/toolz/functoolz.html
  • m

    mammoth-pharmacist-54215

    03/30/2016, 11:32 PM
    in theory you could update memoize to detect a generator and treat it as a special case.
  • w

    witty-crayon-22786

    03/30/2016, 11:34 PM
    yea, should
  • f

    fast-megabyte-33579

    03/31/2016, 2:46 PM
    wow I wish I hadn't turned on 
    verify_config
    - so many things are wrong in my pants.ini
  • w

    witty-crayon-22786

    03/31/2016, 5:05 PM
    hah
  • b

    bored-art-40741

    03/31/2016, 5:22 PM
    @witty-crayon-22786: Having heard no response from Gabriel... thoughts on removing this contrib package for now? Can always revert and fix later
  • w

    witty-crayon-22786

    03/31/2016, 5:22 PM
    yea. seems the way to go.
  • b

    bored-art-40741

    03/31/2016, 5:24 PM
    Going to put together a patch and see if it fixes Jenkins
  • b

    bored-art-40741

    03/31/2016, 6:28 PM
    So, I'd like to raise this lock question again, @witty-crayon-22786 @ancient-stone-50795
  • b

    bored-art-40741

    03/31/2016, 6:28 PM
    I put some work into getting a proper file lock working that also printed out a nice message, and I only found one viable way to do it
1...111111121113...2420Latest