Rename mullvad-metadata -> mullvad-paths

5 files changed, 6 insertions, 77 deletions

diff --git a/mullvad-daemon/src/bin/problem-report.rs b/mullvad-daemon/src/bin/problem-report.rs
index b3b484da73..b5df2d7fa9 100644
--- a/mullvad-daemon/src/bin/problem-report.rs
+++ b/mullvad-daemon/src/bin/problem-report.rs
@@ -15,7 +15,7 @@ extern crate lazy_static;
 extern crate regex;
 
 #[cfg(windows)]
-extern crate mullvad_metadata;
+extern crate mullvad_paths;
 extern crate mullvad_rpc;
 
 use error_chain::ChainedError;
@@ -50,12 +50,10 @@ const LINE_SEPARATOR: &str = "\r\n";
 #[cfg(windows)]
 lazy_static! {
     static ref LOG_DIRECTORY: PathBuf = {
-        use mullvad_metadata::PRODUCT_NAME;
-
         let program_data_dir =
             env::var_os("ALLUSERSPROFILE").expect("Missing %ALLUSERSPROFILE% environment variable");
 
-        PathBuf::from(program_data_dir).join(PRODUCT_NAME)
+        PathBuf::from(program_data_dir).join(mullvad_paths::PRODUCT_NAME)
     };
 }
 
