That will be done subsequently and tracked under a different bug. Don't
you think pushing this fix is better than quarantining the tests?
Harsha
On Wednesday 27 June 2018 08:50 PM, mandy chung wrote:
I would suggest to take the time and replace it with java.nio.file API
and remove revokeall.exe sooner rather than later.
Mandy
On 6/26/18 7:09 AM, Harsha Wardhana B wrote:
Hi All,
Please find the fix for the bug,
https://bugs.openjdk.java.net/browse/JDK-8192953
having webrev at,
http://cr.openjdk.java.net/~hb/8192953/webrev.00/
The fix grants execute permission for revokeall.exe. The paths in the
shell sciprt had to be converted to cygwin paths (/cygwin/c/... )
from windows path (C:/...). Using windows path was causing strange
behavior in cygwin.
revokeall.exe should be removed and the above tests need to be
refactored to use java.nio.Acl* APIs. That plan is in the near
future, and the current fix needs to go in to stop consistent
failures in Mach5.
Please review the above patch and provide feedback if any.
Thanks
Harsha
