pantsbuild / pants / 7614
0%
main: 93%

LAST BUILD BRANCH: cherry-pick-23155-to-2.31.x
DEFAULT BRANCH: main
Ran
Jobs 8
Files 0
Run time –
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
7614

push

travis-ci

stuhood 
Ensure synthetic jar is always created (Argument list too long revisited)

https://rbcommons.com/s/twitter/r/2672/ introduced a synthetic that includes original classpath in its manifest, as a workaround for platform limitation: argument too long.

It however wasn't properly triggered because it expects OSError which was wrapped and rethrown by Executor as Executor.Error.

This review makes the following changes

* Use this all time not just on failures, reasoning is prefer one code path rather than two. As you can see, unit test is a lot easier.
* One issue with creating synthetic jar all time is user code may invoke different classLoader that interpret embedded Class-Path differently, because of this, we disabled synthetic jar for scala repl and nailgun (no benefit anyway)
* 2nd issue is previously creating synthetic jar is rare so it was left in a temp directory uncleaned. Now we either clean it up or if caller supplies a directory will save to that directory users can rerun the cmdline if they want to.
* In manifest change from absolute classpath to relative path, to follow the spec, java classLoader is ok with absolute path, but as we discovered some user code doesn't like this [one](https://github.com/twitter/util/blob/develop/util-eval/src/main/scala/com/twitter/util/Eval.scala#L360)

Testing Done:
https://travis-ci.org/peiyuwang/pants/builds/86456089

Also tetsed in CI/linux on twitter repo, a due diligence we missed last time.

Bugs closed: 2407

Reviewed at https://rbcommons.com/s/twitter/r/3003/
Jobs
ID Job ID Ran Files Coverage
1 7614.1 (CI_FLAGS="-cjlpn 'Various pants self checks'") 0
Travis Job 7614.1
4 7614.4 (CI_FLAGS="-fkmsrcjlp 'Python contrib tests'") 0
Travis Job 7614.4
5 7614.5 (CI_FLAGS="-fkmsrjlpn -i 0/6 'Python integration tests for pants - shard 1'") 0
Travis Job 7614.5
6 7614.6 (CI_FLAGS="-fkmsrjlpn -i 1/6 'Python integration tests for pants - shard 2'") 0
Travis Job 7614.6
7 7614.7 (CI_FLAGS="-fkmsrjlpn -i 2/6 'Python integration tests for pants - shard 3'") 0
Travis Job 7614.7
8 7614.8 (CI_FLAGS="-fkmsrjlpn -i 3/6 'Python integration tests for pants - shard 4'") 0
Travis Job 7614.8
9 7614.9 (CI_FLAGS="-fkmsrjlpn -i 4/6 'Python integration tests for pants - shard 5'") 0
Travis Job 7614.9
10 7614.10 (CI_FLAGS="-fkmsrjlpn -i 5/6 'Python integration tests for pants - shard 6'") 0
Travis Job 7614.10
Source Files on build 7614
Detailed source file information is not available for this build.