docs: make DISTRO_FEATURES description more explicit

Message ID 20220623173710.12957-1-aatrapps@gmail.com
State New
Headers show
Series docs: make DISTRO_FEATURES description more explicit | expand

Commit Message

Aatir Manzur June 23, 2022, 5:37 p.m. UTC
Make DISTRO_FEATURES description more explicit by specifying
that DISTRO_FEATURES alone can't select build configurations,
and need mechanisms like PACKAGECONFIG setup to take care of
selecting build configurations.

Signed-off-by Aatir Manzur <aatrapps@gmail.com>
---
 documentation/ref-manual/features.rst  | 4 +++-
 documentation/ref-manual/variables.rst | 5 ++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

Comments

Michael Opdenacker June 23, 2022, 6:08 p.m. UTC | #1
Hi Aatir,

Thanks for the update!

On 6/23/22 19:37, aatir wrote:
> +      enabled. Note: just enabling :term: `DISTRO_FEATURES` alone doesn't

Oops, you forgot to remove the space here. I fixed that by myself.

> +      enable feature support for packages, mechanisms such as making
> +      :term:`PACKAGECONFIG` track :term:`DISTRO_FEATURES` are used
> +      to enable/disbale package features.


s/disbale/disable/
I fixed that by myself

Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>

I leave a little of time for further reviews, but it looks good to me.
Thanks again, and don't hesitate to send us more improvements!

Cheers
Michael.
Aatir Manzur June 23, 2022, 6:16 p.m. UTC | #2
Great, thanks!

On Thu, Jun 23, 2022 at 2:08 PM Michael Opdenacker <
michael.opdenacker@bootlin.com> wrote:

> Hi Aatir,
>
> Thanks for the update!
>
> On 6/23/22 19:37, aatir wrote:
> > +      enabled. Note: just enabling :term: `DISTRO_FEATURES` alone
> doesn't
>
> Oops, you forgot to remove the space here. I fixed that by myself.
>
> > +      enable feature support for packages, mechanisms such as making
> > +      :term:`PACKAGECONFIG` track :term:`DISTRO_FEATURES` are used
> > +      to enable/disbale package features.
>
>
> s/disbale/disable/
> I fixed that by myself
>
> Reviewed-by: Michael Opdenacker <michael.opdenacker@bootlin.com>
>
> I leave a little of time for further reviews, but it looks good to me.
> Thanks again, and don't hesitate to send us more improvements!
>
> Cheers
> Michael.
>
> --
> Michael Opdenacker, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
>
>

Patch

diff --git a/documentation/ref-manual/features.rst b/documentation/ref-manual/features.rst
index a8d0dac99..17521ac2a 100644
--- a/documentation/ref-manual/features.rst
+++ b/documentation/ref-manual/features.rst
@@ -100,7 +100,9 @@  packages, and they can go beyond simply controlling the installation of
 a package or packages. In most cases, the presence or absence of a
 feature translates to the appropriate option supplied to the configure
 script during the :ref:`ref-tasks-configure` task for
-the recipes that optionally support the feature.
+the recipes that optionally support the feature. Appropriate options
+must be supplied, and enabling/disabling :term:`PACKAGECONFIG` for the
+concerned packages is one way of supplying such options.
 
 Some distro features are also machine features. These select features
 make sense to be controlled both at the machine and distribution
diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
index 28267101d..8f7b01c12 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -1873,7 +1873,10 @@  system and gives an overview of their function and contents.
       optionally support the feature. For example, specifying "x11" in
       :term:`DISTRO_FEATURES`, causes every piece of software built for the
       target that can optionally support X11 to have its X11 support
-      enabled.
+      enabled. Note: just enabling :term: `DISTRO_FEATURES` alone doesn't
+      enable feature support for packages, mechanisms such as making
+      :term:`PACKAGECONFIG` track :term:`DISTRO_FEATURES` are used
+      to enable/disbale package features.
 
       Two more examples are Bluetooth and NFS support. For a more complete
       list of features that ships with the Yocto Project and that you can