fix(tfhe-lints): linter was not run, missing compile time env var

utils/cargo-tfhe-lints-inner/src/main.rs

@@ -27,6 +27,11 @@ fn main() {
         (tool_args.as_slice(), &[] as &[String])
     };
 
+    // The linter calls rustc without cargo, so these variables won't be set. Since we use them in
+    // our code, we need to set them to any value to avoid a compilation error.
+    std::env::set_var("CARGO_PKG_VERSION_MAJOR", "X");
+    std::env::set_var("CARGO_PKG_VERSION_MINOR", "Y");
+
     rustc_tools::cargo_integration(&cargo_args, |args| {
         let mut args = args.to_vec();
         args.extend(rustc_args.iter().skip(1).cloned());

utils/cargo-tfhe-lints/src/main.rs

@@ -1,4 +1,7 @@
-use std::process::{exit, Command};
+use std::{
+    io::{Error, ErrorKind},
+    process::{exit, Command},
+};
 
 fn get_supported_rustc_version() -> &'static str {
     const TOOLCHAIN_FILE: &str = include_str!("../../cargo-tfhe-lints-inner/rust-toolchain.toml");
@@ -26,8 +29,17 @@ fn main() {
         .arg(toolchain.as_str())
         .arg("tfhe-lints-inner")
         .args(&cargo_args)
-        .spawn()
-        .and_then(|mut child| child.wait())
+        .status()
+        .and_then(|res| {
+            if !res.success() {
+                Err(Error::new(
+                    ErrorKind::Other,
+                    format!("Inner process failed with {res}"),
+                ))
+            } else {
+                Ok(())
+            }
+        })
     {
         eprintln!(
             "Command `cargo {toolchain} tfhe-lints-inner {}` failed: {err:?}",