LibC: Do not include errno.h inside unistd.h

POSIX does not mandate this, therefore let's not do it.

Tests/Kernel/bind-local-socket-to-symlink.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <AK/Assertions.h>
+#include <errno.h>
 #include <stdio.h>
 #include <string.h>
 #include <sys/socket.h>

Tests/Kernel/crash-fcntl-invalid-cmd.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <assert.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <stdlib.h>

Tests/Kernel/setpgid-across-sessions-without-leader.cpp

@@ -6,6 +6,7 @@
 
 #include <AK/Assertions.h>
 #include <AK/Format.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <serenity.h>
 #include <stdio.h>

Tests/LibC/TestLibCExec.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <LibTest/TestCase.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <unistd.h>

Tests/LibPthread/TestLibPthreadSpinLocks.cpp

@@ -6,6 +6,7 @@
 
 #include <LibPthread/pthread.h>
 #include <LibTest/TestCase.h>
+#include <errno.h>
 #include <unistd.h>
 
 TEST_CASE(spin_init_process_scope)

Userland/Libraries/LibC/malloc.cpp

@@ -11,6 +11,7 @@
 #include <LibELF/AuxiliaryVector.h>
 #include <LibThread/Lock.h>
 #include <assert.h>
+#include <errno.h>
 #include <mallocdefs.h>
 #include <serenity.h>
 #include <stdio.h>

Userland/Libraries/LibC/netdb.cpp

@@ -10,6 +10,7 @@
 #include <AK/String.h>
 #include <Kernel/Net/IPv4.h>
 #include <arpa/inet.h>
+#include <errno.h>
 #include <netdb.h>
 #include <netinet/in.h>
 #include <stdio.h>

Userland/Libraries/LibC/pthread_integration.cpp

@@ -8,6 +8,7 @@
 #include <AK/NeverDestroyed.h>
 #include <AK/Vector.h>
 #include <bits/pthread_integration.h>
+#include <errno.h>
 #include <sched.h>
 #include <unistd.h>
 

Userland/Libraries/LibC/pthread_tls.cpp

@@ -6,6 +6,7 @@
 
 #include <AK/Atomic.h>
 #include <LibPthread/pthread.h>
+#include <errno.h>
 #include <unistd.h>
 
 #ifndef _DYNAMIC_LOADER

Userland/Libraries/LibC/pty.cpp

@@ -7,6 +7,7 @@
  */
 
 #include <AK/Format.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <pty.h>
 #include <stdlib.h>

Userland/Libraries/LibC/pwd.cpp

@@ -7,7 +7,7 @@
 #include <AK/String.h>
 #include <AK/TemporaryChange.h>
 #include <AK/Vector.h>
-#include <errno_numbers.h>
+#include <errno.h>
 #include <pwd.h>
 #include <stdio.h>
 #include <stdlib.h>

Userland/Libraries/LibC/shadow.cpp

@@ -8,7 +8,7 @@
 #include <AK/String.h>
 #include <AK/TemporaryChange.h>
 #include <AK/Vector.h>
-#include <errno_numbers.h>
+#include <errno.h>
 #include <shadow.h>
 #include <stdio.h>
 #include <stdlib.h>

Userland/Libraries/LibC/spawn.cpp

@@ -15,6 +15,7 @@
 
 #include <AK/Function.h>
 #include <AK/Vector.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <stdlib.h>

Userland/Libraries/LibC/sys/wait.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <assert.h>
+#include <errno.h>
 #include <sys/wait.h>
 #include <syscall.h>
 #include <unistd.h>

Userland/Libraries/LibC/unistd.h

@@ -13,7 +13,6 @@
 
 #pragma once
 
-#include <errno.h>
 #include <fd_set.h>
 #include <limits.h>
 #include <sys/cdefs.h>

Userland/Libraries/LibCore/FileWatcher.cpp

@@ -18,6 +18,7 @@
 #include <Kernel/API/InodeWatcherFlags.h>
 #include <LibCore/DirIterator.h>
 #include <LibCore/Notifier.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <string.h>
 #include <sys/stat.h>

Userland/Libraries/LibIPC/Decoder.cpp

