Merge pull request #1 from DmitryAstafyev/master

Corrections

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "envvars"
-version = "0.1.5"
+version = "0.1.6"
 edition = "2021"
 authors = ["[email protected]"]
 description = "Getting environment variables considering shell configuration"

src/extractor.rs

@@ -60,7 +60,7 @@ impl Extractor {
 
     fn delivery(&mut self) -> Result<(), io::Error> {
         if self.location.exists() {
-            log::warn!(
+            log::info!(
                 "Extractor {:?} already exists. Checking checksum.",
                 self.location
             );

src/profiles/unix.rs

@@ -12,9 +12,18 @@ pub(crate) fn get() -> Result<Vec<Profile>, Error> {
     for shell in read_to_string(shells_file_path)
         .map_err(Error::Io)?
         .split('\n')
+        .filter(|s| !s.starts_with('#') && !s.is_empty())
     {
         let path = Path::new(shell);
-        let profile = match Profile::new(&path.to_path_buf(), vec!["-c"], None) {
+        let profile = match Profile::new(
+            &path.to_path_buf(),
+            if path.ends_with("tcsh") || path.ends_with("csh") {
+                vec!["-ic"]
+            } else {
+                vec!["-i", "-l", "-c"]
+            },
+            None,
+        ) {
             Ok(profile) => profile,
             Err(err) => {
                 log::warn!("Cannot get envvars for {shell}: {err}");