update Pants+IntelliJ docs to say the Plugin's the new hotness, link to plugin's README

Reviewed at https://rbcommons.com/s/twitter/r/891/
Fix a nasty bug when injecting target closures in BuildGraph

Just because a target has been injected doesn't mean that its
closure (and therefore its dependency edges) have been injected.
The current code misses the case where a target was injected
(but not its closure) and then has its closure injected later.

Testing Done:
Added a test that identifies the bug, which now passes (doesn't on master):
tests/python/pants_test/graph test_inject_then_inject_closure.

CI is green: https://travis-ci.org/pantsbuild/pants/builds/41270498

Reviewed at https://rbcommons.com/s/twitter/r/1337/
Fix java_sources missing dep detection.

This seems to just be an oversight in the original implementation.
We noticed incorrect missing dependency errors (we have it set to
error rather than warn) when we started using java_sources in our codegen.
This resolved our issue internally.

The general idea is that a ScalaLibrary should be claiming its
java_sources sources for the purpose of mapping generated classfiles
back to the sources zinc used to generate them.

Testing Done:
CI is green: https://travis-ci.org/pantsbuild/pants/builds/41043159

Reviewed at https://rbcommons.com/s/twitter/r/1336/
Kill all nailgun tasks after an integration test run to keep from leaking NGServer instances.

Before this change, Travis-CI builds are failing with: Cannot create GC thread. Out of system resources.
For example:  https://travis-ci.org/pantsbuild/pants/jobs/40841557

Testing Done:
CI running at: https://travis-ci.org/pantsbuild/pants/builds/41295684

Manual test: while running devpants goal test ./tests/python/pants_test:integration

before:
ps auxww | grep NGServer | wc -l
143

after:
ps auxww | grep NGServer | wc -l
13

Bugs closed: 799

Reviewed at https://rbcommons.com/s/twitter/r/1345/
Remove legacy option dests from all non-jvm-y tasks.

Testing Done:
CI passed.

Reviewed at https://rbcommons.com/s/twitter/r/1341/
Remove legacy dests from jvm task options.

Testing Done:
CI passed.

Reviewed at https://rbcommons.com/s/twitter/r/1343/
Remove support for legacy flag dests in new options.

We no longer use it anywhere.

Testing Done:
CI passes.

Reviewed at https://rbcommons.com/s/twitter/r/1351/
doc site cross references: call them pantsmark, pantsref instead of xmark, xref

<a xmark="foo"> looks like it could be some obscure HTML thing. Switch to <a pantsmark="foo"> which is more obviously a pants thing.

Testing Done:
https://travis-ci.org/pantsbuild/pants/builds/41306457

Reviewed at https://rbcommons.com/s/twitter/r/1348/
Use awk instead of sed to filter ever Nth line out of the integration test output The sed -n "K~N" syntax is not compatible with sed on MacOS.

Testing Done:
compared the list of integration tests listed when modifying the -i N:M parameters to the script.
Here is a good CI run https://travis-ci.org/pantsbuild/pants/builds/41359083

Bugs closed: 800

Reviewed at https://rbcommons.com/s/twitter/r/1350/
fix string formatting so 'test -h' does not crash

Testing Done:
PANTS_DEV=1 ./pants goal test -h

doesn't crash, unlike

./pants goal test -h

https://travis-ci.org/pantsbuild/pants/builds/41402745

Bugs closed: 805

Reviewed at https://rbcommons.com/s/twitter/r/1353/
Add dummy readme in examples/src/../scala/hello for Intellij plugin integration tests.

We added capability to the pants plugin to index non target owning files e.g. deploy scripts or READMEs
here https://rbcommons.com/s/twitter/r/1338/

We added the for the plugins.
Integration tests for Intllij plugin

Testing Done:
yes.
https://github.com/pantsbuild/pants/commit/d0fc5d05e8fae14311bab4cb7e664fa23fb02a8c

Reviewed at https://rbcommons.com/s/twitter/r/1356/
Support for the --kill-nailguns option was inadvertently removed, this puts it back.

The kill-nailguns flag was being read through old_options.cleanup_nailguns

Also updated the nailgun timeout logic to try and pinpoint transient errors in CI if it
happens again.

This reverts the change at https://rbcommons.com/s/twitter/r/1345/ which is redundant with this one (and not as effective.)

Testing Done:
CI Passed.

Bugs closed: 803

Reviewed at https://rbcommons.com/s/twitter/r/1352/
scala_specs: mention specs lib in the docstring; mention junit works, too

because not all Scala tests are scala_tests

Bugs closed: 622

Reviewed at https://rbcommons.com/s/twitter/r/1326/
Cache files created by (specially designed) annotation processors.

These annotation processors (so far, only twitter.common.args) write a file into META-INF which lists all of the files they created. Make pants process that file to decide which files should go in the artifact cache.

Testing Done:
Ran integration tests.

Bugs closed: 132

Reviewed at https://rbcommons.com/s/twitter/r/1250/
Revert "Cache files created by (specially designed) annotation processors."

This reverts commit e556eaa08f9c8e1be64a694d739283235d6989b0.

Apparently, this fails in CI for reasons I don't understand, so I'm
reverting it until I can figure it out.
doc: fix JVMProjects problems I didn't want to think about in rb/1176

Don't claim 'goal jar' is one of "The Usual Commands". It's kind of obscure and scary.

An extra "is" was in code font. Yeah, "is" is valid Python. But this wasn't Python, this was English.

Updated ivy mention: use new flags/config.

Tweaked nailgun description.

While I was in there: removed a section on getting help. It was redundant with
first_tutorial, except that it hadn't kept up to date :-(

Reviewed at https://rbcommons.com/s/twitter/r/1358/
doc: nailgun 5 seconds -> 10 seconds

Bugs closed: 804

Reviewed at https://rbcommons.com/s/twitter/r/1363/
Cache files created by (specially designed) annotation processors.

These annotation processors (so far, only twitter.common.args) write a file into META-INF which lists all of the files they created. Make pants process that file to decide which files should go in the artifact cache.

Testing Done:
Ran integration tests.

Bugs closed: 132

Reviewed at https://rbcommons.com/s/twitter/r/1250/
In IvyImports, make a single call to ivy to map all jar files.

If you have many targets with jars to map in, this is a significant performance increase.

Testing Done:
Added a new unit test.
Ran this in our internal repo where we have a lot of uses of java_protobuf_library() using imports=.

CI running at:  https://travis-ci.org/pantsbuild/pants/builds/41531811

Bugs closed: 286, 809

Reviewed at https://rbcommons.com/s/twitter/r/1364/
Revert "In IvyImports, make a single call to ivy to map all jar files."

This reverts commit 1aaf3b676187cf362ba8b7fed461db3538cd5c00.