Skip to content

refactor(utils): hoist utils::utils into utils #4065

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Oct 25, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/bin/rustup-init.rs
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ use rustup::env_var::RUST_RECURSION_COUNT_MAX;
use rustup::errors::RustupError;
use rustup::is_proxyable_tools;
use rustup::process::Process;
use rustup::utils::utils;
use rustup::utils;

#[tokio::main]
async fn main() -> Result<ExitCode> {
Expand Down
2 changes: 1 addition & 1 deletion src/cli/common.rs
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ use crate::{
notifications::Notification,
process::{terminalsource, Process},
toolchain::{DistributableToolchain, LocalToolchainName, Toolchain, ToolchainName},
utils::{notifications as util_notifications, notify::NotificationLevel, utils},
utils::{self, notifications as util_notifications, notify::NotificationLevel},
};

pub(crate) const WARN_COMPLETE_PROFILE: &str = "downloading with complete profile isn't recommended unless you are a developer of the rust language";
Expand Down
2 changes: 1 addition & 1 deletion src/cli/rustup_mode.rs
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ use crate::{
MaybeResolvableToolchainName, ResolvableLocalToolchainName, ResolvableToolchainName,
Toolchain, ToolchainName,
},
utils::utils::{self, ExitCode},
utils::{self, ExitCode},
};

const TOOLCHAIN_OVERRIDE_ERROR: &str =
Expand Down
2 changes: 1 addition & 1 deletion src/cli/self_update.rs
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ use crate::{
DistributableToolchain, MaybeOfficialToolchainName, ResolvableToolchainName, Toolchain,
ToolchainName,
},
utils::{utils, Notification},
utils::{self, Notification},
DUP_TOOLS, TOOLS,
};

Expand Down
3 changes: 1 addition & 2 deletions src/cli/self_update/unix.rs
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@ use tracing::{error, warn};
use super::install_bins;
use super::shell;
use crate::process::Process;
use crate::utils::utils;
use crate::utils::Notification;
use crate::utils::{self, Notification};

// If the user is trying to install with sudo, on some systems this will
// result in writing root-owned files to the user's home directory, because
Expand Down
3 changes: 1 addition & 2 deletions src/cli/self_update/windows.rs
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,7 @@ use super::{install_bins, report_error, InstallOpts};
use crate::cli::{download_tracker::DownloadTracker, markdown::md};
use crate::dist::TargetTriple;
use crate::process::{terminalsource::ColorableTerminal, Process};
use crate::utils::utils;
use crate::utils::Notification;
use crate::utils::{self, Notification};

pub(crate) fn ensure_prompt(process: &Process) -> Result<()> {
writeln!(process.stdout().lock(),)?;
Expand Down
2 changes: 1 addition & 1 deletion src/cli/setup_mode.rs
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ use crate::{
dist::Profile,
process::Process,
toolchain::MaybeOfficialToolchainName,
utils::utils,
utils,
};

/// The installer for rustup
Expand Down
2 changes: 1 addition & 1 deletion src/config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ use crate::{
CustomToolchainName, DistributableToolchain, LocalToolchainName, PathBasedToolchainName,
ResolvableLocalToolchainName, ResolvableToolchainName, Toolchain, ToolchainName,
},
utils::utils,
utils,
};

#[derive(Debug, ThisError)]
Expand Down
2 changes: 1 addition & 1 deletion src/dist/component/components.rs
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ use crate::dist::component::transaction::Transaction;
use crate::dist::prefix::InstallPrefix;
use crate::errors::RustupError;
use crate::process::Process;
use crate::utils::utils;
use crate::utils;

const COMPONENTS_FILE: &str = "components";

Expand Down
3 changes: 1 addition & 2 deletions src/dist/component/package.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ use crate::dist::component::transaction::*;
use crate::dist::temp;
use crate::errors::*;
use crate::process::Process;
use crate::utils::notifications::Notification;
use crate::utils::utils;
use crate::utils::{self, Notification};

/// The current metadata revision used by rust-installer
pub(crate) const INSTALLER_VERSION: &str = "3";
Expand Down
3 changes: 1 addition & 2 deletions src/dist/component/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@ use crate::dist::Notification;
use crate::dist::DEFAULT_DIST_SERVER;
use crate::errors::RustupError;
use crate::process::TestProcess;
use crate::utils::raw as utils_raw;
use crate::utils::utils;
use crate::utils::{self, raw as utils_raw};

#[test]
fn add_file() {
Expand Down
2 changes: 1 addition & 1 deletion src/dist/component/transaction.rs
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ use crate::dist::prefix::InstallPrefix;
use crate::dist::temp;
use crate::errors::*;
use crate::process::Process;
use crate::utils::utils;
use crate::utils;

/// A Transaction tracks changes to the file system, allowing them to
/// be rolled back in case of an error. Instead of deleting or
Expand Down
2 changes: 1 addition & 1 deletion src/dist/download.rs
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ use crate::dist::notifications::*;
use crate::dist::temp;
use crate::errors::*;
use crate::process::Process;
use crate::utils::utils;
use crate::utils;

const UPDATE_HASH_LEN: usize = 20;

Expand Down
2 changes: 1 addition & 1 deletion src/dist/manifestation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ use crate::dist::temp;
use crate::dist::{Profile, TargetTriple, DEFAULT_DIST_SERVER};
use crate::errors::RustupError;
use crate::process::Process;
use crate::utils::utils;
use crate::utils;

pub(crate) const DIST_MANIFEST: &str = "multirust-channel-manifest.toml";
pub(crate) const CONFIG_FILE: &str = "multirust-config.toml";
Expand Down
2 changes: 1 addition & 1 deletion src/dist/manifestation/tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ use crate::{
errors::RustupError,
process::TestProcess,
test::mock::{dist::*, MockComponentBuilder, MockFile, MockInstallerBuilder},
utils::{raw as utils_raw, utils},
utils::{self, raw as utils_raw},
};

const SHA256_HASH_LEN: usize = 64;
Expand Down
2 changes: 1 addition & 1 deletion src/dist/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ use crate::{
errors::RustupError,
process::Process,
toolchain::ToolchainName,
utils::utils,
utils,
};

pub mod component;
Expand Down
2 changes: 1 addition & 1 deletion src/dist/prefix.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
use std::path::{Path, PathBuf};

use crate::utils::utils;
use crate::utils;

const REL_MANIFEST_DIR: &str = "lib/rustlib";
static V1_COMMON_COMPONENT_LIST: &[&str] = &["cargo", "rustc", "rust-docs"];
Expand Down
4 changes: 1 addition & 3 deletions src/dist/temp.rs
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,7 @@ use std::path::{Path, PathBuf};
pub(crate) use anyhow::{Context as _, Result};
use thiserror::Error as ThisError;

use crate::utils::notify::NotificationLevel;
use crate::utils::raw;
use crate::utils::utils;
use crate::utils::{self, notify::NotificationLevel, raw};

#[derive(Debug, ThisError)]
pub(crate) enum CreatingError {
Expand Down
2 changes: 1 addition & 1 deletion src/fallback_settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ use anyhow::{Context, Result};
use serde::Deserialize;

#[cfg(unix)]
use crate::utils::utils;
use crate::utils;

#[derive(Clone, Debug, Deserialize, Eq, PartialEq, Default)]
pub struct FallbackSettings {
Expand Down
2 changes: 1 addition & 1 deletion src/install.rs
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ use crate::{
errors::RustupError,
notifications::Notification as RootNotification,
toolchain::{CustomToolchainName, LocalToolchainName, Toolchain},
utils::utils,
utils,
};

#[derive(Clone, Debug)]
Expand Down
2 changes: 1 addition & 1 deletion src/settings.rs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ use crate::cli::self_update::SelfUpdateMode;
use crate::dist::Profile;
use crate::errors::*;
use crate::notifications::*;
use crate::utils::utils;
use crate::utils;

#[derive(Clone, Debug, Eq, PartialEq)]
pub struct SettingsFile {
Expand Down
2 changes: 1 addition & 1 deletion src/test/mock/clitools.rs
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ use crate::process;
use crate::test as rustup_test;
use crate::test::const_dist_dir;
use crate::test::this_host_triple;
use crate::utils::utils;
use crate::utils;

use super::{
dist::{
Expand Down
2 changes: 1 addition & 1 deletion src/toolchain.rs
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ use crate::{
dist::PartialToolchainDesc,
env_var, install,
notifications::Notification,
utils::{raw::open_dir_following_links, utils},
utils::{self, raw::open_dir_following_links},
RustupError,
};

Expand Down
Loading
Loading