.gitea/workflows/cargo.yaml

@@ -21,6 +21,8 @@ jobs:
         run: |
           rustup component add rustfmt
           rustup component add clippy
+      - name: install python
+        run: apt update && apt install libpython3-dev -y
       - name: config custom registry
         run: |
           mkdir -p ~/.cargo/
@@ -43,4 +45,4 @@ jobs:
         with:
           # Optional. Commit message for the created commit.
           # Defaults to "Apply automatic changes"
-          commit_message: "ci: automatic cargo CI changes"
+          commit_message: automatic cargo CI changes

.github/dependabot.yaml

@@ -1,7 +0,0 @@
-version: 2
-updates:
-  - package-ecosystem: "github-actions"
-    directory: "/"
-    # Check for updates every Monday
-    schedule:
-      interval: "weekly"

.github/workflows/cargo.yaml

@@ -1,16 +1,10 @@
-name: Rust CI
+name: cargo devel CI
 on:
-  pull_request:
-    branches:
-      - '**'
   push:
     branches:
       - '**'
       # - '!master'
 
-env:
-  CARGO_TERM_COLOR: always
-
 jobs:
   CI:
     runs-on: ubuntu-latest
@@ -48,10 +42,10 @@ jobs:
       - name: cargo fmt
         run: cargo fmt --all
       - name: cargo test
-        run: cargo test --all-features --all-targets --workspace
+        run: cargo test --all-features --all-targets --workspace && cargo test --all-features --workspace --doc
       - name: commit back to repository
         uses: stefanzweifel/git-auto-commit-action@v5
         with:
           # Optional. Commit message for the created commit.
           # Defaults to "Apply automatic changes"
-          commit_message: "ci: automatic Rust CI changes"
+          commit_message: automatic cargo CI changes

.github/workflows/release.yaml

@@ -1,54 +0,0 @@
-name: Release-plz
-
-permissions:
-  pull-requests: write
-  contents: write
-
-on:
-  push:
-    branches:
-      - master
-      - main
-
-jobs:
-
-  # Release unpublished packages.
-  release-plz-release:
-    name: Release-plz release
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-      - name: Install Rust toolchain
-        uses: dtolnay/rust-toolchain@stable
-      - name: Run release-plz
-        uses: MarcoIeni/release-plz-action@v0.5
-        with:
-          command: release
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          CARGO_REGISTRY_TOKEN: ${{ secrets.CARGO_REGISTRY_TOKEN }}
-
-  # Create a PR with the new versions and changelog, preparing the next release.
-  release-plz-pr:
-    name: Release-plz PR
-    runs-on: ubuntu-latest
-    concurrency:
-      group: release-plz-${{ github.ref }}
-      cancel-in-progress: false
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-      - name: Install Rust toolchain
-        uses: dtolnay/rust-toolchain@stable
-      - name: Run release-plz
-        uses: MarcoIeni/release-plz-action@v0.5
-        with:
-          command: release-pr
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          CARGO_REGISTRY_TOKEN: ${{ secrets.CARGO_REGISTRY_TOKEN }}

Cargo.toml

@@ -5,7 +5,7 @@ default-members = [".", "members/libpt-core"]
 
 [workspace.package]
 publish = true
-version = "0.7.3-alpha.2"
+version = "0.6.0"
 edition = "2021"
 authors = ["Christoph J. Scherr <software@cscherr.de>"]
 license = "GPL-3.0-or-later"
@@ -19,10 +19,10 @@ categories = ["command-line-utilities", "development-tools"]
 [workspace.dependencies]
 anyhow = "1.0.79"
 thiserror = "1.0.56"
-libpt-core = { version = "0.5.0", path = "members/libpt-core" }
+libpt-core = { version = "0.4.0", path = "members/libpt-core" }
 libpt-bintols = { version = "0.5.1", path = "members/libpt-bintols" }
