pantsbuild #general

Channels

welcome

github-notifications

announce

enough-analyst-54434

07/22/2015, 2:39 AM

t/c/ has this iirc

witty-crayon-22786

07/22/2015, 2:39 AM

t/c/python?

witty-crayon-22786

07/22/2015, 2:40 AM

ps: we're getting ready to move pants_internal back into source... since there is only one copy of pants as of last night 😃

enough-analyst-54434

07/22/2015, 2:40 AM

excellent

enough-analyst-54434

07/22/2015, 2:44 AM

Yeah - t/c/python - but I remembered wrong, that's not in concurrent

enough-analyst-54434

07/22/2015, 2:44 AM

It may have been in Thermos at one point or even now

enough-analyst-54434

07/22/2015, 2:46 AM

So your own promise implies you're writing your own executor or else ... adding composability layers to make futures work like you expect?

enough-analyst-54434

07/22/2015, 2:47 AM

Using http://pythonhosted.org/futures/ in python2 is ~commonish - I'm not sure if using Future.set_result is though

witty-crayon-22786

07/22/2015, 2:48 AM

adding our own promise to make pantsbuild/pants' work pool a bit more sane

witty-crayon-22786

07/22/2015, 2:49 AM

would we accept a patch to move the work pool to http://pythonhosted.org/futures/ ?

enough-analyst-54434

07/22/2015, 2:50 AM

I can't see why not

enough-analyst-54434

07/22/2015, 2:52 AM

In your list you left out libstout/libprocess!

witty-crayon-22786

07/22/2015, 2:52 AM

eh?

witty-crayon-22786

07/22/2015, 2:52 AM

oh, of composable futures?

witty-crayon-22786

07/22/2015, 2:52 AM

yea.

enough-analyst-54434

07/22/2015, 2:52 AM

I was on reviews for Nick Kallen era Futures and the libstout

enough-analyst-54434

07/22/2015, 2:52 AM

stuff

witty-crayon-22786

07/22/2015, 2:53 AM

Nick Kallen era Futures are basically still what we use... in 33% of files

witty-crayon-22786

07/22/2015, 2:53 AM

except with cancellation and tco

enough-analyst-54434

07/22/2015, 2:55 AM

Ben gave a talk about libprocess at Twitter in 2012? that basically got a bunch of - huh? you can do that in c++? except for Wilhelm, who guffawed out loud at the huhs

witty-crayon-22786

07/22/2015, 2:55 AM

heh

early-mechanic-59271

07/22/2015, 3:40 PM

Shout out to whoever added the test-changed goal. That’s super rad and really make setting up the pull request builder easy.

early-mechanic-59271

07/22/2015, 3:40 PM

That was a pleasant surprise. I thought we were going to need a ci.sh-ish script.

enough-analyst-54434

07/22/2015, 3:48 PM

Foursquare goodies there

mammoth-autumn-2551

07/22/2015, 6:52 PM

<!here|@here> I’m seeing two issues that are occurring when going from pants 0.0.36 -> 0.0.37. I’m going to dig deeper, but if anything sounds familiar, please shout at me: 1) Some issue with loading resources in python — this should be pretty straightforward to make a simple test case for to verify 2) An issue when using requests, it is failing to validate ssl certificates

enough-analyst-54434

07/22/2015, 6:54 PM

The only candidate if these are upgrade issues is: https://rbcommons.com/s/twitter/r/2486/ - python chroots are cached now

enough-analyst-54434

07/22/2015, 6:55 PM

But we've not seen a similar issue - nor twitter/commons

mammoth-autumn-2551

07/22/2015, 6:58 PM

yea that seemed pretty inoccuous… but git bisect will show me the truth

enough-analyst-54434

07/22/2015, 7:00 PM

I'm in this world at the moment working https://github.com/pantsbuild/pants/issues/1831 again, so drop some details in a few issues if you bisect to the pants upgrade

mammoth-autumn-2551

07/22/2015, 7:01 PM