diff --git a/mullvad-daemon/src/cache.rs b/mullvad-daemon/src/cache.rs
deleted file mode 100644
index ba66266657..0000000000
--- a/mullvad-daemon/src/cache.rs
+++ /dev/null
@@ -1,19 +0,0 @@
-use {ErrorKind, Result, ResultExt};
-
-use std::path::PathBuf;
-
-#[cfg(target_os = "linux")]
-pub fn get_cache_dir() -> Result<PathBuf> {
-    use std::fs;
-
-    let dir = PathBuf::from("/var/cache/mullvad-daemon");
-    fs::create_dir_all(&dir).chain_err(|| ErrorKind::NoCacheDir)?;
-    Ok(dir)
-}
-
-#[cfg(any(target_os = "macos", windows))]
-pub fn get_cache_dir() -> Result<PathBuf> {
-    use mullvad_metadata::APP_INFO;
-    ::app_dirs::app_root(::app_dirs::AppDataType::UserCache, &APP_INFO)
-        .chain_err(|| ErrorKind::NoCacheDir)
-}
diff --git a/mullvad-daemon/src/cli.rs b/mullvad-daemon/src/cli.rs
index 1ee12d7b2f..c6dcfd8def 100644
--- a/mullvad-daemon/src/cli.rs
+++ b/mullvad-daemon/src/cli.rs
@@ -9,8 +9,6 @@ pub struct Config {
     pub log_level: log::LevelFilter,
     pub log_file: Option<PathBuf>,
     pub tunnel_log_file: Option<PathBuf>,
-    pub resource_dir: Option<PathBuf>,
-    pub cache_dir: Option<PathBuf>,
     pub log_stdout_timestamps: bool,
     pub run_as_service: bool,
     pub register_service: bool,
@@ -27,8 +25,6 @@ pub fn get_config() -> Config {
     };
     let log_file = matches.value_of_os("log_file").map(PathBuf::from);
     let tunnel_log_file = matches.value_of_os("tunnel_log_file").map(PathBuf::from);
-    let resource_dir = matches.value_of_os("resource_dir").map(PathBuf::from);
-    let cache_dir = matches.value_of_os("cache_dir").map(PathBuf::from);
     let log_stdout_timestamps = !matches.is_present("disable_stdout_timestamps");
 
     let run_as_service = cfg!(windows) && matches.is_present("run_as_service");
@@ -38,8 +34,6 @@ pub fn get_config() -> Config {
         log_level,
         log_file,
         tunnel_log_file,
-        resource_dir,
-        cache_dir,
         log_stdout_timestamps,
         run_as_service,
         register_service,
@@ -72,20 +66,6 @@ fn create_app() -> App<'static, 'static> {
                 .help("Save log from tunnel implementation process to this file path."),
         )
         .arg(
-            Arg::with_name("resource_dir")
-                .long("resource-dir")
-                .takes_value(true)
-                .value_name("DIR")
-                .help("Uses the given directory to read needed resources, such as certificates."),
-        )
-        .arg(
-            Arg::with_name("cache_dir")
-                .long("cache-dir")
-                .takes_value(true)
-                .value_name("DIR")
-                .help("Uses the given directory to read and write cache."),
-        )
-        .arg(
             Arg::with_name("disable_stdout_timestamps")
             .long("disable-stdout-timestamps")
             .help("Don't log timestamps when logging to stdout, useful when running as a systemd service")
diff --git a/mullvad-daemon/src/main.rs b/mullvad-daemon/src/main.rs
index efcf7266a2..9bace35a2e 100644
--- a/mullvad-daemon/src/main.rs
+++ b/mullvad-daemon/src/main.rs
@@ -32,7 +32,7 @@ extern crate tokio_timer;
 extern crate uuid;
 
 extern crate mullvad_ipc_client;
-extern crate mullvad_metadata;
+extern crate mullvad_paths;
 extern crate mullvad_rpc;
 extern crate mullvad_types;
 extern crate talpid_core;
@@ -44,7 +44,6 @@ extern crate talpid_types;
 extern crate windows_service;
 
 mod account_history;
-mod cache;
 mod cli;
 mod geoip;
 mod logging;
@@ -888,11 +887,8 @@ fn run_standalone(config: cli::Config) -> Result<()> {
         warn!("Running daemon as a non-administrator user, clients might refuse to connect");
     }
 
-    let resource_dir = config.resource_dir.unwrap_or_else(|| get_resource_dir());
-    let cache_dir = match config.cache_dir {
-        Some(cache_dir) => cache_dir,
-        None => cache::get_cache_dir()?,
-    };
+    let resource_dir = mullvad_paths::get_resource_dir();
+    let cache_dir = mullvad_paths::get_cache_dir().chain_err(|| "Unable to get cache dir")?;
 
     let daemon = Daemon::new(config.tunnel_log_file, resource_dir, cache_dir)
         .chain_err(|| "Unable to initialize daemon")?;
@@ -917,22 +913,6 @@ fn log_version() {
     )
 }
 
-fn get_resource_dir() -> PathBuf {
-    match env::current_exe() {
-        Ok(mut path) => {
-            path.pop();
-            path
-        }
-        Err(e) => {
-            error!(
-                "Failed finding the install directory. Using working directory: {}",
-                e
-            );
-            PathBuf::from(".")
-        }
-    }
-}
-
 #[cfg(unix)]
 fn running_as_admin() -> bool {
     let uid = unsafe { libc::getuid() };
diff --git a/mullvad-daemon/src/settings.rs b/mullvad-daemon/src/settings.rs
index 776606d1fc..d3abd190d6 100644
--- a/mullvad-daemon/src/settings.rs
+++ b/mullvad-daemon/src/settings.rs
@@ -86,18 +86,8 @@ impl Settings {
         serde_json::to_writer_pretty(file, self).chain_err(|| ErrorKind::WriteError(path))
     }
 
-    #[cfg(unix)]
     fn get_settings_path() -> Result<PathBuf> {
-        let dir = PathBuf::from("/etc/mullvad-daemon");
-        ::std::fs::create_dir_all(&dir).chain_err(|| ErrorKind::DirectoryError)?;
-        Ok(dir.join(SETTINGS_FILE))
-    }
-
-    #[cfg(windows)]
-    fn get_settings_path() -> Result<PathBuf> {
-        use mullvad_metadata::APP_INFO;
-        let dir = ::app_dirs::app_root(::app_dirs::AppDataType::UserConfig, &APP_INFO)
-            .chain_err(|| ErrorKind::DirectoryError)?;
+        let dir = ::mullvad_paths::get_settings_dir().chain_err(|| ErrorKind::DirectoryError)?;
         Ok(dir.join(SETTINGS_FILE))
     }