CloudNebulaProject/ips
1
0
Fork 0
mirror of https://codeberg.org/Toasterson/ips.git synced 2026-04-11 05:40:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove unused imports and the synchronous flag in pkg6repo, adjust comments for clarity and grammar, and refine logging statements.

Browse source
This commit is contained in:
Till Wegmueller 2025-07-27 10:42:03 +02:00
parent a3b2686b0a
commit 8760bf0c4d
No known key found for this signature in database
1 changed files with 3 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
pkg6repo/src/main.rs
View file

@ -7,7 +7,7 @@ use clap::{Parser, Subcommand};
use serde::Serialize; use serde::Serialize;
use std::convert::TryFrom; use std::convert::TryFrom;
use std::path::PathBuf; use std::path::PathBuf;
use tracing::{debug, error, info, warn}; use tracing::{debug, info};
use tracing_subscriber::fmt; use tracing_subscriber::fmt;
use libips::repository::{FileBackend, ReadableRepository, RepositoryVersion, WritableRepository}; use libips::repository::{FileBackend, ReadableRepository, RepositoryVersion, WritableRepository};
@ -81,10 +81,6 @@ enum Commands {
#[clap(short = 'n')] #[clap(short = 'n')]
dry_run: bool, dry_run: bool,
/// Wait for the operation to complete
#[clap(long)]
synchronous: bool,
/// Publishers to remove /// Publishers to remove
publisher: Vec<String>, publisher: Vec<String>,
}, },
@ -325,7 +321,7 @@ enum Commands {
} }
fn main() -> Result<()> { fn main() -> Result<()> {
// Initialize the tracing subscriber with default log level as debug and no decorations // Initialize the tracing subscriber with the default log level as debug and no decorations
fmt::Subscriber::builder() fmt::Subscriber::builder()
.with_max_level(tracing::Level::DEBUG) .with_max_level(tracing::Level::DEBUG)
.without_time() .without_time()
@ -379,14 +375,13 @@ fn main() -> Result<()> {
Commands::RemovePublisher { Commands::RemovePublisher {
repo_uri_or_path, repo_uri_or_path,
dry_run, dry_run,
synchronous,
publisher, publisher,
} => { } => {
info!( info!(
"Removing publishers {:?} from repository {}", "Removing publishers {:?} from repository {}",
publisher, repo_uri_or_path publisher, repo_uri_or_path
); );
debug!("Dry run: {}, Synchronous: {}", dry_run, synchronous); debug!("Dry run: {}", dry_run);
// Open the repository // Open the repository
let mut repo = FileBackend::open(repo_uri_or_path)?; let mut repo = FileBackend::open(repo_uri_or_path)?;
@ -397,13 +392,6 @@ fn main() -> Result<()> {
repo.remove_publisher(p, *dry_run)?; repo.remove_publisher(p, *dry_run)?;
} }
// The synchronous parameter is used to wait for the operation to complete before returning
// For FileBackend, operations are already synchronous, so this parameter doesn't have any effect
// For RestBackend, this would wait for the server to complete the operation before returning
if *synchronous {
debug!("Operation completed synchronously");
}
if *dry_run { if *dry_run {
info!("Dry run completed. No changes were made."); info!("Dry run completed. No changes were made.");
} else { } else {