Fix docs build for libafl_qemu (#1495)

* Fix docs build for libafl_qemu

* turn around

* does this work?

* build all-features in CI

* fixes

* fix ci

.github/workflows/build_and_test.yml

@@ -91,10 +91,9 @@ jobs:
       
     # ---- doc check ----
     - name: Build Docs
-      run: cargo +nightly doc --all-features
+      run: RUSTFLAGS="--cfg docsrs" cargo +nightly doc --all-features
     - name: Test Docs
-      run: cargo +nightly test --doc --all-features
+      run: RUSTFLAGS="--cfg docsrs" cargo +nightly test --doc --all-features
-
     # ---- build normal and examples ----
     - name: Run a normal build
       run: cargo build --verbose

fuzzers/qemu_coverage/build.rs

@@ -5,8 +5,7 @@ macro_rules! assert_unique_feature {
     () => {};
     ($first:tt $(,$rest:tt)*) => {
         $(
-            #[cfg(not(feature = "clippy"))] // ignore multiple definition for clippy
+            #[cfg(all(not(any(doc, feature = "clippy")), feature = $first, feature = $rest))]
-            #[cfg(all(feature = $first, feature = $rest))]
             compile_error!(concat!("features \"", $first, "\" and \"", $rest, "\" cannot be used together"));
         )*
         assert_unique_feature!($($rest),*);

fuzzers/qemu_launcher/build.rs

@@ -5,8 +5,7 @@ macro_rules! assert_unique_feature {
     () => {};
     ($first:tt $(,$rest:tt)*) => {
         $(
-            #[cfg(not(feature = "clippy"))] // ignore multiple definition for clippy
+            #[cfg(all(not(any(doc, feature = "clippy")), feature = $first, feature = $rest))]
-            #[cfg(all(feature = $first, feature = $rest))]
             compile_error!(concat!("features \"", $first, "\" and \"", $rest, "\" cannot be used together"));
         )*
         assert_unique_feature!($($rest),*);

libafl/Cargo.toml

@@ -14,6 +14,7 @@ categories = ["development-tools::testing", "emulators", "embedded", "os", "no-s
 [package.metadata.docs.rs]
 features = ["document-features"]
 all-features = true
+rustc-args = ["--cfg", "docsrs"]
 
 [features]
 default = ["std", "derive", "llmp_compression", "llmp_small_maps", "llmp_broker_timeouts", "rand_trait", "fork", "prelude", "gzip", "regex", "serdeany_autoreg", "tui_monitor", "libafl_bolts/xxh3"]

libafl/build.rs

@@ -8,7 +8,7 @@ fn main() {
 fn main() {
     println!("cargo:rerun-if-changed=build.rs");
     assert!(
-        cfg!(not(feature = "nautilus")),
+        cfg!(all(not(docrs), not(feature = "nautilus"))),
         "The 'nautilus' feature of libafl requires a nightly compiler"
     );
 }

libafl_qemu/Cargo.toml

@@ -13,6 +13,7 @@ categories = ["development-tools::testing", "emulators", "embedded", "os", "no-s
 
 [package.metadata.docs.rs]
 all-features = true
+rustdoc-args = ["--cfg", "docsrs"]
 
 [features]
 default = ["fork", "build_libqasan", "serdeany_autoreg"]

libafl_qemu/libafl_qemu_sys/Cargo.toml

@@ -13,6 +13,7 @@ categories = ["development-tools::testing", "emulators", "embedded", "os", "no-s
 
 [package.metadata.docs.rs]
 all-features = true
+rustdoc-args = ["--cfg", "docsrs"]
 
 [features]
 # The following architecture features are mutually exclusive.

libafl_qemu/libafl_qemu_sys/build_linux.rs

@@ -7,8 +7,7 @@ macro_rules! assert_unique_feature {
     () => {};
     ($first:tt $(,$rest:tt)*) => {
         $(
-            #[cfg(not(feature = "clippy"))] // ignore multiple definition for clippy
+            #[cfg(all(not(any(docsrs, feature = "clippy")), feature = $first, feature = $rest))]
-            #[cfg(all(feature = $first, feature = $rest))]
             compile_error!(concat!("features \"", $first, "\" and \"", $rest, "\" cannot be used together"));
         )*
         assert_unique_feature!($($rest),*);