-libpt-log = { version = "0.6.2-alpha.1", path = "members/libpt-log" }
+libpt-log = { version = "0.5.1", path = "members/libpt-log" }
-libpt-cli = { version = "0.2.2-alpha.1", path = "members/libpt-cli" }
+libpt-cli = { version = "0.1.2", path = "members/libpt-cli" }
 
 [package]
 name = "libpt"
@@ -41,9 +41,8 @@ categories.workspace = true
 [features]
 default = ["log", "core"]
 core = []
-full = ["default", "core", "log", "bintols", "libpt-cli/full"]
+full = ["default", "core", "log", "bintols"]
 log = ["dep:libpt-log"]
-log-crate = ["libpt-cli/log"]
 bintols = ["dep:libpt-bintols", "log"]
 cli = ["dep:libpt-cli", "core", "log"]
 
@@ -59,8 +58,4 @@ crate-type = [
 libpt-core = { workspace = true }
 libpt-bintols = { workspace = true, optional = true }
 libpt-log = { workspace = true, optional = true }
-libpt-cli = { workspace = true, optional = true, features = ["log"] }
+libpt-cli = { workspace = true, optional = true }
-
-[package.metadata.docs.rs]
-all-features = true
-rustdoc-args = ["--cfg", "docsrs"]

README.md

@@ -40,19 +40,12 @@ If you want to use the python variant too, you need to compile with maturing.
 
 ## Installing from [pypi](https://pypi.org)
 
-The Python interface of `libpt` is currently not implemented, but it is planned
+`libpt` has been packaged for [pypi.org](https://pypi.org/project/libpt/).
-to eventually re add it. Meanwhile, you can use a much older version if you
-really want.
 
-> :warning: **This will install a very old version**
+You can install it with `pip install libpt`
->
-> `libpt` has been packaged for [pypi.org](https://pypi.org/project/libpt/).
->
-> You can install it with `pip install libpt`
 
 ## Installing from [crates.io](https://crates.io)
 
-
 `libpt` has been packaged for [crates.io](https://crates.io/crates/libpt).
 
 You can add the library to your project with `cargo add libpt`.

members/libpt-bintols/Cargo.toml

@@ -20,7 +20,3 @@ libpt-core = { workspace = true }
 libpt-log = { workspace = true }
 anyhow = { workspace = true }
 thiserror = { workspace = true }
-
-[package.metadata.docs.rs]
-all-features = true
-rustdoc-args = ["--cfg", "docsrs"]

members/libpt-cli/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "libpt-cli"
 publish.workspace = true
-version = "0.2.2-alpha.2"
+version = "0.1.2"
 edition.workspace = true
 authors.workspace = true
 license.workspace = true
@@ -12,9 +12,11 @@ repository.workspace = true
 keywords.workspace = true
 categories.workspace = true
 
+[package.metadata.docs.rs]
+cargo-args = ["-Zunstable-options", "-Zrustdoc-scrape-examples"]
+
 [features]
 default = []
-full = ["log"]
 log = ["dep:log"]
 
 [dependencies]
@@ -24,14 +26,11 @@ comfy-table = "7.1.1"
 console = "0.15.8"
 dialoguer = { version = "0.11.0", features = ["completion", "history"] }
 embed-doc-image = "0.1.4"
+exitcode = "1.1.2"
+human-panic = "2.0.0"
 indicatif = "0.17.8"
 libpt-log = { workspace = true, optional = false }
 log = { version = "0.4.21", optional = true }
-serde = { version = "1.0.209", features = ["derive"] }
 shlex = "1.3.0"
 strum = { version = "0.26.3", features = ["derive"] }
 thiserror.workspace = true
-
-[package.metadata.docs.rs]
-all-features = true
-rustdoc-args = ["--cfg", "docsrs"]

members/libpt-cli/examples/cli.rs

@@ -1,6 +1,6 @@
 use clap::Parser;
 use libpt_cli::args::VerbosityLevel;
-use libpt_cli::printing;
+use libpt_cli::{clap, printing};
 use libpt_log::{debug, Logger};
 
 /// This is the help

members/libpt-cli/examples/repl.rs

@@ -1,7 +1,7 @@
 use console::style;
-use libpt_cli::printing;
 use libpt_cli::repl::{DefaultRepl, Repl};
-use libpt_log::{debug, Logger};
+use libpt_cli::{clap, printing, strum};
+use libpt_log::{debug, Level, Logger};
 
 use clap::Subcommand;
 use strum::EnumIter;
@@ -73,7 +73,7 @@ fn main() -> anyhow::Result<()> {
                 if !fancy {
                     println!("{}", text.join(" "))
                 } else {
-                    printing::blockprint(text.join(" "), console::Color::Cyan)
+                    printing::blockprint(&text.join(" "), console::Color::Cyan)
                 }
             }
         }

members/libpt-cli/src/args.rs

@@ -4,7 +4,6 @@ use clap::Parser;
 use libpt_log::Level;
 #[cfg(feature = "log")]
 use log;
-use serde::{Deserialize, Serialize};
 
 /// Custom help template for displaying command-line usage information
 ///
@@ -19,7 +18,7 @@ use serde::{Deserialize, Serialize};
 /// # use libpt_cli::args::HELP_TEMPLATE;
 /// use clap::Parser;
 /// #[derive(Parser, Debug, Clone, PartialEq, Eq, Hash)]
-/// #[command(help_template = HELP_TEMPLATE, author, version)]
+/// #[command(help_template = HELP_TEMPLATE)]
 /// pub struct MyArgs {
 ///     /// show more details
 ///     #[arg(short, long)]
@@ -78,9 +77,9 @@ Author: {author-with-newline}
 /// Get the loglevel like this:
 ///
 /// ```no_run
-/// use libpt_cli::args::VerbosityLevel;
+/// # use libpt_cli::args::VerbosityLevel;
 /// use libpt_log::Level;
-/// use clap::Parser;
+/// # use clap::Parser;
 ///
 /// # #[derive(Parser, Debug)]
 /// # pub struct Opts {
@@ -96,7 +95,7 @@ Author: {author-with-newline}
 ///     let level: Level = opts.verbose.level();
 /// }
 /// ```
-#[derive(Parser, Clone, PartialEq, Eq, Hash, Deserialize, Serialize)]
+#[derive(Parser, Clone, PartialEq, Eq, Hash)]
 pub struct VerbosityLevel {
     /// make the output more verbose
     #[arg(

members/libpt-cli/src/lib.rs

@@ -4,3 +4,13 @@
 pub mod args;
 pub mod printing;
 pub mod repl;
+
+pub use clap;
+pub use comfy_table;
+pub use console;
+pub use dialoguer;
+pub use exitcode;
+pub use human_panic;
+pub use indicatif;
+pub use shlex;
+pub use strum;

members/libpt-cli/src/printing.rs

@@ -42,7 +42,7 @@ use console::{style, Color};
 /// # Example
 ///
 /// ```
-/// use console::Color;
+/// use libpt_cli::console::Color;
 /// use libpt_cli::printing::blockprint;
 /// # fn main() {
 /// blockprint("Hello world!", Color::Blue);
@@ -64,7 +64,7 @@ pub fn blockprint(content: impl ToString, color: Color) {
 /// # Example
 ///
 /// ```
-/// use console::Color;
+/// use libpt_cli::console::Color;
 /// use libpt_cli::printing::blockfmt;
 /// # fn main() {
 /// let formatted_content = blockfmt("Hello world!", Color::Blue);
@@ -93,8 +93,8 @@ pub fn blockfmt(content: impl ToString, color: Color) -> String {
 ///
 /// # Example
 /// ```
-/// use comfy_table::{presets, CellAlignment, ContentArrangement};
+/// use libpt_cli::comfy_table::{presets, CellAlignment, ContentArrangement};
-/// use console::Color;
+/// use libpt_cli::console::Color;
 /// use libpt_cli::printing::blockfmt_advanced;
 /// # fn main() {
 /// println!(

members/libpt-cli/src/repl/default.rs

@@ -27,8 +27,8 @@ use libpt_log::trace;
 ///
 /// ```no_run
 /// use libpt_cli::repl::{DefaultRepl, Repl};
-/// use clap::Subcommand;
+/// use libpt_cli::clap::Subcommand;
-/// use strum::EnumIter;
+/// use libpt_cli::strum::EnumIter;
 ///
 /// #[derive(Subcommand, Debug, EnumIter, Clone)]
 /// enum ReplCommand {

members/libpt-core/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "libpt-core"
 publish.workspace = true
-version = "0.5.0"
+version = "0.4.0"
 edition.workspace = true
 authors.workspace = true
 license.workspace = true
@@ -14,9 +14,6 @@ categories.workspace = true
 
 [dependencies]
 anyhow = "1.0.79"
+libpt-log = { workspace = true }
 
 [dev-dependencies]
-
-[package.metadata.docs.rs]
-all-features = true
-rustdoc-args = ["--cfg", "docsrs"]

members/libpt-core/src/lib.rs

@@ -8,16 +8,3 @@
 
 /// macros to make things faster in your code
 pub mod macros;
-
-/// ## Get the name of the crate that uses your library
-///
-/// Let's say you're writing the library `foo` and need the name of the crate that uses `foo`. With
-/// this function, you can get the name of the crate that uses `foo`.
-///
-/// Will return [None] if [`std::env::current_exe()`] errors or if conversion to [String] from [std::ffi::OsStr] fails.
-pub fn get_crate_name() -> Option<String> {
-    if let Ok(exe) = std::env::current_exe() {
-        return Some(exe.file_stem()?.to_str()?.to_string());
-    }
-    None
-}

members/libpt-log/Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "libpt-log"
 publish.workspace = true
-version = "0.6.2-alpha.2"
+version = "0.5.1"
 edition.workspace = true
 authors.workspace = true
 license.workspace = true
@@ -13,17 +13,11 @@ keywords.workspace = true
 categories.workspace = true
 
 [dependencies]
-tracing = "0.1.40"
+tracing = "0.1.37"
-tracing-appender = "0.2.3"
+tracing-appender = "0.2.2"
-tracing-subscriber = { version = "0.3.18", features = ["env-filter"] }
+tracing-subscriber = "0.3.17"
 anyhow = { workspace = true }
 thiserror = { workspace = true }
-libpt-core = { workspace = true, optional = false }
-chrono = "0.4.38"
 
 [dev-dependencies]
 gag = "1.0.0"
-
-[package.metadata.docs.rs]
-all-features = true
-rustdoc-args = ["--cfg", "docsrs"]

members/libpt-log/src/error.rs

@@ -17,9 +17,7 @@ pub enum Error {
     /// Could not assign logger as the global default
     #[error("Could not assign logger as global default")]
     SetGlobalDefaultFail(#[from] SetGlobalDefaultError),
-    /// any other error type, wrapped in [`anyhow::Error`]
+    /// any other error type, wrapped in [anyhow::Error](anyhow::Error)
     #[error(transparent)]
     Other(#[from] anyhow::Error),
-    #[error("Tried to open the logfile, but logging to file was not requested")]
-    LogfileButNoFilelog,
 }

members/libpt-log/src/lib.rs

@@ -18,7 +18,7 @@
 #![warn(clippy::pedantic, clippy::style, clippy::nursery)]
 
 use std::{
-    fmt::{self, Debug},
+    fmt,
     path::PathBuf,
     sync::atomic::{AtomicBool, Ordering},
 };
@@ -31,9 +31,8 @@ use error::Error;
 /// I'm just repackaging it a little to make it more ergonomic
 pub use tracing;
 pub use tracing::{debug, error, info, trace, warn, Level};
-use tracing_subscriber::{
+use tracing_appender::{self};
-    fmt::format::FmtSpan, layer::SubscriberExt as _, util::SubscriberInitExt, Layer,
+use tracing_subscriber::fmt::{format::FmtSpan, time};
-};
 
 use anyhow::{bail, Result};
 /// The log level used when none is specified
@@ -62,7 +61,7 @@ static INITIALIZED: AtomicBool = AtomicBool::new(false);
 /// # }
 ///
 /// ```
-#[derive(PartialEq, Eq, PartialOrd, Ord, Debug)]
+#[derive(PartialEq, Eq, PartialOrd, Ord, Hash, Debug)]
 #[allow(clippy::struct_excessive_bools)] // it's just true/false values, not states, and I don't
                                          // need to reinvent the wheel
 pub struct LoggerBuilder {
@@ -95,8 +94,6 @@ pub struct LoggerBuilder {
     show_time: bool,
     /// show timestamps as uptime (duration since the logger was initialized)
     uptime: bool,
-    /// log when span things happen
-    span_events: FmtSpan,
 }
 
 impl LoggerBuilder {
@@ -131,72 +128,100 @@ impl LoggerBuilder {
     /// This function will return an error if a global Logger was aready initialized. This module
     /// uses the [tracing] crate for logging, so if a [tracing] logger is initialized elsewhere,
     /// this method will error.
-    #[allow(clippy::missing_panics_doc)]
     pub fn build(self) -> Result<Logger> {
         // only init if no init has been performed yet
         if INITIALIZED.load(Ordering::Relaxed) {
             warn!("trying to reinitialize the logger, ignoring");
             bail!(Error::Usage("logging is already initialized".to_string()));
         }
-        let layer = tracing_subscriber::fmt::layer()
+        let subscriber = tracing_subscriber::fmt::Subscriber::builder()
+            .with_level(self.display_level)
+            .with_max_level(self.max_level)
             .with_ansi(self.ansi)
             .with_target(self.display_target)
             .with_file(self.display_filename)
             .with_thread_ids(self.display_thread_ids)
             .with_line_number(self.display_line_number)
             .with_thread_names(self.display_thread_names)
-            .with_span_events(self.span_events.clone())
+            .with_span_events(FmtSpan::FULL);
-            .with_filter(tracing::level_filters::LevelFilter::from_level(
+        // HACK: somehow find a better solution for this
-                self.max_level,
+        // I know this is hacky, but I couldn't get it any other way. I couldn't even find a
-            ));
+        // project that could do it any other way. You can't apply one after another, because the
-        if self.log_to_file {
+        // type is changed every time. When using `Box<dyn Whatever>`, some methods complain about
-            tracing_subscriber::registry()
+        // not being in trait bounds.
-                .with(layer.and_then({
+        match (self.log_to_file, self.show_time, self.pretty, self.uptime) {
-                    tracing_subscriber::fmt::layer()
+            (true, true, true, true) => {
-                        .with_writer(
+                let subscriber = subscriber
-                            self.logfile()?
+                    .with_writer(new_file_appender(self.log_dir))
-                                .expect("logging to file is requested but logfile returned None"),
+                    .with_timer(time::uptime())
-                        )
+                    .pretty()
-                        .with_ansi(self.ansi)
+                    .finish();
-                        .with_target(self.display_target)
+                tracing::subscriber::set_global_default(subscriber)?;
-                        .with_file(self.display_filename)
+            }
-                        .with_thread_ids(self.display_thread_ids)
+            (true, true, true, false) => {
-                        .with_line_number(self.display_line_number)
+                let subscriber = subscriber
-                        .with_thread_names(self.display_thread_names)
+                    .with_writer(new_file_appender(self.log_dir))
-                        .with_span_events(self.span_events.clone())
+                    .pretty()
-                        .with_filter(tracing::level_filters::LevelFilter::from_level(
+                    .finish();
-                            self.max_level,
+                tracing::subscriber::set_global_default(subscriber)?;
-                        ))
+            }
-                }))
+            (true, false, true, _) => {
-                .init();
+                let subscriber = subscriber
-        } else {
+                    .with_writer(new_file_appender(self.log_dir))
-            tracing_subscriber::registry().with(layer).init();
+                    .without_time()
+                    .pretty()
+                    .finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (true, true, false, true) => {
+                let subscriber = subscriber
+                    .with_writer(new_file_appender(self.log_dir))
+                    .with_timer(time::uptime())
+                    .finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (true, true, false, false) => {
+                let subscriber = subscriber
+                    .with_writer(new_file_appender(self.log_dir))
+                    .finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (true, false, false, _) => {
+                let subscriber = subscriber
+                    .with_writer(new_file_appender(self.log_dir))
+                    .without_time()
+                    .finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (false, true, true, true) => {
+                let subscriber = subscriber.pretty().with_timer(time::uptime()).finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (false, true, true, false) => {
+                let subscriber = subscriber.pretty().with_timer(time::uptime()).finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (false, false, true, _) => {
+                let subscriber = subscriber.without_time().pretty().finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (false, true, false, true) => {
+                let subscriber = subscriber.with_timer(time::uptime()).finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (false, true, false, false) => {
+                let subscriber = subscriber.finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
+            (false, false, false, _) => {
+                let subscriber = subscriber.without_time().finish();
+                tracing::subscriber::set_global_default(subscriber)?;
+            }
         }
-
         INITIALIZED.store(true, Ordering::Relaxed);
         Ok(Logger {})
     }
 
-    /// Opens a new file for logging to.
-    ///
-    /// Format: `{log_dir}/{consumer_name}_2024-09-01.log`
-    ///
-    /// Will be none if [`Self::log_to_file`] is [false].
-    fn logfile(&self) -> Result<Option<std::fs::File>> {
-        if !self.log_to_file {
-            return Err(Error::LogfileButNoFilelog.into());
-        }
-        let mut path = self.log_dir.clone();
-        std::fs::create_dir_all(&path)?;
-        path.push(format!(
-            "{}_{}.log",
-            libpt_core::get_crate_name().unwrap_or_else(|| "logfile".to_string()),
-            chrono::Local::now().date_naive()
-        ));
-        let file = std::fs::File::create(path)?;
-        Ok(Some(file))
-    }
-
     /// enable or disable logging to and creating of logfiles
     ///
     /// If you want to log to a file, don't forget to set [`Self::log_dir`]!
@@ -322,15 +347,6 @@ impl LoggerBuilder {
         self.max_level = max_level;
         self
     }
-
-    /// set how span events are handled
-    ///
-    /// Default: [`FmtSpan::NONE`]
-    #[must_use]
-    pub const fn span_events(mut self, span_events: FmtSpan) -> Self {
-        self.span_events = span_events;
-        self
-    }
 }
 
 impl Default for LoggerBuilder {
@@ -349,12 +365,11 @@ impl Default for LoggerBuilder {
             pretty: false,
             show_time: true,
             uptime: false,
-            span_events: FmtSpan::NONE,
         }
     }
 }
 
-/// ## Logger for `libpt`
+/// ## Logger for [`pt`](libpt)
 ///
 /// A logger is generally a functionality that let's you write information from your library or
 /// application in a more structured manner than if you just wrote all information to `stdout` or
@@ -463,3 +478,7 @@ impl Default for Logger {
             .expect("building a Logger failed")
     }
 }
+
+fn new_file_appender(log_dir: PathBuf) -> tracing_appender::rolling::RollingFileAppender {
+    tracing_appender::rolling::daily(log_dir, format!("{}.log", env!("CARGO_CRATE_NAME")))
+}

scripts/publish.sh

@@ -1,6 +1,6 @@
 #!/bin/bash
 set -e
-cargo check --all-features --workspace
+cargo check --all-features
 echo ">>>>>>>> SELECT A NEW VERSION"
 cargo ws version
 NEW_VERSION=$(cat Cargo.toml | rg '^\s*version\s*=\s*"([^"]*)"\s*$' -or '$1')