Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add EFI boot support for Windows 7 64-Bit #59

Merged
merged 2 commits into from Aug 10, 2016
Merged

Conversation

Holzhaus
Copy link
Contributor

This adds UEFI boot support for Windows 7. In PR #25 only added that for Windows >=8:

According to my test on Windows 7/Windows 10 ISO I suspect that UEFI support will be compatible with Windows >=8(Although Windows 7 installation media does has efi files it's not following UEFI removable bootable device spec and I'm currently failed to make it boot).

The key is to extract the bootmgfw.efi file from sources/install.wim and move it to EFI/Boot/Bootx64.efi.

Downside is that this relies on p7zip for file extraction (and thus introduces a new dependency).

Also, I'm not completely sure if my Windows 7 detection mechanism is correct:

  1. I assumed that the MinServer value in in sources/cversion.ini begins with 7 if we're dealing with Windows 7 and that this file exists in all relevant Windows versions.
  2. If we're using Windows 7, bootmgr.efi does only exist if it has UEFI support and is 64-Bit (since 32-Bit Windows 7 has no UEFI support).

I can't really verify this assumptions. They are true for my Windows 7 Pro 64-Bit installation disk.

@brlin-tw
Copy link
Collaborator

brlin-tw commented Aug 3, 2016

NIce work! The p7zip dependency should be fine as long as Debian software archive provides it.
Please also modify the debian control file to include p7zip-full as it's dependency.

@Holzhaus
Copy link
Contributor Author

Holzhaus commented Aug 4, 2016

Done.

@slacka slacka merged commit 0fb4db5 into slacka:master Aug 10, 2016
@slacka
Copy link
Owner

slacka commented Aug 10, 2016

Nicely done. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants