Ignore empty paths in windows registry search (#156)

Fixes #149

Author: DonJayamanne

.devcontainer/linux-x64/Dockerfile

@@ -1,4 +1,4 @@
-FROM amd64/ubuntu
+FROM amd64/ubuntu:24.04
 
 # Setup Python
 RUN apt-get update && \

.github/workflows/pr-check.yml

@@ -133,7 +133,7 @@ jobs:
         if: startsWith( matrix.os, 'ubuntu') || startsWith( matrix.os, 'macos')
         run: |
           pyenv install --list
-          pyenv install 3.12.5 3.8.19
+          pyenv install 3.12.6 3.8.19
         shell: bash
 
         # pyenv-win install list has not updated for a while

crates/pet-windows-registry/src/environments.rs

@@ -101,6 +101,13 @@ fn get_registry_pythons_from_key_for_company(
                     Ok(install_path_key) => {
                         let env_path: String =
                             install_path_key.get_value("").ok().unwrap_or_default();
+                        if env_path.is_empty() {
+                            warn!(
+                                "Install path is empty {}\\Software\\Python\\{}\\{}",
+                                key_container, company, installed_python
+                            );
+                            continue;
+                        }
                         let env_path = norm_case(PathBuf::from(env_path));
                         if is_windows_app_folder_in_program_files(&env_path) {
                             trace!(

crates/pet/tests/ci_homebrew_container.rs

@@ -50,7 +50,7 @@ fn verify_python_in_homebrew_contaner() {
     let python3_12 = PythonEnvironment {
         kind: Some(PythonEnvironmentKind::Homebrew),
         executable: Some(PathBuf::from("/home/linuxbrew/.linuxbrew/bin/python3")),
-        version: Some("3.12.5".to_string()), // This can change on CI, so we don't check it
+        version: Some("3.12.6".to_string()), // This can change on CI, so we don't check it
         symlinks: Some(vec![
             PathBuf::from("/home/linuxbrew/.linuxbrew/bin/python3"),
             PathBuf::from("/home/linuxbrew/.linuxbrew/bin/python3.12"),
@@ -61,19 +61,19 @@ fn verify_python_in_homebrew_contaner() {
             PathBuf::from("/home/linuxbrew/.linuxbrew/opt/[email protected]/bin/python3"),
             PathBuf::from("/home/linuxbrew/.linuxbrew/opt/[email protected]/bin/python3.12"),
             // On CI the Python version can change with minor updates, so we don't check the full version.
-            // PathBuf::from("/home/linuxbrew/.linuxbrew/Cellar/[email protected]/3.12.5/bin/python3.12"),
+            // PathBuf::from("/home/linuxbrew/.linuxbrew/Cellar/[email protected]/3.12.6/bin/python3.12"),
         ]),
         ..Default::default()
     };
     let python3_11 = PythonEnvironment {
         kind: Some(PythonEnvironmentKind::Homebrew),
         executable: Some(PathBuf::from("/home/linuxbrew/.linuxbrew/bin/python3.11")),
-        version: Some("3.11.9".to_string()), // This can change on CI, so we don't check it
+        version: Some("3.11.10".to_string()), // This can change on CI, so we don't check it
         symlinks: Some(vec![
             PathBuf::from("/home/linuxbrew/.linuxbrew/bin/python3.11"),
             PathBuf::from("/home/linuxbrew/.linuxbrew/opt/[email protected]/bin/python3.11"),
             // On CI the Python version can change with minor updates, so we don't check the full version.
-            // PathBuf::from("/home/linuxbrew/.linuxbrew/Cellar/[email protected]/3.11.9/bin/python3.11"),
+            // PathBuf::from("/home/linuxbrew/.linuxbrew/Cellar/[email protected]/3.11.10/bin/python3.11"),
         ]),
         ..Default::default()
     };