Modules refactor

src/main.rs

@@ -1,34 +1,18 @@
-#[cfg(target_os = "freebsd")]
-mod freebsd;
-#[cfg(target_os = "linux")]
-mod linux;
-#[cfg(target_os = "macos")]
-mod macos;
-#[cfg(unix)]
-mod tmux;
-#[cfg(unix)]
-mod unix;
-#[cfg(target_os = "windows")]
-mod windows;
-
 mod config;
 mod ctrlc;
 mod error;
 mod executor;
-mod generic;
-mod git;
-mod node;
 mod report;
 #[cfg(feature = "self-update")]
 mod self_update;
+mod steps;
 mod terminal;
 mod utils;
-mod vim;
 
 use self::config::Config;
 use self::error::{Error, ErrorKind};
-use self::git::{Git, Repositories};
 use self::report::Report;
+use self::steps::*;
 use self::terminal::*;
 use failure::{Fail, ResultExt};
 use std::borrow::Cow;
@@ -78,8 +62,8 @@ fn run() -> Result<(), Error> {
     env_logger::init();
 
     let base_dirs = directories::BaseDirs::new().ok_or(ErrorKind::NoBaseDirectories)?;
-    let git = Git::new();
-    let mut git_repos = Repositories::new(&git);
+    let git = git::Git::new();
+    let mut git_repos = git::Repositories::new(&git);
 
     let config = Config::read(&base_dirs)?;
     let mut report = Report::new();

src/steps/generic.rs

@@ -1,7 +1,7 @@
-use super::error::{Error, ErrorKind};
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::{self, Check, PathExt};
+use crate::error::{Error, ErrorKind};
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::{self, Check, PathExt};
 use directories::BaseDirs;
 use failure::ResultExt;
 use std::path::PathBuf;

src/steps/git.rs

@@ -1,7 +1,7 @@
-use super::error::Error;
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::{which, Check};
+use crate::error::Error;
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::{which, Check};
 use log::{debug, error};
 use std::collections::HashSet;
 use std::io;

src/steps/mod.rs

@@ -0,0 +1,9 @@
+pub mod generic;
+pub mod git;
+pub mod node;
+pub mod os;
+#[cfg(unix)]
+pub mod tmux;
+pub mod vim;
+
+pub use self::os::*;

src/steps/node.rs

@@ -1,7 +1,7 @@
-use super::error::{Error, ErrorKind};
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::{which, Check, PathExt};
+use crate::error::{Error, ErrorKind};
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::{which, Check, PathExt};
 use directories::BaseDirs;
 use failure::ResultExt;
 use std::path::PathBuf;

src/steps/os/freebsd.rs

@@ -1,7 +1,7 @@
-use super::error::{Error, ErrorKind};
-use super::executor::Executor;
-use super::terminal::{print_separator, print_warning};
-use super::utils::Check;
+use crate::error::{Error, ErrorKind};
+use crate::executor::Executor;
+use crate::terminal::{print_separator, print_warning};
+use crate::utils::Check;
 use failure::ResultExt;
 use std::path::PathBuf;
 use std::process::Command;

src/steps/os/linux.rs

@@ -1,7 +1,7 @@
-use super::error::{Error, ErrorKind};
-use super::executor::Executor;
-use super::terminal::{print_separator, print_warning};
-use super::utils::{which, Check};
+use crate::error::{Error, ErrorKind};
+use crate::executor::Executor;
+use crate::terminal::{print_separator, print_warning};
+use crate::utils::{which, Check};
 use failure::ResultExt;
 use std::fs;
 use std::path::PathBuf;

src/steps/os/macos.rs

@@ -1,7 +1,7 @@
-use super::error::Error;
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::Check;
+use crate::error::Error;
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::Check;
 
 #[must_use]
 pub fn upgrade_macos(dry_run: bool) -> Option<(&'static str, bool)> {

src/steps/os/mod.rs

@@ -0,0 +1,10 @@
+#[cfg(target_os = "freebsd")]
+pub mod freebsd;
+#[cfg(target_os = "linux")]
+pub mod linux;
+#[cfg(target_os = "macos")]
+pub mod macos;
+#[cfg(unix)]
+pub mod unix;
+#[cfg(target_os = "windows")]
+pub mod windows;

src/steps/os/unix.rs

@@ -1,7 +1,7 @@
-use super::error::Error;
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::{which, Check};
+use crate::error::Error;
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::{which, Check};
 use directories::BaseDirs;
 
 pub fn run_zplug(base_dirs: &BaseDirs, dry_run: bool) -> Option<(&'static str, bool)> {

src/steps/os/windows.rs

@@ -1,7 +1,7 @@
-use super::error::{Error, ErrorKind};
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::{self, which, Check};
+use crate::error::{Error, ErrorKind};
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::{self, which, Check};
 use failure::ResultExt;
 use log::error;
 use std::path::PathBuf;

src/steps/tmux.rs

@@ -1,8 +1,7 @@
-use super::error::{Error, ErrorKind};
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::which;
-use super::utils::{Check, PathExt};
+use crate::error::{Error, ErrorKind};
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::{which, Check, PathExt};
 use directories::BaseDirs;
 use failure::ResultExt;
 use std::env;

src/steps/vim.rs

@@ -1,7 +1,7 @@
-use super::error::Error;
-use super::executor::Executor;
-use super::terminal::print_separator;
-use super::utils::{which, Check, PathExt};
+use crate::error::Error;
+use crate::executor::Executor;
+use crate::terminal::print_separator;
+use crate::utils::{which, Check, PathExt};
 use directories::BaseDirs;
 use std::fs;
 use std::path::PathBuf;