pantsbuild #general

Re travis weirdness, always remember that for some reason github or travis (no idea which) rebases your changes onto current master before running your CI. So you can have other people’s changes in your CI that you don’t have in your branch. Many times have I bashed my head against the wall on that one.

mammoth-pharmacist-54215

01/27/2016, 8:02 PM

@happy-kitchen-89482: yeah, thankfully @witty-crayon-22786 was able to help before I did to much head bashing.

better-park-26723

01/27/2016, 8:19 PM

@witty-crayon-22786: I added

plugins

to PYTHONPATH in the

pants

script (which is what it says to do in https://pantsbuild.github.io/howto_plugin.html )

better-park-26723

01/27/2016, 8:20 PM

>>Update your
pants
wrapper script to include the
plugins/
directory on
PYTHONPATH:

Copy code

PYTHONPATH=plugins:${PYTHONPATH}
export PYTHONPATH

better-park-26723

01/27/2016, 8:20 PM

How do you do it in pants.ini?

witty-crayon-22786

01/27/2016, 8:21 PM

Copy code

[DEFAULT]
# ..
pythonpath: [
    "%(buildroot)s/plugins",
]

better-park-26723

01/27/2016, 8:22 PM

One thing I noticed that's really weird is that this works:

Copy code

[DEFAULT]
pants_version: 0.0.67

backend_packages: [
    "hello_world",
  ]

But this doesn't parse:```[DEFAULT] pants_version: 0.0.67 backend_packages: [ "hello_world", ]```

better-park-26723

01/27/2016, 8:23 PM

Copy code

Exception message: File contains parsing errors: /private/tmp/tp/hello_task/pants.ini
	[line  6]: ']'

witty-crayon-22786

01/27/2016, 8:26 PM

mm... yea, quirks of the python ini lib.

bored-art-40741

01/27/2016, 8:36 PM

Have to have a tab/spaces before the trailing bracket

limited-country-20626

01/27/2016, 8:36 PM

We have a loose source pants plugin on github if you ever need an example @better-park-26723. https://github.com/foursquare/fsqio.

bored-art-40741

01/27/2016, 9:24 PM

@rough-minister-58256: If we're talking about deal breakers in CR tools though, not being able to assign non-committers as a reviewer is a massive one

rough-minister-58256

01/27/2016, 9:24 PM

agree, its awkward

happy-kitchen-89482

01/27/2016, 9:51 PM

And not being able to designate one person as the reviewer and cc others is also unfortunate

ancient-stone-50795

01/27/2016, 9:51 PM

Hi twitterfolk, I have a question about publishing jars. I have several 'java_library' targets that have no sources. publishing hits them and first complains they have no

provides=artifact(...)

but if I provide that, it fails because there is no source. Should I just change these to

target()

or is there some other way to get publish to ignore a java_library() ?

bored-art-40741

01/27/2016, 9:54 PM

If it has no source it seems like it must be an aggregator

bored-art-40741

01/27/2016, 9:55 PM

And therefore makes more sense as target()

ancient-stone-50795

01/27/2016, 10:10 PM

That's what I did, so far so good. Thanks.

ancient-stone-50795

01/27/2016, 10:11 PM

How about the prompting that happens when you publish a jar. Is that coming from maven or ivy?

Copy code

Publish com.squareup#common-annotation-processors with revision Semver(0.0.1-SNAPSHOT) ? [y|N]

I'm trying to figure out how to just turn it always on.

ancient-stone-50795

01/27/2016, 10:11 PM

(for running from scripts)

witty-crayon-22786

01/27/2016, 10:13 PM

yes | ./pants publish

ancient-stone-50795

01/27/2016, 10:13 PM

ok, that works.

witty-crayon-22786

01/27/2016, 10:13 PM

(although john always balked when i suggested it)

ancient-stone-50795

01/27/2016, 10:14 PM

ancient-stone-50795

01/27/2016, 10:14 PM

ancient-stone-50795

01/27/2016, 10:14 PM

ancient-stone-50795

01/27/2016, 10:14 PM