1 |
On Wed, 2020-06-17 at 12:57 +0200, Ulrich Mueller wrote: |
2 |
> > > > > > On Wed, 17 Jun 2020, Michał Górny wrote: |
3 |
> |
4 |
> > +# @FUNCTION: kernel-install_pkg_pretend |
5 |
> > +# @DESCRIPTION: |
6 |
> > +# Check for missing optional dependencies and output warnings. |
7 |
> > +kernel-install_pkg_pretend() { |
8 |
> > + debug-print-function ${FUNCNAME} "${@}" |
9 |
> > + |
10 |
> > + if ! has_version -d sys-kernel/linux-firmware; then |
11 |
> > + ewarn "sys-kernel/linux-firmware not found installed on your system." |
12 |
> > + ewarn "This package provides various firmware files that may be needed" |
13 |
> > + ewarn "for your hardware to work. If in doubt, it is recommended" |
14 |
> > + ewarn "to pause or abort the build process and install it before" |
15 |
> > + ewarn "resuming." |
16 |
> > + |
17 |
> > + if use initramfs; then |
18 |
> > + elog |
19 |
> > + elog "If you decide to install linux-firmware later, you can rebuild" |
20 |
> > + elog "the initramfs via issuing a command equivalent to:" |
21 |
> > + elog |
22 |
> > + elog " emerge --config ${CATEGORY}/${PN}" |
23 |
> > + fi |
24 |
> > + fi |
25 |
> > +} |
26 |
> |
27 |
> Should we really warn about a package that (in its default |
28 |
> configuration) can only be installed if the user accepts non-free |
29 |
> licenses? |
30 |
|
31 |
That's one of the reasons it's only a warning and not a USE flag. |
32 |
|
33 |
> I would think that even without such a warning, users will be well aware |
34 |
> if some devices of their system will need additional firmware. Also, |
35 |
> some people prefer the separate packages from sys-firmware which tend to |
36 |
> be more lightweight (though I am aware that some of them may be |
37 |
> considered legacy packages). |
38 |
> |
39 |
|
40 |
This has been requested by users, some of whom apparently forget that |
41 |
they need to manually install one more package for their system to even |
42 |
boot. If it saves a few people from having to go through recovery, it's |
43 |
worth it. |
44 |
|
45 |
-- |
46 |
Best regards, |
47 |
Michał Górny |