WIP - expect failures

mozilla · Nov 25, 2023 · d20a2ca · d20a2ca
1 parent 993ab57
commit d20a2ca
Show file tree

Hide file tree

Showing 26 changed files with 57 additions and 73 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/automation/emit_coverage_info.sh b/automation/emit_coverage_info.sh
@@ -54,4 +54,4 @@ cargo +nightly run -p sync-test || true
 cargo +nightly run -p protobuf-gen -- tools/protobuf_files.toml || true
 cargo +nightly run -p systest || true
 
-env RUSTFLAGS="--cfg __appsvc_ci_hack $RUSTFLAGS" cargo +nightly test --all --all-features --no-fail-fast || true
+env RUSTFLAGS="--cfg __appsvc_ci_sqlite_hack $RUSTFLAGS" cargo +nightly test --all --all-features --no-fail-fast || true
diff --git a/automation/tests.py b/automation/tests.py
@@ -277,11 +277,11 @@ def print_rust_environment():
 
 def calc_rust_env(package, features):
     if features == RustFeatures.ALL:
-        # nss-sys's --features handling is broken.  Workaround it by using a
-        # custom --cfg.  This shouldn't be this way!
+        # Anything needing special support for sqlite might fail to link it.
+        # `--all-features` in CI hits this - it shouldn't be this way!
         return {
             **os.environ,
-            'RUSTFLAGS' : "--cfg __appsvc_ci_hack"
+            'RUSTFLAGS' : "--cfg __appsvc_ci_sqlite_hack"
         }
     else:
         return None

diff --git a/components/as-ohttp-client/Cargo.toml b/components/as-ohttp-client/Cargo.toml
@@ -14,7 +14,7 @@ uniffi = "0.24.1"
 thiserror = "1.0"
 bhttp = "0.3"
 parking_lot = "0.12"
-rusqlite = { version = "0.29.0", features = ["bundled"] }
+rusqlite = "0.29.0"
 
 [dependencies.ohttp]
 version = "0.4"
@@ -23,5 +23,8 @@ git = "https://github.com/martinthomson/ohttp.git"
 rev = "fc3f4c787d1f6a6a87bf5194f7152cc906b02973"
 features = ["client", "server", "app-svc", "external-sqlite"]
 
+[dev-dependencies]
+rusqlite = { version = "0.29.0", features = ["bundled"] }
+
 [build-dependencies]
 uniffi = { version = "0.24.1", features=["build"]}
diff --git a/components/autofill/Cargo.toml b/components/autofill/Cargo.toml
@@ -25,14 +25,11 @@ thiserror = "1.0"
 types = { path = "../support/types" }
 uniffi = "0.24.1"
 url = { version = "2.2", features = ["serde"] }
-
-[dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "bundled", "serde_json", "unlock_notify"]
+rusqlite = { version = "0.29.0", features = ["functions"] }
 
 [dev-dependencies]
 env_logger = { version = "0.7", default-features = false }
-libsqlite3-sys = "0.26.0"
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 
 [build-dependencies]
 nss_build_common = { path = "../support/rc_crypto/nss/nss_build_common" }

