test

ci: build: remove redundant source

.gitea/workflows/gentoo-utils.yml

@@ -180,3 +180,16 @@ jobs:
         run: |
           meson setup -Ddocs=enabled docs
           ninja rustdoc -C docs
+
+  grep:
+    runs-on: brutalisk
+    needs: [build-oci-image]
+    container:
+      image: ${{ vars.REGISTRY_URL }}/${{ gitea.repository }}:${{ needs.build-oci-image.outputs.image_tag }}
+    steps:
+      - name: Checkout repo
+        uses: actions/checkout@v5
+
+      - name: grep for patterns
+        run: |
+          git grep -E 'todo!|dbg!' -- '*.rs' && exit 1 || exit 0

check_commands.txt

@@ -3,3 +3,4 @@ ninja rustfmt -C build
 ninja rustdoc -C build
 ninja clippy -C build
 ninja test -C build
+git grep -E 'todo!|dbg!' -- '*.rs' && exit 1 || exit 0

fuzz/atom/parser/fuzz.rs

@@ -4,25 +4,12 @@
 use core::slice;
 use gentoo_utils::{Parseable, atom::Atom};
 use std::{
+    env::{self},
     io::{self, Write},
-    sync::{LazyLock, Mutex},
 };
 
-#[derive(Debug)]
-struct State {
-    input: io::Stdin,
-    output: io::Stdout,
-}
-
 #[unsafe(no_mangle)]
 pub unsafe extern "C" fn LLVMFuzzerTestOneInput(input: *const u8, len: usize) -> i32 {
-    static PIPES: LazyLock<Mutex<State>> = LazyLock::new(|| {
-        Mutex::new(State {
-            input: io::stdin(),
-            output: io::stdout(),
-        })
-    });
-
     let slice = unsafe { slice::from_raw_parts(input, len) };
     let str = str::from_utf8(slice).expect("expected ascii input");
 
@@ -30,13 +17,13 @@ pub unsafe extern "C" fn LLVMFuzzerTestOneInput(input: *const u8, len: usize) ->
         return -1;
     }
 
-    let mut state = PIPES.lock().unwrap();
-
-    writeln!(&mut state.output, "{str}").unwrap();
-
+    let stdin = io::stdin();
+    let mut stdout = io::stdout();
     let mut buffer = String::new();
 
-    state.input.read_line(&mut buffer).unwrap();
+    writeln!(&mut stdout, "{str}").unwrap();
+
+    stdin.read_line(&mut buffer).unwrap();
 
     let control = match buffer.as_str().trim() {
         "0" => Ok(()),
@@ -48,23 +35,32 @@ pub unsafe extern "C" fn LLVMFuzzerTestOneInput(input: *const u8, len: usize) ->
 
     match (control, gentoo_utils) {
         (Ok(_), Ok(_)) => {
-            eprintln!("agreement that {str} is valid");
+            if env::var("FUZZER_LOG").is_ok() {
+                eprintln!("agreement that {str} is valid");
+            }
         }
         (Err(_), Err(_)) => {
-            eprintln!("agreement that {str} is invalid");
+            if env::var("FUZZER_LOG").is_ok() {
+                eprintln!("agreement that {str} is invalid");
+            }
         }
         (Ok(_), Err(rest)) => {
             panic!("disagreement on {str}\ncontrol:Ok\ngentoo-utils:Err({rest})");
         }
         (Err(_), Ok(atom))
-            if atom
-                .usedeps()
-                .iter()
-                .any(|usedep| atom.usedeps().iter().filter(|u| usedep == *u).count() > 1) =>
+            if atom.usedeps().iter().any(|usedep| {
+                atom.usedeps()
+                    .iter()
+                    .filter(|u| *usedep.flag() == *u.flag())
+                    .count()
+                    > 1
+            }) =>
         {
-            eprintln!(
-                "disagreement, but we will allow it since its probably because of duplicated usdeps"
-            );
+            if env::var("FUZZER_LOG").is_ok() {
+                eprintln!(
+                    "disagreement, but we will allow it since its probably because of duplicated usdeps"
+                );
+            }
         }
         (Err(_), Ok(_)) => {
             panic!("disagreement on {str}\ncontrol:Err\ngentoo-utils:Ok")

scripts/atom.py

@@ -2,12 +2,13 @@
 
 import sys
 from portage.dep import Atom
+from portage.exception import InvalidAtom
 
 for line in sys.stdin.buffer:
     try:
         Atom(line.decode().strip())
         sys.stdout.buffer.write(b"0\n")
-    except:
+    except InvalidAtom:
         sys.stdout.buffer.write(b"1\n")
     finally:
         sys.stdout.buffer.flush()

src/repo/mod.rs

@@ -102,7 +102,7 @@ impl Iterator for Ebuilds {
                     Ok(ebuild) => break Some(Ok(ebuild)),
                     Err(e) => break Some(Err(e)),
                 },
-                _ => todo!(),
+                Err(e) => break Some(Err(Error::ReadDir(self.0.clone(), e))),
             }
         }
     }