@phil-opp I agree, that looks simpler.

EDIT: now I remember why I did it that way. The compiler complains about temporary value is freed at the end of this statement

error[E0716]: temporary value dropped while borrowed
  --> build.rs:8:29
   |
8  |     let kernel = Path::new(&std::env::var_os("CARGO_BIN_FILE_KERNEL_kernel").unwrap());
   |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ - temporary value is freed at the end of this statement
   |                             |
   |                             creates a temporary value which is freed while still in use
...
18 |     bootloader::BiosBoot::new(&kernel)
   |                               ------- borrow later used here
   |
help: consider using a `let` binding to create a longer lived value
   |
8  ~     let binding = std::env::var_os("CARGO_BIN_FILE_KERNEL_kernel").unwrap();
9  ~     let kernel = Path::new(&binding);
   |

If you have a different way, let me know

Hello, I think that in the Cargo.toml file you should change ovmf_prebuilt to ovmf-prebuilt because the first package doesn't exists in crates.io but the second does.

The compiler complains about temporary value is freed at the end of this statement

I think we can use PathBuf instead of Path to take ownership of the string. I pushed 63284c6 with that change.

Hello, I think that in the Cargo.toml file you should change ovmf_prebuilt to ovmf-prebuilt because the first package doesn't exists in crates.io but the second does.

Thanks, done in 0d24cef.

Could you try whether the new version works for you? Then we can finally merge this!

The new version is working very good, thanks!

Ok great, thanks for testing!