Commit 3128cb7
test: avoid running fsync on directory on AIX
On AIX the underlying fsync system call returns EBADF on a file
descriptor for an open directory. So avoid running fsync on it.
PR-URL: #21298
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Signed-off-by: Beth Griggs <Bethany.Griggs@uk.ibm.com>1 parent 97cc0fc commit 3128cb7
1 file changed
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -35,7 +35,11 @@ function stat_resource(resource) { | |||
| 35 | 35 | if (typeof resource === 'string') { | |
| 36 | 36 | return fs.statSync(resource); | |
| 37 | 37 | } else { | |
| 38 | + const stats = fs.fstatSync(resource); | ||
| 38 | 39 | // ensure mtime has been written to disk | |
| 40 | + // except for directories on AIX where it cannot be synced | ||
| 41 | + if (common.isAIX && stats.isDirectory()) | ||
| 42 | + return stats; | ||
| 39 | 43 | fs.fsyncSync(resource); | |
| 40 | 44 | return fs.fstatSync(resource); | |
| 41 | 45 | } | |
0 commit comments