]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
include: move ALLPERMS definition to compat.h 50793/head
authorLucian Petrut <lpetrut@cloudbasesolutions.com>
Thu, 30 Mar 2023 12:14:10 +0000 (12:14 +0000)
committerLucian Petrut <lpetrut@cloudbasesolutions.com>
Fri, 31 Mar 2023 07:09:37 +0000 (07:09 +0000)
commita2966c08171e006acfd9c32b00a1294f428c7e62
treebad98437e441e10dca8a2eb9201356e4bf0e5e5a
parentaa9a83029a9a3e0133e866b452d3074cb8403c95
include: move ALLPERMS definition to compat.h

The Windows CI job started to fail as some libcephfs tests that use
ALLPERMS have been moved [1] to a separate file which doesn't have
the ALLPERMS definition.

  /ceph/src/test/libcephfs/suidsgid.cc:240:36: error: ‘ALLPERMS’ was
  not declared in this scope
  240 |   ASSERT_EQ(stx.stx_mode & (mode_t)ALLPERMS, before_mode);

We'll move this definition to compat.h so that we won't have to
redefine it in each file that uses it.

Note that we're moving the Windows "fs_compat.h" include up,
ensuring that the constants used by ALLPERMS are defined.

[1] https://github.com/ceph/ceph/commit/254eeb2730881ee48b4743e1422749d5b643f52a

Signed-off-by: Lucian Petrut <lpetrut@cloudbasesolutions.com>
src/include/compat.h
src/test/libcephfs/test.cc
src/test/libcephfs/vxattr.cc