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

Adding the option to build syslinux into initramfs #3632

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

phoefler
Copy link
Contributor

This PR also builds syslinux into initramfs if INITRAMFS_PARTED_SUPPORT variable is set to yes (in project options file)

@stefansaraev
Copy link
Contributor

huh but why?

@phoefler
Copy link
Contributor Author

the same reason @sraue added parted and mkfs.ext4 as an option: we use a project specific initramfs.conf file which listens for a kernel command line parameter and can automatically copy KERNEL & SYSTEM to ram, repartition the harddisk to two partitions, format the two partitions, copies KERNEL & SYSTEM back to part1 and install syslinux bootloader to the harddisk ;-)
That way we can change thousand of boxes from onepartition OS to OE without a problem ;-)

@stefansaraev
Copy link
Contributor

huh. I dont really understand the reasoning. this (and repartitioning) should be done via some systemd service like our fs-resize.service. init should be as minimal as possible. @sraue ?

and you miss gptmbr.bin and syslinux. not everything everywhere is ext4. EDIT: doesnt really matter, as this seems to be for some very specific project of yours.

but well. ok...

@phoefler
Copy link
Contributor Author

additional features (filesystems, loaders) are welcome if someone else uses this feature ;-)

@chewitt
Copy link
Contributor

chewitt commented Dec 20, 2015

@phoefler - I'm doing cleanup/review of old PR's and last update on this was ~13 months ago. Is this still required?

@phoefler
Copy link
Contributor Author

phoefler commented Jan 1, 2016

@chewitt Yes, we would still appreciate this change in master

@chewitt
Copy link
Contributor

chewitt commented Jan 1, 2016

@sraue - please review and comment

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.

4 participants