pantsbuild #general

Channels

welcome

github-notifications

announce

witty-crayon-22786

01/26/2016, 8:46 PM

but yea. probably easier once nick splits resolution from fetching.

bored-art-40741

01/26/2016, 8:46 PM

Fully clean run (that writes artifacts) followed by fully cached run

bored-art-40741

01/26/2016, 8:46 PM

What I've already changed buys us everything we would otherwise get from putting it in the workspace

witty-crayon-22786

01/26/2016, 8:46 PM

so... i'll work on the tool caching thing then?

bored-art-40741

01/26/2016, 8:46 PM

The actual racey thing (resolution) is moved into the workspace with my change

bored-art-40741

01/26/2016, 8:47 PM

The artifact repository remains global, but ivy ostensibly locks properly around fetching / returning those artifacts

bored-art-40741

01/26/2016, 8:47 PM

Yeah, the tool caching needs to be done no matter what

witty-crayon-22786

01/26/2016, 8:47 PM

ah, that's excellent. having the reports live in the cache dir was always a bit crazy

witty-crayon-22786

01/26/2016, 8:47 PM

ok. will work on the tool caching thing.

bored-art-40741

01/26/2016, 8:48 PM

yeah I had no idea you could tell ivy to separate those things until John pointed it out in a github review yesterday

bored-art-40741

01/26/2016, 8:48 PM

I agree, crazy town

wide-energy-11069

01/26/2016, 9:00 PM

hi, can i get a quick shipit to unblock plugin ci? https://rbcommons.com/s/twitter/r/3379/

wide-energy-11069

01/26/2016, 9:00 PM

🚢

ancient-stone-50795

01/26/2016, 9:01 PM

@wide-energy-11069 done

wide-energy-11069

01/26/2016, 9:01 PM

thanks

wide-energy-11069

01/27/2016, 12:13 AM

can i get access to change the status of plugin issue? https://github.com/pantsbuild/intellij-pants-plugin/issues/

wide-energy-11069

01/27/2016, 12:13 AM

or can only committers do that?

bored-art-40741

01/27/2016, 12:15 AM

Um, who owns the plugin repo, really?

bored-art-40741

01/27/2016, 12:16 AM

At this point I'm surprised that you don't

witty-crayon-22786

01/27/2016, 12:16 AM

@bored-art-40741: committers

bored-art-40741

01/27/2016, 12:16 AM

Is committers an org-wide role?

witty-crayon-22786

01/27/2016, 12:16 AM

yea.

bored-art-40741

01/27/2016, 12:16 AM

And if so, maybe we need to reconsider that when there are people who don't fit the "ten commits to pants" pattern but are, as here, active committers to the plugin

witty-crayon-22786

01/27/2016, 12:22 AM

https://github.com/pantsbuild/pants/issues/1640#issuecomment-175310023

witty-crayon-22786

01/27/2016, 12:22 AM

@bored-art-40741: ^

bored-art-40741

01/27/2016, 12:23 AM

Couple notes

bored-art-40741

01/27/2016, 12:24 AM

(1) It's safe to locally (i.e. repo-local) cache ivy resolves

bored-art-40741

01/27/2016, 12:24 AM

We did this for a long time

bored-art-40741

01/27/2016, 12:24 AM

But you're right that it's not safe to remotely cache it

witty-crayon-22786

01/27/2016, 12:24 AM

sure. not super useful, but safe