libbpf-cargo: Clean up crate-structure mess

It goes against every Cargo project preconception to have certain
modules be compiled from both lib.rs and main.rs and it leads to
shenanigans such as #[allow(dead_code)] tags scattered all over the
place, which is its own can of worms when done at such a high level. Do
it properly and export necessary functionality from the library, for
consumption by the binary. To prevent any misconceptions of there being
a stable API contract for functionality *only* being used by the binary,
put it into an hidden module.

Signed-off-by: Daniel Müller <deso@posteo.net>

Author: d-e-s-o

libbpf-cargo/src/build.rs

@@ -279,7 +279,7 @@ fn extract_clang_or_default(clang: Option<&PathBuf>) -> PathBuf {
 }
 
 #[allow(clippy::too_many_arguments)]
-pub(crate) fn build(
+pub fn build(
     debug: bool,
     manifest_path: Option<&PathBuf>,
     clang: Option<&PathBuf>,

libbpf-cargo/src/gen/mod.rs

@@ -1,4 +1,4 @@
-pub mod btf;
+mod btf;
 
 use std::borrow::Cow;
 use std::collections::BTreeMap;
@@ -1352,7 +1352,7 @@ fn gen_project(
     Ok(())
 }
 
-pub(crate) fn gen(
+pub fn gen(
     debug: bool,
     manifest_path: Option<&PathBuf>,
     rustfmt_path: Option<&PathBuf>,

libbpf-cargo/src/lib.rs

@@ -77,20 +77,15 @@ use anyhow::Result;
 use tempfile::tempdir;
 use tempfile::TempDir;
 
-// libbpf-cargo binary is the primary consumer of the following modules. As such,
-// we do not use all the symbols. Silence any unused code warnings.
-#[allow(dead_code)]
 mod build;
-#[allow(dead_code)]
 mod gen;
-#[allow(dead_code)]
 mod make;
-#[allow(dead_code)]
 mod metadata;
 
 #[cfg(test)]
 mod test;
 
+
 /// `SkeletonBuilder` builds and generates a single skeleton.
 ///
 /// This interface is meant to be used in build scripts.
@@ -285,3 +280,20 @@ impl SkeletonBuilder {
         Ok(())
     }
 }
+
+
+/// Implementation details shared with the binary.
+///
+/// NOT PART OF PUBLIC API SURFACE!
+#[doc(hidden)]
+pub mod __private {
+    pub mod build {
+        pub use crate::build::build;
+    }
+    pub mod gen {
+        pub use crate::gen::gen;
+    }
+    pub mod make {
+        pub use crate::make::make;
+    }
+}

libbpf-cargo/src/main.rs

@@ -9,11 +9,10 @@ use clap::Args;
 use clap::Parser;
 use clap::Subcommand;
 
-#[doc(hidden)]
-mod build;
-mod gen;
-mod make;
-mod metadata;
+use libbpf_cargo::__private::build;
+use libbpf_cargo::__private::gen;
+use libbpf_cargo::__private::make;
+
 
 #[doc(hidden)]
 #[derive(Debug, Parser)]

libbpf-cargo/src/metadata.rs

@@ -24,7 +24,7 @@ struct PackageMetadata {
 }
 
 #[derive(Debug, Clone)]
-pub struct UnprocessedObj {
+pub(crate) struct UnprocessedObj {
     /// Package the object belongs to
     pub package: String,
     /// Path to .c
@@ -142,7 +142,10 @@ fn get_package(
 }
 
 /// Returns the `target_directory` and a list of objects to compile.
-pub fn get(debug: bool, manifest_path: Option<&PathBuf>) -> Result<(PathBuf, Vec<UnprocessedObj>)> {
+pub(crate) fn get(
+    debug: bool,
+    manifest_path: Option<&PathBuf>,
+) -> Result<(PathBuf, Vec<UnprocessedObj>)> {
     let mut cmd = MetadataCommand::new();
 
     if let Some(path) = manifest_path {