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

place efiboot.img and macboot.img in /isolinux #189

Merged
merged 1 commit into from
Jun 14, 2021

Conversation

sobjerke
Copy link
Contributor

#188 create efi image files under /isolinux, not /images

@Conan-Kudo
Copy link
Member

I think @FGrose changed this to match Lorax, it might be worth him looking at this...

@sobjerke
Copy link
Contributor Author

Seems like the change happened in this commit back in 2019:
e4c96c2

However, when mkefiboot tries to create the image, the directory /images does not exist, and efiboot.img never gets created, resulting in iso image that is not bootable at all.

I can see two options here

  1. make sure /images is created before mkefiboot is run (and update any other references to the path)
  2. revert the change and put the files in /isolinux

@Conan-Kudo
Copy link
Member

@sobjerke I would prefer a change to make it so that /images existed. Aligning with Lorax makes things less painful down the road.

@sobjerke
Copy link
Contributor Author

@Conan-Kudo and @FGrose: Does this look like the correct place to create /images?

diff --git a/imgcreate/live.py b/imgcreate/live.py
index 673c54d..0eb6fdb 100644
--- a/imgcreate/live.py
+++ b/imgcreate/live.py
@@ -237,6 +237,7 @@ class LiveImageCreatorBase(LoopImageCreator):
             return

         # XXX-BCL: does this need --label?
+        makedirs(isodir + "/images")
         subprocess.call(["mkefiboot", isodir + "/EFI/BOOT",
                          isodir + "/images/efiboot.img"])

With this change and a modified version of #190 to reflect the changed path of efiboot.img, I am now able to create a live image of CentOS 8 that is bootable both via bios and efi.

If you agree, I will replace #189 with a new PR.

mikhailnov pushed a commit to mikhailnov/livecd-tools that referenced this pull request May 31, 2021
@mikhailnov
Copy link
Contributor

This is also needed for ROSA

@Conan-Kudo
Copy link
Member

Let's take this for now and fix it later.

@Conan-Kudo Conan-Kudo merged commit f5d76fa into livecd-tools:main Jun 14, 2021
@sobjerke sobjerke deleted the img-path branch June 15, 2021 06:24
@sobjerke sobjerke restored the img-path branch June 15, 2021 06:24
@sobjerke sobjerke deleted the img-path branch June 15, 2021 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants