refactor: avoid std::error::Error imports

Author: xtqqczze

src/uu/chcon/src/errors.rs

@@ -86,7 +86,7 @@ impl Error {
     }
 }
 
-pub(crate) fn report_full_error(mut err: &dyn std::error::Error) -> String {
+pub(crate) fn report_full_error(mut err: &dyn core::error::Error) -> String {
     let mut desc = String::with_capacity(256);
     write!(desc, "{err}").unwrap();
     while let Some(source) = err.source() {

src/uu/cp/src/cp.rs

@@ -128,7 +128,7 @@ impl Display for BackupError {
     }
 }
 
-impl std::error::Error for BackupError {}
+impl core::error::Error for BackupError {}
 
 impl UError for CpError {
     fn code(&self) -> i32 {

src/uu/more/src/more.rs

@@ -78,7 +78,7 @@ impl std::fmt::Display for MoreError {
     }
 }
 
-impl std::error::Error for MoreError {}
+impl core::error::Error for MoreError {}
 
 const BELL: char = '\x07'; // Printing this character will ring the bell
 

src/uu/mv/src/hardlink.rs

@@ -71,8 +71,8 @@ impl std::fmt::Display for HardlinkError {
     }
 }
 
-impl std::error::Error for HardlinkError {
-    fn source(&self) -> Option<&(dyn std::error::Error + 'static)> {
+impl core::error::Error for HardlinkError {
+    fn source(&self) -> Option<&(dyn core::error::Error + 'static)> {
         match self {
             Self::Io(e) => Some(e),
             Self::Metadata { error, .. } => Some(error),

src/uu/runcon/src/errors.rs

@@ -81,7 +81,7 @@ impl Error {
     }
 }
 
-pub(crate) fn write_full_error<W>(writer: &mut W, err: &dyn std::error::Error) -> std::fmt::Result
+pub(crate) fn write_full_error<W>(writer: &mut W, err: &dyn core::error::Error) -> std::fmt::Result
 where
     W: Write,
 {
@@ -110,7 +110,7 @@ impl RunconError {
     }
 }
 
-impl std::error::Error for RunconError {}
+impl core::error::Error for RunconError {}
 impl UError for RunconError {
     fn code(&self) -> i32 {
         self.code

src/uu/split/src/number.rs

@@ -14,7 +14,6 @@
 //!
 //! [radix]: https://en.wikipedia.org/wiki/Radix
 //! [positional notation]: https://en.wikipedia.org/wiki/Positional_notation
-use std::error::Error;
 use std::fmt::{self, Display, Formatter, Write};
 use uucore::translate;
 
@@ -28,7 +27,7 @@ impl Display for Overflow {
     }
 }
 
-impl Error for Overflow {}
+impl core::error::Error for Overflow {}
 
 /// A number in arbitrary radix expressed in a positional notation.
 ///

src/uu/tr/src/operation.rs

@@ -17,7 +17,6 @@ use nom::{
 };
 use std::{
     char,
-    error::Error,
     fmt::{Debug, Display},
     io::{BufRead, Write},
 };
@@ -136,7 +135,7 @@ impl Display for BadSequence {
     }
 }
 
-impl Error for BadSequence {}
+impl core::error::Error for BadSequence {}
 impl UError for BadSequence {}
 
 #[derive(Debug, Clone, Copy)]

src/uucore/build.rs

@@ -8,7 +8,7 @@ use std::fs::File;
 use std::io::Write;
 use std::path::{Path, PathBuf};
 
-pub fn main() -> Result<(), Box<dyn std::error::Error>> {
+pub fn main() -> Result<(), Box<dyn core::error::Error>> {
     let out_dir = env::var("OUT_DIR")?;
 
     let mut embedded_file = File::create(Path::new(&out_dir).join("embedded_locales.rs"))?;
@@ -63,7 +63,7 @@ pub fn main() -> Result<(), Box<dyn std::error::Error>> {
 ///
 /// Returns an error if the `CARGO_MANIFEST_DIR` environment variable is not set
 /// or if the current directory structure does not allow determining the project root.
-fn project_root() -> Result<PathBuf, Box<dyn std::error::Error>> {
+fn project_root() -> Result<PathBuf, Box<dyn core::error::Error>> {
     let manifest_dir = env::var("CARGO_MANIFEST_DIR")?;
     let uucore_path = Path::new(&manifest_dir);
 
@@ -135,7 +135,7 @@ fn embed_single_utility_locale(
     project_root: &Path,
     util_name: &str,
     locales_to_embed: &(String, Option<String>),
-) -> Result<(), Box<dyn std::error::Error>> {
+) -> Result<(), Box<dyn core::error::Error>> {
     // Embed utility-specific locales
     embed_component_locales(embedded_file, locales_to_embed, util_name, |locale| {
         project_root
@@ -171,7 +171,7 @@ fn embed_all_utility_locales(
     embedded_file: &mut File,
     project_root: &Path,
     locales_to_embed: &(String, Option<String>),
-) -> Result<(), Box<dyn std::error::Error>> {
+) -> Result<(), Box<dyn core::error::Error>> {
     use std::fs;
 
     // Discover all uu_* directories
@@ -221,7 +221,7 @@ fn embed_all_utility_locales(
 fn embed_static_utility_locales(
     embedded_file: &mut File,
     locales_to_embed: &(String, Option<String>),
-) -> Result<(), Box<dyn std::error::Error>> {
+) -> Result<(), Box<dyn core::error::Error>> {
     use std::env;
 
     writeln!(
@@ -294,9 +294,9 @@ fn get_locales_to_embed() -> (String, Option<String>) {
 fn for_each_locale<F>(
     locales: &(String, Option<String>),
     mut f: F,
-) -> Result<(), Box<dyn std::error::Error>>
+) -> Result<(), Box<dyn core::error::Error>>
 where
-    F: FnMut(&str) -> Result<(), Box<dyn std::error::Error>>,
+    F: FnMut(&str) -> Result<(), Box<dyn core::error::Error>>,
 {
     f(&locales.0)?;
     if let Some(ref system_locale) = locales.1 {
@@ -317,7 +317,7 @@ fn embed_locale_file(
     locale_key: &str,
     locale: &str,
     component: &str,
-) -> Result<(), Box<dyn std::error::Error>> {
+) -> Result<(), Box<dyn core::error::Error>> {
     use std::fs;
 
     if locale_path.exists() || locale_path.is_file() {
@@ -359,7 +359,7 @@ fn embed_all_locales_for_component<F>(
     embedded_file: &mut File,
     component_name: &str,
     path_builder: &F,
-) -> Result<(), Box<dyn std::error::Error>>
+) -> Result<(), Box<dyn core::error::Error>>
 where
     F: Fn(&str) -> PathBuf,
 {
@@ -391,7 +391,7 @@ fn embed_component_locales<F>(
     locales: &(String, Option<String>),
     component_name: &str,
     path_builder: F,
-) -> Result<(), Box<dyn std::error::Error>>
+) -> Result<(), Box<dyn core::error::Error>>
 where
     F: Fn(&str) -> PathBuf,
 {

src/uucore/src/lib/features/backup_control.rs

@@ -88,7 +88,6 @@ use crate::{
 use clap::ArgMatches;
 use std::{
     env,
-    error::Error,
     ffi::{OsStr, OsString},
     fmt::{Debug, Display},
     path::{Path, PathBuf},
@@ -169,7 +168,7 @@ impl UError for BackupError {
     }
 }
 
-impl Error for BackupError {}
+impl core::error::Error for BackupError {}
 
 impl Display for BackupError {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {

src/uucore/src/lib/features/buf_copy/common.rs

@@ -21,7 +21,7 @@ impl std::fmt::Display for Error {
     }
 }
 
-impl std::error::Error for Error {}
+impl core::error::Error for Error {}
 
 impl UError for Error {
     fn code(&self) -> i32 {