Skip to content

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