src/command/share.rs

@@ -1,5 +1,6 @@
 //! globally available tmux commands.
 use std::{
+	env::var,
 	ffi::OsString,
 	process::exit
 };
@@ -11,12 +12,7 @@ use tmux_interface::{
 	commands
 };
 
-use crate::{
+use crate::{ error, flag, util };
-	env::{ self, env_var },
-	error,
-	flag,
-	util
-};
 
 pub fn attach(pargs: &mut Arguments) {
 	//	don't allow unflagged nests
@@ -24,7 +20,7 @@ pub fn attach(pargs: &mut Arguments) {
 
 	//	consume optional flags
 	let read_only = pargs.contains(flag::READ_ONLY);
-	let detach_other = pargs.contains(flag::DETACH);
+	let detach_other = pargs.contains(flag::DETACHED);
 
 	let args = pargs.clone().finish();
 	let target: String;
@@ -123,7 +119,7 @@ pub fn list() {
 	}
 
 	//	get attached session symbol
-	let attach_symbol = env_var(env::ATTACH_SYMBOL);
+	let attach_symbol = var("REMUX_ATTACH_SYMBOL").unwrap_or("*".to_string());
 
 	//	pretty print session list
 	println!("sessions:");
@@ -149,13 +145,9 @@ pub fn new(pargs: &mut Arguments) {
 	//	don't allow unflagged nesting
 	util::prevent_nest();
 
-	//	get optional flags
+	//	get optional flag
-	let detached = pargs.contains(flag::DETACH);
 	let target_dir: Result<String, Error> = pargs.value_from_str(flag::TARGET);
 
-	//	get environment variables
-	let window_name = env_var(env::NEW_WINDOW_NAME);
-
 	//	get target or fallback
 	let args = pargs.clone().finish();
 	let title: String;
@@ -172,18 +164,10 @@ pub fn new(pargs: &mut Arguments) {
 	let mut new = commands::NewSession::new();
 	new = new.session_name(title);
 	if let Some(command) = command { new.shell_command = Some(command.to_string_lossy()); }
-	if detached { new.detached = true; }
 	if let Ok(target_dir) = target_dir { new = new.start_directory(target_dir); }
 
-	let mut tmux = Tmux::new().add_command(new);
+	Tmux::new()
-
+		.add_command(new)
-	//	rename window if var not empty
+		.output().ok();
-	if !window_name.is_empty() {
-		let auto_name = commands::RenameWindow::new()
-			.new_name(window_name);
-		tmux = tmux.add_command(auto_name);
-	}
-
-	tmux.output().ok();
 }
 

src/env.rs

@@ -1,13 +0,0 @@
-use std::env::var;
-
-pub type EnvVar = (&'static str, &'static str);
-
-pub static ATTACH_SYMBOL: EnvVar	= ("REMUX_ATTACH_SYMBOL", "*");
-pub static NEW_WINDOW_NAME: EnvVar	= ("REMUX_NEW_WINDOW", "");
-
-pub fn env_var(envvar: EnvVar) -> String {
-	var(envvar.0).unwrap_or(envvar.1.to_string())
-}
-
-pub fn tmux() -> bool { !var("TMUX").unwrap_or("".to_string()).is_empty() }
-

src/flag.rs

@@ -1,7 +1,7 @@
 
 type Flag = [&'static str;2];
 
-pub static DETACH: Flag		= ["-d", "--detach"];
+pub static DETACHED: Flag	= ["-d", "--detached"];
 pub static HELP: Flag		= ["-h", "--help"];
 pub static NEST: Flag		= ["-n", "--nest"];
 pub static QUIET: Flag		= ["-q", "--quiet"];

src/main.rs

@@ -6,7 +6,6 @@ use std::{
 use pico_args::Arguments;
 
 mod command;
-mod env;
 mod error;
 mod flag;
 mod help;

src/util.rs

@@ -1,5 +1,5 @@
 use std::{
-	env::current_dir,
+	env::{ current_dir, var },
 	path::PathBuf,
 	process::exit
 };
@@ -11,10 +11,7 @@ use tmux_interface::{
 	variables::session::SessionsCtl
 };
 
-use crate::{
+use crate::error;
-	env,
-	error
-};
 
 ///	return a Vec of all sessions or None
 pub fn get_sessions() -> Option<Vec<Session>> {
@@ -26,9 +23,10 @@ pub fn get_sessions() -> Option<Vec<Session>> {
 
 ///	show the tmux nest text if env var is not unset
 pub fn prevent_nest() {
-	if env::tmux() {
+	let tmux = var("TMUX").ok();
+	if tmux.is_some() && tmux.unwrap() != "" {
 		println!("Sessions should be nested with care; unset TMUX or use the '-n' flag to allow.");
-		exit(6);
+		exit(1);
 	}
 }