That didn't answer all my questions, at least not in a way that I can
understand.
How is this useful given that we disable jtreg failure handlers for the
headful tests ?
-phil.
On 12/30/19, 11:33 AM, Sergey Bylokhov wrote:
On 12/23/19 9:15 pm, Phil Race wrote:
I am not sure what the right mailing list(s) are for this change.
It definitely isn't a core-libs change. I think build-dev may be better.
Previous changes to these configs were discussed here, so I have send
it here as well.
I am also unclear when this failure handler is invoked and how all
this machinery works.
It is only useful for headful tests and so I'd only want it enabled
in such a case.
And we disable the failure handlers in the headful test jobs anyway
because they seem
focused on taking pointless core dumps ...> So we need something that
can be used with headful tests only and doesn't involve
re-enabling the other handlers.
It could be useful for other tests as well and may be able to identify
problems such as:
- Suggestions "to open under debugger" from the native asserts
- Various error dialogs from the OS
And it does not spend much resources compared to current handlers.
Also why exclude Windows ? No easy way to get the screenshot ?
There is no command-line program that can take a screenshot on windows
by default
-phil.
On 12/11/19 1:00 AM, Sergey Bylokhov wrote:
Hello.
Please review the fix for JDK 14.
Bug: https://bugs.openjdk.java.net/browse/JDK-8233827
Fix: http://cr.openjdk.java.net/~serb/8233827/webrev.01
This change adds the "screen capture on the test failure" feature on
macOS and Linux.
- On Linux, it is implemented by the "gnome-screenshot" command(in
case of
multiscreen+xinerama the whole big screen will be saved to the
"screen.png" file).
- On macOS it is implemented by the "screencapture" command, note
that I have
used 1 file per screen, if the number of screens less than 5,
other files will be ignored.