fuchsia: base_unittests (mostly) compiling

base/files/file_util_posix.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/files/file_util.h"
 
 #include <dirent.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <libgen.h>
 #include <limits.h>
 #include <stddef.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <sys/errno.h>
 #include <sys/mman.h>
 #include <sys/param.h>
 #include <sys/stat.h>
 #include <sys/time.h>
 #include <sys/types.h>
 #include <time.h>
 #include <unistd.h>
 
 #include "base/environment.h"
 #include "base/files/file_enumerator.h"
@@ skipping 22 matching lines @@
 
 #if defined(OS_ANDROID)
 #include "base/android/content_uri_utils.h"
 #include "base/os_compat_android.h"
 #endif
 
 #if !defined(OS_IOS)
 #include <grp.h>
 #endif
 
+#if !defined(OS_FUCHSIA)
+#include <sys/errno.h>

[Nico, 2017/05/17 15:52:53]

Doesn't the warning just let you know that this should be <errno.h>, not
<sys/errno.h>? Have you tried just deleting "sys/"?

[scottmg, 2017/05/17 16:52:51]

Ah, I assumed this was a Fuchsia-specific thing, but I just check a Linux and
Android sys/errno.h and they just do #include <errno.h> without the #warning. So
I just deleted this. (We already #include <errno.h> earlier in this file.

+#endif
+
 // We need to do this on AIX due to some inconsistencies in how AIX
 // handles XOPEN_SOURCE and ALL_SOURCE.
 #if defined(OS_AIX)
 extern "C" char* mkdtemp(char* path);
 #endif
 
 namespace base {
 
 namespace {
 
@@ skipping 939 matching lines @@
     return false;
 
   DeleteFile(from_path, true);
   return true;
 }
 
 }  // namespace internal
 
 #endif  // !defined(OS_NACL_NONSFI)
 }  // namespace base