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

Use KERNEL_OUTPUTDIR for integrated initramfs configuration #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

laomaiweng
Copy link

--integrated-initramfs builds the kernel in 2 passes, and modifies the kernel configuration in-between.
However it uses ${KERNEL_DIR}/.config, which is in the kernel source tree. While this does work for in-tree builds, it makes out-of-tree builds fail since the source tree then appears unclean (and the written .config file only contains the CONFIG_INITRAMFS_* items anyway).

This PR switches to ${KERNEL_OUTPUTDIR}/.config which should be the right path both for in-tree and out-of-tree builds.

--integrated-initramfs builds the kernel in 2 passes, and modifies the kernel configuration in-between.
However it used ${KERNEL_DIR}/.config, which is in the kernel source tree. While this did work for in-tree builds, it made out-of-tree builds fail since the source tree then appeared unclean (and the written .config file only contained the CONFIG_INITRAMFS_* items anyway).
This commit switches to ${KERNEL_OUTPUTDIR}/.config which is the right path both for in-tree and out-of-tree builds.

Also, some additional quotes are added.
@laomaiweng
Copy link
Author

Out of the scope of this PR as I don't know how it should be fixed, but the case ${compress_ext} l.1090 seems wrong, as $compress_ext is empty at this point. Perhaps ${COMPRESS_INITRD_TYPE} was meant instead?

@laomaiweng
Copy link
Author

Pretty please fix this? :)

laomaiweng added a commit to laomaiweng/laomaiweng-overlay that referenced this pull request Oct 18, 2017
Apply patch for PRs Sabayon/genkernel-next#46 and Sabayon/genkernel-next#49.

Package-Manager: Portage-2.3.11, Repoman-2.3.3
RepoMan-Options: --ignore-arches
Manifest-Sign-Key: 0xAA0929D0263BE9D6C260BDC5FF79BF270F5F912C
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.

1 participant