[libc-commits] [libc] [libc] Implement lchown in unistd (PR #230851)

[Pavel Labath via libc-commits]

================
@@ -0,0 +1,43 @@
+//===----------------------------------------------------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+///
+/// \file
+/// Syscall wrapper for lchown.
+///
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_LIBC_SRC___SUPPORT_OSUTIL_LINUX_SYSCALL_WRAPPERS_LCHOWN_H
+#define LLVM_LIBC_SRC___SUPPORT_OSUTIL_LINUX_SYSCALL_WRAPPERS_LCHOWN_H
+
+#include "hdr/fcntl_macros.h"
+#include "hdr/types/gid_t.h"
+#include "hdr/types/uid_t.h"
+#include "src/__support/OSUtil/linux/syscall.h" // syscall_checked
+#include "src/__support/common.h"
+#include "src/__support/error_or.h"
+#include "src/__support/macros/config.h"
+#include <sys/syscall.h> // For syscall numbers
+
+namespace LIBC_NAMESPACE_DECL {
+namespace linux_syscalls {
+
+LIBC_INLINE ErrorOr<int> lchown(const char *path, uid_t owner, gid_t group) {
+#ifdef SYS_fchownat
----------------
labath wrote:

We're not terribly consistent about this, though I'd suggest using `#if` in all branches, if one of them has to be...

```suggestion
#if defined(SYS_fchownat)
```

https://github.com/llvm/llvm-project/pull/230851