On 29-10-2018 21:45, Noah Watkins wrote:
TEMP_FAILURE_RETRY is included in src/include/compat.h, but that isn’t included in blkdev.cc, so I am not sure how this is supposed to compile? What am I missing?
This is probably because musl c doesn't provide that macro (found in
unistd.h on glibc). I suspect then that blkdev.cc should include the
compat header.
This is indeed fixable by including compat.h
ACCESSPERMS is a bit more complicated, unless we lift the definition for
missing that also compat.h
./src/client/posix_acl.cc:6:#ifndef ACCESSPERMS
./src/client/posix_acl.cc:7:#define ACCESSPERMS (S_IRWXU|S_IRWXG|S_IRWXO)
./src/client/posix_acl.cc:115: *mode_p = (*mode_p & ~ACCESSPERMS) | mode;
./src/client/posix_acl.cc:177: *mode_p = (*mode_p & ~ACCESSPERMS) | mode;
./src/common/common_init.cc:129: if (!(ret & (~ACCESSPERMS))) {
Let me know if you don't want to make a PR for this yourself, then I'll
put one in.
--WjW