diff --git a/components/logins/Cargo.toml b/components/logins/Cargo.toml
@@ -28,13 +28,11 @@ sync-guid = { path = "../support/guid", features = ["rusqlite_support", "random"
 thiserror = "1.0"
 anyhow = "1.0"
 uniffi = "0.24.1"
-
-[dependencies.rusqlite]
-version = "0.29.0"
-features = ["limits", "unlock_notify"]
+rusqlite = "0.29.0"
 
 [build-dependencies]
 uniffi = { version = "0.24.1", features = ["build"] }
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 
 [dev-dependencies]
 more-asserts = "0.2"

diff --git a/components/places/Cargo.toml b/components/places/Cargo.toml
@@ -34,15 +34,13 @@ sync-guid = { path = "../support/guid", features = ["rusqlite_support", "random"
 thiserror = "1.0"
 anyhow = "1.0"
 uniffi = "0.24.1"
-
-[dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "window", "bundled", "unlock_notify"]
+rusqlite = { version = "0.29.0", features = ["functions", "window"] }
 
 [dev-dependencies]
 pretty_assertions = "0.6"
 tempfile = "3.1"
 env_logger = {version = "0.7", default-features = false}
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 sql-support = { path = "../support/sql" }
 
 [build-dependencies]

diff --git a/components/push/Cargo.toml b/components/push/Cargo.toml
@@ -13,7 +13,7 @@ bincode = "1.2"
 lazy_static = "1.4"
 base64 = "0.21"
 log = "0.4"
-rusqlite = { version = "0.29.0", features = ["bundled", "unlock_notify"] }
+rusqlite = "0.29.0"
 url = "2.2"
 viaduct = { path = "../viaduct" }
 error-support = { path = "../support/error" }
@@ -24,6 +24,7 @@ uniffi = "0.24.1"
 types = { path = "../support/types" }
 
 [build-dependencies]
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 uniffi = { version = "0.24.1", features = ["build"] }
 
 [dev-dependencies]

diff --git a/components/suggest/Cargo.toml b/components/suggest/Cargo.toml
@@ -14,7 +14,7 @@ interrupt-support = { path = "../support/interrupt" }
 once_cell = "1.5"
 parking_lot = ">=0.11,<=0.12"
 remote_settings = { path = "../remote_settings" }
-rusqlite = { version = "0.29.0", features = ["functions", "bundled"] }
+rusqlite = { version = "0.29.0", features = ["functions"] }
 serde = { version = "1", features = ["derive"] }
 serde_json = "1"
 sql-support = { path = "../support/sql" }
@@ -26,6 +26,7 @@ env_logger = { version = "0.7", default-features = false }
 expect-test = "1.4"
 hex = "0.4"
 rc_crypto = { path = "../support/rc_crypto" }
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 
 [build-dependencies]
 uniffi = { version = "0.24.1", features = ["build"] }
diff --git a/components/support/guid/Cargo.toml b/components/support/guid/Cargo.toml
@@ -6,7 +6,7 @@ license = "MPL-2.0"
 edition = "2021"
 
 [dependencies]
-rusqlite = { version = "0.29.0", optional = true, features = ["bundled"] }
+rusqlite = { version = "0.29.0", optional = true }
 serde = { version = "1", optional = true }
 rand = { version = "0.8", optional = true }
 base64 = { version = "0.21", optional = true }
@@ -20,3 +20,4 @@ default = ["serde_support"]
 
 [dev-dependencies]
 serde_test = "1"
+rusqlite = { version = "0.29.0", features = ["bundled"] }
diff --git a/components/support/interrupt/Cargo.toml b/components/support/interrupt/Cargo.toml
@@ -8,7 +8,7 @@ edition = "2021"
 [dependencies]
 lazy_static = "1.4"
 parking_lot = ">=0.11,<=0.12"
+rusqlite = { version = "0.29.0", features = ["functions"] }
 
-[dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "limits", "bundled", "unlock_notify"]
+[dev-dependencies]
+rusqlite = { version = "0.29.0", features = ["bundled"] }
diff --git a/components/support/rc_crypto/Cargo.toml b/components/support/rc_crypto/Cargo.toml
@@ -17,8 +17,6 @@ nss = { path = "nss" }
 hawk = { version = "3.2", default-features = false, optional = true }
 ece = { version = "2.1", default-features = false, features = ["serializable-keys"], optional = true }
 
-[dev-dependencies]
-
 [dev-dependencies.ece]
 version = "2.0"
 default-features = false

diff --git a/components/support/rc_crypto/nss/nss_sys/src/lib.rs b/components/support/rc_crypto/nss/nss_sys/src/lib.rs
@@ -9,8 +9,11 @@
 mod bindings;
 pub use bindings::*;
 
-// So we link against the SQLite lib imported by parent crates
+// We need a sqlite - so we link against the SQLite lib imported by parent crates
 // such as places and logins.
+// But the "gecko" feature is confusingly different - we rely on an external linker
+// to put the bits together. `__appsvc_ci_sqlite_hack`` works around `--all-features`
+// errors in CI, where linker-related build failures are unavoidable.
 #[allow(unused_extern_crates)]
-#[cfg(any(not(feature = "gecko"), __appsvc_ci_hack))]
+#[cfg(any(not(feature = "gecko"), __appsvc_ci_sqlite_hack))]
 extern crate libsqlite3_sys;
diff --git a/components/support/sql/Cargo.toml b/components/support/sql/Cargo.toml
@@ -18,13 +18,11 @@ ffi-support = "0.4"
 thiserror = "1.0"
 tempfile = "3.1.0"
 prettytable-rs = { version = "0.10", optional = true }
-
-[dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "limits", "bundled", "unlock_notify"]
+rusqlite = { version = "0.29.0", features = ["functions", "limits"] }
 
 [dev-dependencies]
 env_logger = {version = "0.7", default-features = false}
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 
 [build-dependencies]
 nss_build_common = { path = "../rc_crypto/nss/nss_build_common" }
diff --git a/components/support/types/Cargo.toml b/components/support/types/Cargo.toml
@@ -10,7 +10,7 @@ edition = "2021"
 serde = "1"
 serde_derive = "1"
 serde_json = "1"
+rusqlite = "0.29.0"
 
-[dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "bundled"]
+[dev-dependencies]
+rusqlite = { version = "0.29.0", features = ["bundled"] }
diff --git a/components/sync15/Cargo.toml b/components/sync15/Cargo.toml
@@ -66,6 +66,9 @@ viaduct = { path = "../viaduct", optional = true }
 
 [dev-dependencies]
 env_logger = { version = "0.7", default-features = false }
+# rc_cryto needs nss, which needs sqlite...
+rusqlite = { version = "0.29.0", features = ["bundled"] }
+
 
 [build-dependencies]
 uniffi = { version = "0.24.1", features = ["build"] }
diff --git a/components/tabs/Cargo.toml b/components/tabs/Cargo.toml
@@ -12,7 +12,7 @@ error-support = { path = "../support/error" }
 interrupt-support = { path = "../support/interrupt" }
 lazy_static = "1.4"
 log = "0.4"
-rusqlite = { version = "0.29.0", features = ["bundled", "unlock_notify"] }
+rusqlite = "0.29.0"
 serde = "1"
 serde_derive = "1"
 serde_json = "1"
@@ -25,6 +25,7 @@ url = "2.1" # mozilla-central can't yet take 2.2 (see bug 1734538)
 
 [dev-dependencies]
 env_logger = { version = "0.8.0", default-features = false, features = ["termcolor", "atty", "humantime"] }
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 tempfile = "3.1"
 
 [build-dependencies]

diff --git a/components/webext-storage/Cargo.toml b/components/webext-storage/Cargo.toml
@@ -24,19 +24,13 @@ sql-support = { path = "../support/sql" }
 sync15 = {path = "../../components/sync15", features=["sync-engine"]}
 sync-guid = { path = "../support/guid", features = ["rusqlite_support", "random"] }
 url = { version = "2.1", features = ["serde"] }
-
-[dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "bundled", "serde_json", "unlock_notify"]
+rusqlite = { version = "0.29.0",features = ["functions", "serde_json"] }
 
 [dev-dependencies]
 env_logger = { version = "0.7", default-features = false }
-tempfile = "3"
-# A *direct* dep on the -sys crate is required for our build.rs
-# to see the DEP_SQLITE3_LINK_TARGET env var that cargo sets
-# on its behalf.
-libsqlite3-sys = "0.26.0"
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 sql-support = { path = "../support/sql" }
+tempfile = "3"
 
 [build-dependencies]
 nss_build_common = { path = "../support/rc_crypto/nss/nss_build_common" }
diff --git a/components/webext-storage/build.rs b/components/webext-storage/build.rs
diff --git a/components/webext-storage/ffi/Cargo.toml b/components/webext-storage/ffi/Cargo.toml
@@ -17,3 +17,6 @@ serde_json = "1"
 
 [dependencies.webext-storage]
 path = ".."
+
+[dev-dependencies]
+rusqlite = { version = "0.29.0", features = ["bundled"] }
diff --git a/examples/autofill-utils/Cargo.toml b/examples/autofill-utils/Cargo.toml
@@ -18,7 +18,7 @@ fxa-client = { path = "../../components/fxa-client" }
 error-support = { path = "../../components/support/error" }
 interrupt-support = { path = "../../components/support/interrupt" } # XXX - should be removed once we do interrupts correctly!
 log = "0.4"
-rusqlite = {version = "0.29.0", features = ["functions", "bundled", "serde_json", "unlock_notify"]}
+rusqlite = {version = "0.29.0", features = ["bundled"]}
 serde_json = "1"
 sql-support = { path = "../../components/support/sql" }
 structopt = "0.3"

diff --git a/examples/places-autocomplete/Cargo.toml b/examples/places-autocomplete/Cargo.toml
@@ -22,10 +22,7 @@ clap = "2.33"
 tempfile = "3.1"
 rand = "0.8"
 find-places-db = "0.2"
-
-[dev-dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "bundled"]
+rusqlite = { version = "0.29.0", features = ["bundled"] }
 
 # While we don't have a replacement for termion on Windows yet (and thus
 # our example doesn't work on Windows), it does get further in the compilation

diff --git a/examples/sync-pass/Cargo.toml b/examples/sync-pass/Cargo.toml
@@ -27,7 +27,4 @@ tempfile = "3"
 serde_json = "1.0"
 url = "2.2"
 viaduct-reqwest = { path = "../../components/support/viaduct-reqwest" }
-
-[dev-dependencies.rusqlite]
-version = "0.29.0"
-features = ["limits"]
+rusqlite = { version = "0.29.0", features = ["bundled"] }
diff --git a/megazords/full/Cargo.toml b/megazords/full/Cargo.toml
@@ -24,5 +24,7 @@ autofill = { path = "../../components/autofill" }
 crashtest = { path = "../../components/crashtest" }
 error-support = { path = "../../components/support/error" }
 suggest = { path = "../../components/suggest" }
+# Force the "bundled" feature????
+rusqlite = { version = "0.29.0" }
 
 lazy_static = "1.4"
diff --git a/megazords/ios-rust/Cargo.toml b/megazords/ios-rust/Cargo.toml
@@ -27,3 +27,6 @@ sync15 = {path = "../../components/sync15"}
 error-support = { path = "../../components/support/error" }
 sync_manager = { path = "../../components/sync_manager" }
 as-ohttp-client = { path = "../../components/as-ohttp-client" }
+
+# Force the "bundled" feature????
+rusqlite = { version = "0.29.0" }
diff --git a/testing/separated/places-tests/Cargo.toml b/testing/separated/places-tests/Cargo.toml
@@ -21,7 +21,4 @@ url = "2.2"
 dogear = "0.4"
 tempfile = "3.1"
 env_logger = { version = "0.7", default-features = false }
-
-[dev-dependencies.rusqlite]
-version = "0.29.0"
-features = ["functions", "bundled"]
+rusqlite = { version = "0.29.0", features = ["bundled"] }