On 21/11/2019 19:53, Vicente Romero wrote:
Hi,
I think I have covered all the proposed fixes so far. This is the last
iteration of the webrev [1], all the current changes are in this one,
the code hasn't been split into different webrevs. I'm also forwarding
to build-dev as there are some build related changes too. The CSR for
this change is at [2]
Would it be possible to summarize what will remain in
test/jdk/tools/pack200 after this removal? The webrev makes it looks
like badattr.jar is being added but since it already exists then I'm not
sure whether to believe it. pack200-verifier/data/golden.jar is another
one as it looks like JAR file that is generated by the tests today is
being checked in, maybe `hg add` in error?
The change to flags-cflag.m4 to add LP64=1 on Windows will need eyes,
it's not immediately obvious to me which shared code compiled on Windows
is impacted by this.
-Alan