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

    rough-minister-58256

    03/31/2016, 8:43 PM
    that error has been present in the OSX CI for a while now
  • r

    rough-minister-58256

    03/31/2016, 8:43 PM
  • r

    rough-minister-58256

    03/31/2016, 8:44 PM
    seems pretty consistent there
  • r

    rough-minister-58256

    03/31/2016, 8:45 PM
    I think Benjy’s is different tho? that one almost looks retryable
  • b

    bored-art-40741

    03/31/2016, 8:48 PM
  • w

    witty-crayon-22786

    03/31/2016, 8:54 PM
    @bored-art-40741: shipits with comments
  • b

    bored-art-40741

    03/31/2016, 9:02 PM
    thanks, saw them. Agree on both counts, will fix
  • w

    witty-crayon-22786

    03/31/2016, 9:12 PM
    so... what's the cheapest way to check whether sets intersect, without fully computing the intersection?
  • m

    many-engine-77019

    03/31/2016, 9:13 PM
    any + in?
  • w

    witty-crayon-22786

    03/31/2016, 9:13 PM
    yea, maybe... was hoping there might be a lazy version of 
    set1 & set2
    , which is presumably native
  • w

    witty-crayon-22786

    03/31/2016, 9:14 PM
    oh!
  • w

    witty-crayon-22786

    03/31/2016, 9:14 PM
    isdisjoint
  • w

    witty-crayon-22786

    03/31/2016, 9:14 PM
    nice.
  • w

    witty-crayon-22786

    03/31/2016, 9:14 PM
    Copy code
    isdisjoint(other)
Return True if the set has no elements in common with other. Sets are disjoint if and only if their intersection is the empty set.

New in version 2.6.
  • w

    wide-energy-11069

    03/31/2016, 10:46 PM
    had the wishful thinking that making 
    —verify-config
    default would be a one-line change
  • w

    witty-crayon-22786

    03/31/2016, 11:01 PM
    heh. does it break all the tests?
  • w

    witty-crayon-22786

    03/31/2016, 11:02 PM
    @wide-energy-11069: note that you can defer making it the default in 
    pantsbuild/pants
    by overriding it in config
  • w

    wide-energy-11069

    03/31/2016, 11:04 PM
    ah true that. i wonder if there’s ever need to change the source code then?
  • w

    wide-energy-11069

    03/31/2016, 11:04 PM
    actually the tests do not seem too bad to fix. question still stands ^
  • w

    witty-crayon-22786

    03/31/2016, 11:15 PM
    @wide-energy-11069: the value in 
    pantsbuild/pants
    pants.ini only affects that repo
  • w

    witty-crayon-22786

    03/31/2016, 11:15 PM
    not other users
  • w

    witty-crayon-22786

    03/31/2016, 11:15 PM
    so you have to change the default value in the code/`register_options`, to apply for users.
  • w

    wide-energy-11069

    03/31/2016, 11:18 PM
    make sense. i’ll proceed to add it as a global option
  • w

    witty-crayon-22786

    03/31/2016, 11:20 PM
    ...hm?
  • w

    wide-energy-11069

    03/31/2016, 11:33 PM
    @witty-crayon-22786: my impression (could be very wrong) is that it mostly concerns the new pants adopters, unless we want to have it end up in everyone’s repo by default?
  • w

    witty-crayon-22786

    04/01/2016, 1:05 AM
    @wide-energy-11069: we want it in everyone's repo by default
  • w

    witty-crayon-22786

    04/01/2016, 1:06 AM
    imo.
  • w

    wide-energy-11069

    04/01/2016, 1:06 AM
    sg
  • h

    happy-kitchen-89482

    04/01/2016, 1:29 AM
    FWIW retrying worked
  • l

    limited-country-20626

    04/01/2016, 4:58 AM
    =stu. I love this feature.
1...111511161117...2420Latest