Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 527304 - sys-apps/debianutils: please make 'installkernel' executable optional
Summary: sys-apps/debianutils: please make 'installkernel' executable optional
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: Normal critical (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-29 09:04 UTC by Michał Górny
Modified: 2018-01-02 14:45 UTC (History)
0 users

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


Attachments
Suggested patch (debianutils-4.4-r1.ebuild.diff,1.39 KB, patch)
2014-10-29 09:22 UTC, Michał Górny
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2014-10-29 09:04:39 UTC
I would like to provide an alternative implementation of /sbin/installkernel, so I would appreciate if the debianutils one could be made USE-optional.
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2014-10-29 09:22:28 UTC
Created attachment 387706 [details, diff]
Suggested patch

This one introduces controllable USE=installkernel to toggle it. I've removed USE=kernel_linux references because they are redundant then. We also need to p.u.mask:

  sys-apps/debianutils installkernel

in default/bsd and prefix profiles, and un-p.u.mask in prefix/linux.
Comment 2 SpanKY gentoo-dev 2016-12-05 22:26:44 UTC
you want to provide an ebuild that installs /sbin/installkernel ?  or you want to, on your own systems, install some local script ?

the kernel build looks in ~/bin/ first.
Comment 3 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2016-12-05 23:20:13 UTC
Yes, I wanted to provide an ebuild with it. To install in gummiboot-compliant location.
Comment 4 SpanKY gentoo-dev 2016-12-06 03:49:59 UTC
your patch/idea looks fine.  feel free to rebase/push :).
Comment 5 Larry the Git Cow gentoo-dev 2018-01-02 13:32:59 UTC
The bug has been closed via the following commit(s):

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

commit 8845fb86370df98660919bfad9445df896322356
Author:     Lars Wendler <polynomial-c@gentoo.org>
AuthorDate: 2018-01-02 13:32:39 +0000
Commit:     Lars Wendler <polynomial-c@gentoo.org>
CommitDate: 2018-01-02 13:32:54 +0000

    sys-apps/debianutils: Bump to version 4.8.4
    
    Use "installkernel" USE flag to control installation of installkernel
    shell script.
    
    Closes: https://bugs.gentoo.org/527304
    Package-Manager: Portage-2.3.19, Repoman-2.3.6

 sys-apps/debianutils/Manifest                 |  1 +
 sys-apps/debianutils/debianutils-4.8.4.ebuild | 41 +++++++++++++++++++++++++++
 sys-apps/debianutils/metadata.xml             | 11 ++++---
 3 files changed, 49 insertions(+), 4 deletions(-)
Comment 6 Jason A. Donenfeld gentoo-dev 2018-01-02 14:43:28 UTC
This USE flag should be +installkernel. Otherwise it will break many existing setups that rely on this behavior. Notably, this breaks existing handbook advice that says to call `make install` on the kernel tree.
Comment 7 Jason A. Donenfeld gentoo-dev 2018-01-02 14:45:51 UTC
Whoops looks like it already is and something else is happening. Sorry for the noise. Reclosing.