Skip to content

Commit

Permalink
module load minimap
Browse files Browse the repository at this point in the history
resulted in minimap/x86_64/20160105 loaded, while minimap/x86_64/2.1 was marked as default.
module list said minimap/x86_64/2.1 was loaded but in reality minimap/x86_64/20160105 is loaded.
the reason this happened is because we broke out of the loop in the wrong place
  • Loading branch information
Frederik Delaere committed Oct 18, 2018
1 parent 9ee94b0 commit 4683e56
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/rsmodules.rs
Original file line number Diff line number Diff line change
Expand Up @@ -444,9 +444,9 @@ fn module_action(rsmod: &mut Rsmodule, action: &str) {
// possible non existing file
if Path::new(&testpath).exists() && Path::new(&testpath).is_file() {
modulefile = PathBuf::from(&testpath);
break 'outer;
}

break 'outer;
}

if found && !module.0.starts_with(mdl) {
Expand Down Expand Up @@ -476,9 +476,9 @@ fn module_action(rsmod: &mut Rsmodule, action: &str) {
eprintln!("Module {} not found.", selected_module);
::std::process::exit(super::CRASH_MODULE_NOT_FOUND);
}

// check of another version is already loaded
// and replace it with the current one

let mut replaced_module: bool = false;
let mut other: String = String::new();
if is_other_version_of_module_loaded(selected_module) && action == "load" {
Expand Down

0 comments on commit 4683e56

Please sign in to comment.