Add a system wide configuration file sharkdp#668

CHANGELOG.md

@@ -22,6 +22,7 @@
 ## Features
 
 - Use a pager when `bat --list-languages` is called, see #1394 (@stku1985)
+- Support a system wide configuration file on unix platforms `/etc/bat/config`
 
 ## Bugfixes
 

src/bin/bat/app.rs

@@ -7,7 +7,7 @@ use atty::{self, Stream};
 
 use crate::{
     clap_app,
-    config::{get_args_from_config_file, get_args_from_env_var},
+    config::{get_args_from_config_files, get_args_from_env_var},
 };
 use clap::ArgMatches;
 
@@ -61,7 +61,7 @@ impl App {
 
             // Read arguments from bats config file
             let mut args = get_args_from_env_var()
-                .unwrap_or_else(get_args_from_config_file)
+                .unwrap_or_else(get_args_from_config_files)
                 .chain_err(|| "Could not parse configuration file")?;
 
             // Put the zero-th CLI argument (program name) first

src/bin/bat/config.rs

@@ -13,6 +13,12 @@ pub fn config_file() -> PathBuf {
         .unwrap_or_else(|| PROJECT_DIRS.config_dir().join("config"))
 }
 
+pub fn system_config_file() -> Option<PathBuf> {
+    PROJECT_DIRS
+        .system_config_dir()
+        .map(|path| path.join("config"))
+}
+
 pub fn generate_config_file() -> bat::error::Result<()> {
     let config_file = config_file();
     if config_file.is_file() {
@@ -86,12 +92,16 @@ pub fn generate_config_file() -> bat::error::Result<()> {
     Ok(())
 }
 
-pub fn get_args_from_config_file() -> Result<Vec<OsString>, shell_words::ParseError> {
-    Ok(fs::read_to_string(config_file())
-        .ok()
-        .map(|content| get_args_from_str(&content))
-        .transpose()?
-        .unwrap_or_else(Vec::new))
+pub fn get_args_from_config_files() -> Result<Vec<OsString>, shell_words::ParseError> {
+    let system_config = system_config_file().and_then(|path| fs::read_to_string(path).ok());
+    let user_config = fs::read_to_string(config_file()).ok();
+
+    Ok(system_config
+        .iter()
+        .chain(user_config.iter())
+        .filter_map(|content| get_args_from_str(&content).ok())
+        .flatten()
+        .collect())
 }
 
 pub fn get_args_from_env_var() -> Option<Result<Vec<OsString>, shell_words::ParseError>> {

src/bin/bat/directories.rs

@@ -9,6 +9,7 @@ use lazy_static::lazy_static;
 pub struct BatProjectDirs {
     cache_dir: PathBuf,
     config_dir: PathBuf,
+    system_config_dir: Option<PathBuf>,
 }
 
 impl BatProjectDirs {
@@ -26,9 +27,16 @@ impl BatProjectDirs {
 
         let config_dir = config_dir_op.map(|d| d.join("bat"))?;
 
+        #[cfg(target_family = "unix")]
+        let system_config_dir = Some(PathBuf::from("/etc/bat"));
+
+        #[cfg(target_family = "windows")]
+        let system_config_dir = None;
+
         Some(BatProjectDirs {
             cache_dir,
             config_dir,
+            system_config_dir,
         })
     }
 
@@ -58,6 +66,10 @@ impl BatProjectDirs {
     pub fn config_dir(&self) -> &Path {
         &self.config_dir
     }
+
+    pub fn system_config_dir(&self) -> Option<&PathBuf> {
+        self.system_config_dir.as_ref()
+    }
 }
 
 lazy_static! {