Mpult warning parse (#33)

* Added rudementary warning/error parsing

* move parsing code to programs where it belongs

* Now parsing errors that cross two lines

* fix bug where clean sometimes fails

* Merge something, idk

* Improve info module (#15)

* Version checker (#18)

* Added version checker

* fix bug that makes in not work

* remove itex-version-info.toml from list

* merge

* improve tex_filename (#20)

* update deps and change default build_artifacts path to itex-build

* change version to 1.4.0-dev

* Merge better errors (#27)

* Merge global ITex settings (#29)

* Add the safe-build command (#30)

* fix some bugs

* Merge release infra (#19)

* fix formating

* fix an oopsie

* windows env var fix

* merge

* merge

* use RUST_LOG when in a debug build

* fix debug printing and triple printing

* fix formating

---------

Co-authored-by: Maximilian Pult <it@optik-riede.de>

src/build.rs

@@ -1,5 +1,6 @@
 use crate::clean::clean_build_artifacts_folder;
 use crate::prelude::*;
+
 use std::io::{stdout, Write};
 
 pub fn build(debug: bool, draft_mode: bool, settings: Option<Settings>) {
@@ -15,12 +16,12 @@ pub fn build(debug: bool, draft_mode: bool, settings: Option<Settings>) {
     let pdflatex = PDFLatex::from_settings(settings.clone());
     let bibtex = Bibtex::from_settings(settings.clone());
 
-    pdflatex.run();
-    let bibtex_output = bibtex.run();
-    pdflatex.run();
-    let pdflatex_output = pdflatex.run();
+    pdflatex.run(false);
+    let bibtex_output = bibtex.run(true);
+    pdflatex.run(false);
+    let pdflatex_output = pdflatex.run(true);
 
-    if debug || settings.debug() || !pdflatex_output.status.success() {
+    if debug || settings.debug() {
         println!("{}", console::style("--- PDFLatex Output ---").blue().bold());
         stdout().write_all(&pdflatex_output.stdout).unwrap();
         println!("{}", console::style("--- Bibtex Output ---").blue().bold());

src/count.rs

@@ -8,7 +8,7 @@ pub fn count() {
 
     let texcount = Texcount::from_settings(settings);
 
-    let output = texcount.run();
+    let output = texcount.run(true);
 
     std::io::stdout().write_all(&output.stdout).unwrap();
 }

src/programs/bibtex.rs

@@ -30,7 +30,7 @@ impl Executable for Bibtex {
         }
     }
 
-    fn run(&self) -> std::process::Output {
+    fn run(&self, _print_errors: bool) -> std::process::Output {
         let output = std::process::Command::new(self.exe_path.clone()).args(self.args.clone()).output();
 
         if output.is_err() {

src/programs/mod.rs

@@ -14,6 +14,6 @@ use std::path::PathBuf;
 
 pub trait Executable {
     fn from_settings(settings: crate::Settings) -> Self;
-    fn run(&self) -> std::process::Output;
+    fn run(&self, print_error: bool) -> std::process::Output;
     fn set_executable_path(&mut self, path: PathBuf);
 }

src/programs/pdflatex.rs

@@ -33,7 +33,7 @@ impl Executable for PDFLatex {
         }
     }
 
-    fn run(&self) -> std::process::Output {
+    fn run(&self, print_errors: bool) -> std::process::Output {
         let output = std::process::Command::new(self.exe_path.clone()).args(self.args.clone()).output();
 
         if output.is_err() {
@@ -45,6 +45,10 @@ impl Executable for PDFLatex {
 
         let output = unwrap_result!(output, "Failed to read output of pdflatex");
 
+        if print_errors {
+            Self::check_error(&output);
+        }
+
         output
     }
 
@@ -63,3 +67,34 @@ impl Executable for PDFLatex {
         }
     }
 }
+
+impl PDFLatex {
+    fn check_error(output: &std::process::Output) {
+        let stdout = std::str::from_utf8(&output.stdout);
+        if stdout.is_err() {
+            return;
+        }
+        let stdout = stdout.unwrap().to_lowercase();
+        let mut buffer: &str = "";
+
+        for line in stdout.lines() {
+            // check buffer to see if this iteration is a continuation of last error line
+            if !buffer.is_empty() {
+                // This is the continuation
+                println!("{}{}", style(buffer).yellow().bold(), style(line).yellow().bold());
+                buffer = "";
+            } else {
+                // This is new
+                if line.to_ascii_lowercase().contains("warning") || line.to_ascii_lowercase().contains("error") {
+                    buffer = line;
+                }
+            }
+        }
+
+        let stderr = std::str::from_utf8(&output.stderr);
+        if stderr.is_err() {
+            return;
+        }
+        let stderr = stderr.unwrap();
+    }
+}

src/programs/texcount.rs

@@ -26,7 +26,7 @@ impl Executable for Texcount {
         }
     }
 
-    fn run(&self) -> std::process::Output {
+    fn run(&self, _print_errors: bool) -> std::process::Output {
         let output = std::process::Command::new(self.exe_path.clone()).args(self.args.clone()).output();
 
         if output.is_err() {