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

Makefile fixes #7

Merged
merged 2 commits into from
Apr 7, 2016
Merged

Makefile fixes #7

merged 2 commits into from
Apr 7, 2016

Conversation

plusky
Copy link
Contributor

@plusky plusky commented Apr 7, 2016

No description provided.

@gozora gozora merged commit d27c270 into gozora:master Apr 7, 2016
@jsmeix
Copy link

jsmeix commented Apr 7, 2016

@gozora
it would help me a lot for building ebiso
in a clean way in the openSUSE build service
if this enhancement was included in an ebiso release.

@gozora
Copy link
Owner

gozora commented Apr 7, 2016

Really have no problem with this merge, I'm bit worried about building ebiso on 32-bit architectures.
Not sure whether there aren't some pitfalls.

@jsmeix
Copy link

jsmeix commented Apr 7, 2016

At least for now I do not build ebiso on 32-bit architectures.
I have in ebiso.spec

# ebiso is only used by rear to make UEFI bootable ISO images
# on SLE12 systems so that no other architecture except x86_64 is needed
# (if also needed for 32-bit it could be "ExclusiveArch: %ix86 x86_64"):
ExclusiveArch:  x86_64

But it is nice to have when ebiso can be built
also on non-x6_64 architectures without the need
for changes in its code.

@gozora
Copy link
Owner

gozora commented Apr 7, 2016

@jsmeix currently I can't recall nothing that could avoid creating EFI bootable ISOs on 32-bit architectures, but it was never tested and I wold not like to put people into trouble especially in such stressful situations like bare metal restores.
Anyhow I'll put 32-bit testing on my todo list ...

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