Whamcloud - gitweb
AOSP: Fix e2fsdroid build with musl
authorColin Cross <ccross@android.com>
Tue, 17 Aug 2021 00:17:03 +0000 (17:17 -0700)
committerTheodore Ts'o <tytso@mit.edu>
Tue, 13 Sep 2022 14:37:11 +0000 (10:37 -0400)
The e2fsdroid build fails with musl because config.h is not included
before ext2fs.h, which causes HAVE_SYS_TYPES_H not to be defined
resulting in a missing definition for dev_t.

Include config.h at the top of each .c file, and remove extra
config.h include from perms.h.

Bug: 190084016
Test: m USE_HOST_MUSL=true fastboot
Change-Id: I95b3fff3f10ba85c00ec049811dd6b5d412e5dd2
From AOSP commit: 09c63d5edd35e3ca8366be0d92aad922d8895ac1

contrib/android/e2fsdroid.c
contrib/android/perms.c
contrib/android/perms.h
e2fsck/jfs_user.h

index f5d24b8..6e51414 100644 (file)
@@ -1,11 +1,11 @@
 #define _GNU_SOURCE
 
+#include "config.h"
 #include <stdio.h>
 #include <getopt.h>
 #include <string.h>
 #include <unistd.h>
 #include <limits.h>
-#include <ext2fs/ext2fs.h>
 
 #include "perms.h"
 #include "base_fs.h"
index 9c5ec05..dd05644 100644 (file)
@@ -1,6 +1,7 @@
 #ifndef _GNU_SOURCE
 # define _GNU_SOURCE //asprintf
 #endif
+#include "config.h"
 #include "perms.h"
 #include "support/nls-enable.h"
 #include <time.h>
index 7987127..9ea3f95 100644 (file)
@@ -1,7 +1,6 @@
 #ifndef ANDROID_PERMS_H
 # define ANDROID_PERMS_H
 
-# include "config.h"
 # include <ext2fs/ext2fs.h>
 
 typedef void (*fs_config_f)(const char *path, int dir,
index 4ad2005..969cd1b 100644 (file)
@@ -11,6 +11,8 @@
 #ifndef _JFS_USER_H
 #define _JFS_USER_H
 
+#include "config.h"
+
 #ifdef DEBUGFS
 #include <stdio.h>
 #include <stdlib.h>
@@ -27,7 +29,6 @@
 /*
  * Pull in the definition of the e2fsck context structure
  */
-#include "config.h"
 #include "e2fsck.h"
 #endif