[libcxx-commits] [PATCH] D121140: [libc++][AIX] AIX allows for changing permissions of symlinks

Zarko Todorovski via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Mar 21 11:39:43 PDT 2022


This revision was automatically updated to reflect the committed changes.
Closed by commit rG0f0520003af4: [libc++][AIX] AIX allows for changing permissions of symlinks (authored by ZarkoCA).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121140/new/

https://reviews.llvm.org/D121140

Files:
  libcxx/test/std/input.output/filesystems/fs.op.funcs/fs.op.permissions/permissions.pass.cpp


Index: libcxx/test/std/input.output/filesystems/fs.op.funcs/fs.op.permissions/permissions.pass.cpp
===================================================================
--- libcxx/test/std/input.output/filesystems/fs.op.funcs/fs.op.permissions/permissions.pass.cpp
+++ libcxx/test/std/input.output/filesystems/fs.op.funcs/fs.op.permissions/permissions.pass.cpp
@@ -8,8 +8,6 @@
 
 // UNSUPPORTED: c++03
 
-// XFAIL: LIBCXX-AIX-FIXME
-
 // <filesystem>
 
 // void permissions(const path& p, perms prms,
@@ -164,7 +162,7 @@
         {perms::owner_all, perms::group_all, perm_options::remove},
     };
     for (auto const& TC : cases) {
-#if defined(__APPLE__) || defined(__FreeBSD__) || defined(__NetBSD__)
+#if defined(__APPLE__) || defined(__FreeBSD__) || defined(__NetBSD__) || defined(_AIX)
         // On OS X symlink permissions are supported. We should get an empty
         // error code and the expected permissions.
         const auto expected_link_perms = TC.expected;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121140.417037.patch
Type: text/x-patch
Size: 977 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220321/f64a8ed5/attachment.bin>


More information about the libcxx-commits mailing list