+1
Am 2015-01-30 um 12:25 schrieb Attila Szegedi:
Thanks for the approval. I have identified yet another change that needs to be made, though; a
test for as backported for JDK-8066232 was incomplete (it was incomplete in 9 too - I
investigated the history of it in 9 and saw that it was just fixed later in a 9-only
changeset). A new webrev is at <http://cr.openjdk.java.net/~attila/8071991/webrev.01>
with a trivial addition to test/script/basic/JDK-8066232.js. I also updated the bug to reflect
it's now a more general "fix all build/test issues introduced by the last two Nashorn
backports".
I can confirm that with these two changes all tests pass, so this should be the
final word on this.
Attila.
On Jan 30, 2015, at 12:07 PM, Seán Coffey <[email protected]> wrote:
Looks fine. Thanks for jumping on this. Approved for jdk8u-dev. Please add
noreg-build and 9-na labels.
regards,
Sean.
On 30/01/2015 11:05, Hannes Wallnoefer wrote:
+1
Am 2015-01-30 um 12:00 schrieb Attila Szegedi:
Please review JDK-8071991 at <http://cr.openjdk.java.net/~attila/8071991/webrev.00>
for <https://bugs.openjdk.java.net/browse/JDK-8071991>
Unfortunately, backporting JDK-8067139 bit-by-bit managed to introduce a
compile error as 8u-dev Nashorn diverges from 9 Nashorn in Parser.java. I
failed to notice and fix that in time, so I expect 8u-dev build now breaks.
Fixing post haste. I will suggest we backport the parser changes too so this
can't cause issues in the future.
Thanks and apologies,
Attila.
