Could I get the following small doc clarification reviewed please?
http://cr.openjdk.java.net/~michaelm/8225479/webrev.1/
Thanks
Michael
On 08/07/2019 19:47, Patrick Concannon wrote:
Hi Daniel,
Thanks for your feedback. I made the fixes that you requested in your
last email, and uploaded a new webrev. It can be accessed at the
following link: http://cr.openjdk.java.net/~aefimov/pconcann/8226756/01/
Looks good to me Patrick!
Hi all
Would you like to review this patch?
Bug: https://bugs.openjdk.java.net/browse/JDK-8227422
Webrev: http://cr.openjdk.java.net/~fyuan/8227422/webrev.00/
In this patch, I add read permission before cleaning up the test directory to
make sure the directory is removable on Windows
s
Hi Frank,
thanks for fixing this issue, in test setup we are setting the permissions explicitly to 333, can we set the permissions 733 in test setup itself ?.
i changed the permission to 733 in test setup and run the test at my Linux box and test ran successfully without any problem.
th
Hi Vyom
The test requires the directory to be unreadable.
Thanks
Frank
From: Vyom Tewari26 [mailto:vtewa...@in.ibm.com]
Sent: Wednesday, July 10, 2019 1:53 PM
To: frank.y...@oracle.com
Cc: net-dev@openjdk.java.net
Subject: Re: RFR (13) [testbug]: 8227422:
sun/net/www/protocol/file/Di
that's what i suspected, but test is passing at my local env even after i changes the permission to 733.
Vyom
- Original message -From: "Frank Yuan" To: "'Vyom Tewari26'" Cc: Subject: [EXTERNAL] RE: RFR (13) [testbug]: 8227422: sun/net/www/protocol/file/DirPermissionDenied.java failed on