@@ -11,6 +11,7 @@
 #include <LibIPC/Decoder.h>
 #include <LibIPC/Dictionary.h>
 #include <LibIPC/File.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <string.h>
 #include <sys/socket.h>

Userland/Libraries/LibPthread/pthread.cpp

@@ -12,6 +12,7 @@
 #include <Kernel/API/Syscall.h>
 #include <LibSystem/syscall.h>
 #include <bits/pthread_integration.h>
+#include <errno.h>
 #include <limits.h>
 #include <pthread.h>
 #include <serenity.h>

Userland/Services/KeyboardPreferenceLoader/main.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <LibCore/ConfigFile.h>
+#include <errno.h>
 #include <spawn.h>
 #include <stdio.h>
 #include <unistd.h>

Userland/Services/LaunchServer/Launcher.cpp

@@ -13,6 +13,7 @@
 #include <AK/StringBuilder.h>
 #include <LibCore/ConfigFile.h>
 #include <LibDesktop/AppFile.h>
+#include <errno.h>
 #include <serenity.h>
 #include <spawn.h>
 #include <stdio.h>

Userland/Utilities/find.cpp

@@ -9,6 +9,7 @@
 #include <AK/OwnPtr.h>
 #include <AK/Vector.h>
 #include <LibCore/DirIterator.h>
+#include <errno.h>
 #include <getopt.h>
 #include <grp.h>
 #include <pwd.h>

Userland/Utilities/flock.cpp

@@ -4,6 +4,7 @@
  * SPDX-License-Identifier: BSD-2-Clause
  */
 
+#include <errno.h>
 #include <spawn.h>
 #include <stdio.h>
 #include <stdlib.h>

Userland/Utilities/mkdir.cpp

@@ -8,6 +8,7 @@
 #include <AK/LexicalPath.h>
 #include <AK/StringBuilder.h>
 #include <LibCore/ArgsParser.h>
+#include <errno.h>
 #include <sys/stat.h>
 #include <unistd.h>
 

Userland/Utilities/mktemp.cpp

@@ -7,6 +7,7 @@
 #include <AK/LexicalPath.h>
 #include <AK/Random.h>
 #include <LibCore/ArgsParser.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <string.h>

Userland/Utilities/ping.cpp

@@ -6,6 +6,7 @@
 
 #include <LibCore/ArgsParser.h>
 #include <arpa/inet.h>
+#include <errno.h>
 #include <netdb.h>
 #include <netinet/in.h>
 #include <netinet/ip_icmp.h>

Userland/Utilities/sleep.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <LibCore/ArgsParser.h>
+#include <errno.h>
 #include <signal.h>
 #include <stdio.h>
 #include <string.h>

Userland/Utilities/test-unveil.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <LibCore/ArgsParser.h>
+#include <errno.h>
 #include <stdio.h>
 #include <string.h>
 #include <unistd.h>

Userland/Utilities/test_efault.cpp

@@ -6,6 +6,7 @@
 
 #include <AK/Assertions.h>
 #include <AK/Types.h>
+#include <errno.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <sys/mman.h>

Userland/Utilities/traceroute.cpp

@@ -8,6 +8,7 @@
 #include <LibCore/ArgsParser.h>
 #include <LibCore/ElapsedTimer.h>
 #include <arpa/inet.h>
+#include <errno.h>
 #include <netdb.h>
 #include <netinet/in.h>
 #include <netinet/ip_icmp.h>

Userland/Utilities/tt.cpp

@@ -5,6 +5,7 @@
  */
 
 #include <LibCore/ArgsParser.h>
+#include <errno.h>
 #include <mman.h>
 #include <pthread.h>
 #include <stdio.h>

Userland/Utilities/useradd.cpp

@@ -6,6 +6,7 @@
 #include <AK/String.h>
 #include <LibCore/ArgsParser.h>
 #include <ctype.h>
+#include <errno.h>
 #include <pwd.h>
 #include <stdio.h>
 #include <string.h>

Userland/Utilities/watch.cpp

@@ -9,6 +9,7 @@
 #include <AK/Time.h>
 #include <AK/Vector.h>
 #include <LibCore/ArgsParser.h>
+#include <errno.h>
 #include <spawn.h>
 #include <stdio.h>
 #include <sys/time.h>