[Bug 4351] New: double #include for _BIONIC_

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=4351

           Summary: double #include for _BIONIC_
           Product: Busybox
           Version: unspecified
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: trivial
          Priority: P5
         Component: Other
        AssignedTo: unassigned at busybox.net
        ReportedBy: mjt+busybox at tls.msk.ru
                CC: busybox-cvs at busybox.net
   Estimated Hours: 0.0


Commit 14bd16ac56e81e4912341cb731dc54af0dbe9e6c introduced a nice change:

--- a/util-linux/swaponoff.c
+++ b/util-linux/swaponoff.c
@@ -25,6 +25,9 @@
 #include "libbb.h"
 #include <mntent.h>
 #include <sys/swap.h>
+#ifndef __BIONIC__
+# include <sys/swap.h>
+#endif

 #if ENABLE_FEATURE_MOUNT_LABEL
 # include "volume_id.h"

So, sys/swap.h gets include twice for anything but _BIONIC_, and once for
_BIONIC_.  I guess the first #include were intended to go away with
introduction of the #if conditional.  Does it actually build with bionic?

Thanks,

/mjt

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.