Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 797613 - sys-kernel/gentoo-kernel: missing BDEPEND on cpio
Summary: sys-kernel/gentoo-kernel: missing BDEPEND on cpio
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Distribution Kernel Project
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2021-06-22 16:34 UTC by David Michael
Modified: 2024-02-18 21:54 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
fix.patch (0001-sys-kernel-gentoo-kernel-add-missing-BDEPEND-on-cpio.patch,958 bytes, patch)
2021-06-22 16:35 UTC, David Michael
Details | Diff
fix.patch (0001-kernel-build.eclass-add-cpio-to-BDEPEND.patch,866 bytes, patch)
2021-06-22 18:21 UTC, David Michael
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Michael 2021-06-22 16:34:53 UTC
The dist config needs cpio, which apparently is not installed in stage3s.

>   GEN     kernel/kheaders_data.tar.xz
> /var/tmp/portage/sys-kernel/gentoo-kernel-5.12.12/work/linux-5.12/kernel/gen_kheaders.sh: line 71: cpio: command not found
> make[2]: *** [/var/tmp/portage/sys-kernel/gentoo-kernel-5.12.12/work/linux-5.12/kernel/Makefile:155: kernel/kheaders_data.tar.xz] Error 127
> make[1]: *** [/var/tmp/portage/sys-kernel/gentoo-kernel-5.12.12/work/linux-5.12/Makefile:1856: kernel] Error 2
Comment 1 David Michael 2021-06-22 16:35:53 UTC
Created attachment 717687 [details, diff]
fix.patch
Comment 2 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2021-06-22 18:06:38 UTC
(In reply to David Michael from comment #1)
> Created attachment 717687 [details, diff] [details, diff]
> fix.patch

That's probably an issue common to all kernels, so please add BDEP to the eclass instead.
Comment 3 Mike Gilbert gentoo-dev 2021-06-22 18:21:07 UTC
It probably depends on the kernel config, so an eclass change might not be correct in all cases.

In this case, I think it is required by CONFIG_IKHEADERS.
Comment 4 David Michael 2021-06-22 18:21:24 UTC
Created attachment 717702 [details, diff]
fix.patch
Comment 5 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2021-06-22 18:35:23 UTC
(In reply to Mike Gilbert from comment #3)
> It probably depends on the kernel config, so an eclass change might not be
> correct in all cases.
> 
> In this case, I think it is required by CONFIG_IKHEADERS.

Our configs are common to all kernels.
Comment 6 Sam James archtester Gentoo Infrastructure gentoo-dev Security 2021-06-22 18:36:01 UTC
(In reply to Michał Górny from comment #5)
> (In reply to Mike Gilbert from comment #3)
> > It probably depends on the kernel config, so an eclass change might not be
> > correct in all cases.
> > 
> > In this case, I think it is required by CONFIG_IKHEADERS.
> 
> Our configs are common to all kernels.

We could make it conditional on savedconfig etc but I don't see the point, yeah.
Comment 7 Larry the Git Cow gentoo-dev 2021-06-22 18:36:29 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=d128c175a58caf79730022fc0cc8b6114ca2600f

commit d128c175a58caf79730022fc0cc8b6114ca2600f
Author:     David Michael <fedora.dm0@gmail.com>
AuthorDate: 2021-06-22 18:19:04 +0000
Commit:     Michał Górny <mgorny@gentoo.org>
CommitDate: 2021-06-22 18:35:52 +0000

    kernel-build.eclass: add cpio to BDEPEND
    
    When CONFIG_IKHEADERS is set, a script is run that requires cpio to
    be installed.  The default config files enable this setting.
    
    Closes: https://bugs.gentoo.org/797613
    Signed-off-by: David Michael <fedora.dm0@gmail.com>
    Signed-off-by: Michał Górny <mgorny@gentoo.org>

 eclass/kernel-build.eclass | 1 +
 1 file changed, 1 insertion(+)