From 27088eda547f2aeaa99d42ee7d28f4f8bfbd97be Mon Sep 17 00:00:00 2001
From: Christian Heussy <heussy@gmail.com>
Date: Sun, 8 Oct 2023 20:13:55 -0700
Subject: [PATCH] feat: honor `no_default_flags` setting

Problem: `init_c_cfg` permits the user to set a `cc::Build`
configuration. The setting for `no_default_flags` is always overridden by
`uses_android_sdk`. This prevents a user from building with
`no_default_flags = true`.

Solution: Do not override `no_default_flags` value. In the event
`uses_android_sdk` returns true, continue to set `no_default_flags` to
true. Note that `no_default_flags` defaults to false.

Issue: https://github.com/rust-lang/cmake-rs/issues/188
---
 src/lib.rs | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/lib.rs b/src/lib.rs
index 035cead..f617923 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -509,8 +509,13 @@ impl Config {
             .opt_level(0)
             .debug(false)
             .warnings(false)
-            .host(&host)
-            .no_default_flags(ndk);
+            .host(&host);
+
+        // Maintain backwards compatibility by setting `no_default_flag`
+        if ndk {
+            c_cfg.no_default_flags(true);
+        }
+
         if !ndk {
             c_cfg.target(&target);
         }
@@ -521,8 +526,13 @@ impl Config {
             .opt_level(0)
             .debug(false)
             .warnings(false)
-            .host(&host)
-            .no_default_flags(ndk);
+            .host(&host);
+
+        // Maintain backwards compatibility by setting `no_default_flag`
+        if ndk {
+            cxx_cfg.no_default_flags(true);
+        }
+
         if !ndk {
             cxx_cfg.target(&target);
         }