leyoda/rqbit
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Migrate linux configuration to a better named path.

Browse source
This commit is contained in:
Igor Katson 2024-10-06 10:54:00 +01:00
parent 61a01b4253
commit aaca88be39
No known key found for this signature in database
GPG key ID: B4EC22B66D61A3F5
1 changed files with 57 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

59
crates/librqbit_core/src/directories.rs
View file

@ -1,6 +1,61 @@
use anyhow::Context;
use tracing::{info, warn};
pub fn get_configuration_directory_linux(
application: &str,
) -> anyhow::Result<directories::ProjectDirs> {
// https://github.com/ikatson/rqbit/issues/252
// On Linux, "com.rqbit" isn't used resulting in weird folder names like "session".
// The code below migrates from old configuration to new.
let legacy_dir =
directories::ProjectDirs::from("com", "rqbit", application).with_context(|| {
format!("cannot determine project directory for com.rqbit.{application}")
})?;
let new_dir =
directories::ProjectDirs::from("com", "rqbit", &format!("com.rqbit.{application}"))
.with_context(|| {
format!("cannot determine project directory for com.rqbit.{application}")
})?;
for (old, new) in [
(legacy_dir.cache_dir(), new_dir.cache_dir()),
(legacy_dir.config_dir(), new_dir.config_dir()),
(legacy_dir.data_dir(), new_dir.data_dir()),
] {
match (old.exists(), new.exists()) {
(true, true) => {
warn!(
?old,
?new,
"can't migrate configuraiton as both directories exist, not sure what to do"
)
}
(true, false) => {
info!(
?old,
?new,
"migrating configuration directories as rqbit was upgraded"
);
if let Err(e) = std::fs::rename(old, new) {
warn!(?old, ?new, "error migrating: {e:#}");
}
}
// In these cases, nothing to migrate, so do nothing.
(false, true) => {}
(false, false) => {}
}
}
Ok(new_dir)
}
pub fn get_configuration_directory(application: &str) -> anyhow::Result<directories::ProjectDirs> {
directories::ProjectDirs::from("com", "rqbit", application)
.with_context(|| format!("cannot determine project directory for com.rqbit.{application}"))
#[cfg(not(any(target_os = "macos", target_os = "windows")))]
{
get_configuration_directory_linux(application)
}
#[cfg(any(target_os = "macos", target_os = "windows"))]
{
directories::ProjectDirs::from("com", "rqbit", application).with_context(|| {
format!("cannot determine project directory for com.rqbit.{application}")
})
}
}