Re: RFR: 8274750: java/io/File/GetXSpace.java failed: '/dev': 191488 != 190976 [v2]

2021-10-28 Thread Brian Burkhalter
On Thu, 28 Oct 2021 19:33:06 GMT, Roger Riggs wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8274750: Change location of check that file system is not /dev on macOS > > LTGM > > (While you're in there you cou

Re: RFR: 8274750: java/io/File/GetXSpace.java failed: '/dev': 191488 != 190976 [v2]

2021-10-28 Thread Naoto Sato
On Thu, 28 Oct 2021 18:15:33 GMT, Brian Burkhalter wrote: >> Please consider this proposed change to ignore comparing the total size of >> `/dev` as returned by `DF(1)` and `STAT(2)` on macOS. > > Brian Burkhalter has updated the pull request incrementally with one > additional commit since the

Re: RFR: 8274750: java/io/File/GetXSpace.java failed: '/dev': 191488 != 190976 [v2]

2021-10-28 Thread Roger Riggs
On Thu, 28 Oct 2021 18:15:33 GMT, Brian Burkhalter wrote: >> Please consider this proposed change to ignore comparing the total size of >> `/dev` as returned by `DF(1)` and `STAT(2)` on macOS. > > Brian Burkhalter has updated the pull request incrementally with one > additional commit since the

Re: RFR: 8274750: java/io/File/GetXSpace.java failed: '/dev': 191488 != 190976 [v2]

2021-10-28 Thread Brian Burkhalter
On Tue, 26 Oct 2021 20:27:47 GMT, Brian Burkhalter wrote: >> test/jdk/java/io/File/GetXSpace.java line 210: >> >>> 208: if (Platform.isOSX() && s.name().equals("/dev")) { >>> 209: out.println("/dev:\n Skipping size comparison for /dev on >>> macOS"); >>> 210: re

Re: RFR: 8274750: java/io/File/GetXSpace.java failed: '/dev': 191488 != 190976 [v2]

2021-10-28 Thread Brian Burkhalter
> Please consider this proposed change to ignore comparing the total size of > `/dev` as returned by `DF(1)` and `STAT(2)` on macOS. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision: 8274750: Change location of check that file syst