]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: fix install packages function 42739/head
authorMatan Breizman <Matan.Brz@gmail.com>
Tue, 10 Aug 2021 17:21:38 +0000 (17:21 +0000)
committerMatan Breizman <Matan.Brz@gmail.com>
Tue, 10 Aug 2021 20:40:53 +0000 (20:40 +0000)
remove_all() doesn't return bool,
instead it returns static_cast<std::uintmax_t>(-1) on error.

Signed-off-by: Matan Breizman <Matan.Brz@gmail.com>
src/rgw/rgw_lua.cc

index bd08f06bcd99b5affa11586a04ed57bddd3fa8ef..46c194db987094dccfb1076d70a40cc6f228d179 100644 (file)
@@ -198,7 +198,8 @@ int install_packages(const DoutPrefixProvider *dpp, rgw::sal::Store* store, opti
   // luarocks directory cleanup
   std::error_code ec;
   const auto& luarocks_path = store->get_luarocks_path();
-  if (!std::filesystem::remove_all(luarocks_path, ec) &&
+  if (std::filesystem::remove_all(luarocks_path, ec)
+      == static_cast<std::uintmax_t>(-1) &&
       ec != std::errc::no_such_file_or_directory) {
     output.append("failed to clear luarock directory: ");
     output.append(ec.message());