pantsbuild #general

Channels

welcome

github-notifications

announce

happy-kitchen-89482

10/30/2015, 11:56 PM

=you

happy-kitchen-89482

10/30/2015, 11:57 PM

I was looking at the wrong thing

rough-minister-58256

10/30/2015, 11:58 PM

https://travis-ci.org/pantsbuild/pants/jobs/88456785#L1670

rough-minister-58256

10/30/2015, 11:59 PM

that seems to be the breakage

rough-minister-58256

10/30/2015, 11:59 PM

wonder if its just transient

rough-minister-58256

10/31/2015, 12:00 AM

don’t see any direct causes. any objections to restarting the job?

mammoth-pharmacist-54215

10/31/2015, 12:04 AM

@rough-minister-58256: probably okay but if its transient it would be a good idea if @bright-dinner-81701 took a look

rough-minister-58256

10/31/2015, 12:05 AM

yeah, saved a copy of the error for him - think he might be out for the day

rough-minister-58256

10/31/2015, 12:09 AM

seems to be doing better

happy-kitchen-89482

10/31/2015, 12:09 AM

Yeah, might be transient

happy-kitchen-89482

10/31/2015, 12:09 AM

network thing

rough-minister-58256

10/31/2015, 12:09 AM

will file a JIRA against cpesto to followup just in case

rough-minister-58256

10/31/2015, 12:20 AM

looks good now.

mysterious-action-44812

10/31/2015, 4:39 PM

is there anything other than https://gist.github.com/nitay/706a75faa8e6129ef87b I need to put in order to get a thrift library built? right now with that it doesn’t seem to do anything - no error nothing, just continues on but doesn’t actually compile any thrift

mysterious-action-44812

10/31/2015, 4:40 PM

gistfile1.txt

mysterious-action-44812

10/31/2015, 4:41 PM

(downstream targets fail on compile errors with symbols missing)

many-engine-77019

11/01/2015, 12:40 AM

it seems https://rbcommons.com/s/twitter/r/2778/ invalidated the instructions for working on pant’s jvm tools (http://pantsbuild.github.io/howto_develop.html#developing-and-debugging-a-jvm-tool and zinc/README.md)

many-engine-77019

11/01/2015, 12:53 AM

oh that’s not too bad

witty-crayon-22786

11/02/2015, 4:15 PM

@enough-analyst-54434: re: https://rbcommons.com/s/twitter/r/3058/ ... anything i'm missing?

enough-analyst-54434

11/02/2015, 4:47 PM

Not that I can tell, I spelunked and could not figure out why I ever wrote that test comment

witty-crayon-22786

11/02/2015, 4:47 PM

cool.

enough-analyst-54434

11/02/2015, 4:48 PM

@mysterious-action-44812: thrift configuration is still baroque - you'd need things like this: https://github.com/pantsbuild/pants/blob/master/pants.ini#L63

enough-analyst-54434

11/02/2015, 4:48 PM

The generated thrift code has its own runtime deps and I'm guessing your missing symbols come from those. The config I point to injects these deps automatically, and w/o this you could miss symbols

enough-analyst-54434

11/02/2015, 4:49 PM

You might get better service by filing a bug with the actual full build logs + the info you have given

mysterious-action-44812

11/02/2015, 5:04 PM

thx @enough-analyst-54434 I actually found the issue - it was missing the scrooge plugin line - however just adding it to backehnd_packages didn’t work, i ended up copying the entire thing into my source tree and adding a local pants-plugins thing

mysterious-action-44812

11/02/2015, 5:04 PM

that seems to do it

enough-analyst-54434

11/02/2015, 5:06 PM

Aha - that is definitely not ideal. How do you run pants now? If not via a hand-built pex, this is recommended: http://pantsbuild.github.io/install.html#virtualenv-based-installation

mysterious-action-44812

11/02/2015, 5:08 PM

ah yeah i use the virtualenv based

enough-analyst-54434

11/02/2015, 5:08 PM

OK - I broke master CI, but it appears this issue is known?:

Copy code

FAILURE: Failed to resolve dependencies for contrib/node/examples/src/node/web-build-tool:
	/home/travis/.cache/pants/bin/node/linux/x86_64/v4.0.0/unpacked/node/bin/npm install failed with exit code 1

mysterious-action-44812

11/02/2015, 5:08 PM

will try pex