Project crashes due unavalibity of ui files when compiling app from crates.io

[qarmin]

The problem is that ui is loaded in runtime.

GTK-rs allows to include ui into binary:

        let glade_src = include_str!("../ui/main_window.glade").to_string();
        let builder = Builder::from_string(glade_src.as_str());

        //// Windows
        let window_main: gtk::Window = builder.object("window_main").unwrap();

https://github.com/qarmin/czkawka/blob/aeeb59d9a6b3429f4c35a9c1f71914293b4981d6/czkawka_gui/src/gui_data.rs#L85-L89

[xlmnxp]

that will not work as we expect it
include_str! work in compile not in runtime

[qarmin]

Yes it works in compile time(builder don't need to be set in runtime), so this code

blue-recorder/src/main.rs

Lines 36 to 52 in 58585c3

	let builder: Builder;
	let user_interface_path_abs = {
	let mut current_exec_dir = std::env::current_exe().unwrap();
	current_exec_dir.pop();
	current_exec_dir
	}
	.join(Path::new("interfaces/main.ui"));
	if user_interface_path_abs.exists() {
	builder = Builder::from_file(user_interface_path_abs);
	} else {
	builder = Builder::from_file(
	std::env::var("INTERFACES_DIR")
	.unwrap_or(String::from("interfaces/"))
	.add("main.ui"),
	);
	}

 let builder: Builder; 
 let user_interface_path_abs = { 
     let mut current_exec_dir = std::env::current_exe().unwrap(); 
     current_exec_dir.pop(); 
     current_exec_dir 
 } 
 .join(Path::new("interfaces/main.ui")); 
  
 if user_interface_path_abs.exists() { 
     builder = Builder::from_file(user_interface_path_abs); 
 } else { 
     builder = Builder::from_file( 
         std::env::var("INTERFACES_DIR") 
             .unwrap_or(String::from("interfaces/")) 
             .add("main.ui"), 
     ); 
 } 

could be simplified to

        let ui_src = include_str!("../interfaces/main.ui").to_string();
        let builder = Builder::from_string(ui_src.as_str());

[xlmnxp]

Yes it works in compile time(builder don't need to be set in runtime), so this code

blue-recorder/src/main.rs

Lines 36 to 52 in 58585c3

	let builder: Builder;
	let user_interface_path_abs = {
	let mut current_exec_dir = std::env::current_exe().unwrap();
	current_exec_dir.pop();
	current_exec_dir
	}
	.join(Path::new("interfaces/main.ui"));
	if user_interface_path_abs.exists() {
	builder = Builder::from_file(user_interface_path_abs);
	} else {
	builder = Builder::from_file(
	std::env::var("INTERFACES_DIR")
	.unwrap_or(String::from("interfaces/"))
	.add("main.ui"),
	);
	}

 let builder: Builder; 
 let user_interface_path_abs = { 
     let mut current_exec_dir = std::env::current_exe().unwrap(); 
     current_exec_dir.pop(); 
     current_exec_dir 
 } 
 .join(Path::new("interfaces/main.ui")); 
  
 if user_interface_path_abs.exists() { 
     builder = Builder::from_file(user_interface_path_abs); 
 } else { 
     builder = Builder::from_file( 
         std::env::var("INTERFACES_DIR") 
             .unwrap_or(String::from("interfaces/")) 
             .add("main.ui"), 
     ); 
 } 

could be simplified to

        let ui_src = include_str!("../interfaces/main.ui").to_string();
        let builder = Builder::from_string(ui_src.as_str());

That will remove the need to INTERFACES_DIR environment

[xlmnxp]

I did it 👍🏼

[xlmnxp]

thank you