ivandavidov / minimal-linux-script

One script which generates live Linux ISO image with minimal effort. Based on the first published version of Minimal Linux Live: http://github.com/ivandavidov/minimal
GNU General Public License v3.0
219 stars 70 forks source link

Script Syslinux Fix #1

Closed AwlsomeAlex closed 7 years ago

AwlsomeAlex commented 7 years ago

Due to a change in the Linux Kernel starting with 4.6.2+, an attempt to fix the missing dependency of ldlinux.c32/syslinux is added, but instead ends up breaking the kernel completely with this script. What I did to fix this was to use the .iso creation process used in the normal Minimal Linux Live project, which manually adds the Syslinux Dependencies onto the .iso. Also by using genisoimage instead of directly building from the kernel, it gives people more room to add to the Rootfs, Kernel, etc. Also one minor fix was changing "find . | cpio -R +0:+0 -H newc -o | gzip > ../../rootfs.cpio.gz" to "find . | cpio -R root:root -H newc -o | gzip > ../../rootfs.cpio.gz" due to it not compiling without root being the value. It's your call if you want to merge this or not, but it future-proofs the Linux Kernel for now.

AwlsomeAlex commented 7 years ago

I just saw that you posted something about this problem. This may be a bit overcomplicated for a permanent solution, but it works with my machine (Linux Mint 18) where the old method didn't even compile, and it makes it a bit easier to make the patch work without the need of the Linux Kernel handling it.

ivandavidov commented 7 years ago

Great work!

I may not merge it directly though but I will definitely update the process to include the Syslinux stuff based on your work.

--Ivan Davidov

On Sat, Oct 1, 2016 at 8:18 AM, AwlsomeAlex notifications@github.com wrote:

I just saw that you posted something about this problem. This may be a bit overcomplicated for a permanent solution, but it works with my machine (Linux Mint 18) where the old method didn't even compile, and it makes it a bit easier to make the patch work without the need of the Linux Kernel handling it.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ivandavidov/minimal-linux-script/pull/1#issuecomment-250893334, or mute the thread https://github.com/notifications/unsubscribe-auth/AH9QpGGX9ARDjT8zJDqb_OG1_cNN_B9Hks5qve0wgaJpZM4KLq-t .

AwlsomeAlex commented 7 years ago

Alrighty. Thanks, glad I can help.

AwlsomeAlex commented 7 years ago

Also @ivandavidov on my fork of Minimal Linux Live Script I will be testing new Linux Kernel Releases (RC as well) and once they are stable, I will push it to the main repository (Your Repository) if that's fine by you, just so you don't have to be worrying if the script works after every linux update. Also I was playing around with Busybox 1.25.0, even though it's unstable, it compiles and works fine, so it's your call if you want to change it from Busybox 1.24.2 to 1.25.0.

ivandavidov commented 7 years ago

In my repository I'd rather stick with the stable versions because the unstable/testing versions may or may not work properly on most machines. Besides, most people rarely need up-to-date software, they only need stable, fully working solution. This is why I intend to stick with stable software versions.

Of course, once BusyBox release the next stable version, feel free to provide another pull request if I haven't updated the script already. The same applies for the kernel.

AwlsomeAlex commented 7 years ago

Okay, will do! @ivandavidov Also I'll be sure to test it on multiple machines as well as multiple Virtualization Platforms.