Re: [DistroKit] [PATCH v3 4/4] ptxconfig: create menu for DistroKit-specific rules

[Alexander Dahl <ada@thorsis.com>]

Hello Roland,

Am Wed, Jun 19, 2024 at 02:32:37PM +0200 schrieb Roland Hieber:
> This way the main menu doesn't get cluttered up with random recipes.
> 
> Link: https://lore.distrokit.org/distrokit/20240613-whenever-enduring-698a9b93370b@thorsis.com/
> Suggested-by: Alexander Dahl <ada@thorsis.com>
> Reviewed-by: Michael Olbrich <m.olbrich@pengutronix.de>
> Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> ---
>  configs/ptxconfig      | 5 +++++
>  rules/distrokit.in     | 5 +++++
>  rules/rauc-udev.in     | 2 +-
>  rules/repart-config.in | 2 +-
>  4 files changed, 12 insertions(+), 2 deletions(-)
>  create mode 100644 rules/distrokit.in
> 
> diff --git a/configs/ptxconfig b/configs/ptxconfig
> index c7cdb62e16a5..fda7d198e6de 100644
> --- a/configs/ptxconfig
> +++ b/configs/ptxconfig
> @@ -2,8 +2,13 @@
>  # Automatically generated file; DO NOT EDIT.
>  # PTXdist 2024.05.0
>  #
> +
> +#
> +# DistroKit                     
> +#
>  PTXCONF_RAUC_UDEV=y
>  PTXCONF_REPART_CONFIG=y
> +# end of DistroKit                     
>  
>  #
>  # ------------------------------------
> diff --git a/rules/distrokit.in b/rules/distrokit.in
> new file mode 100644
> index 000000000000..559a5bde0ecd
> --- /dev/null
> +++ b/rules/distrokit.in
> @@ -0,0 +1,5 @@
> +## SECTION=project_specific
> +
> +menu "DistroKit                     "
> +source "generated/distrokit.in"
> +endmenu
> diff --git a/rules/rauc-udev.in b/rules/rauc-udev.in
> index f960c537f58f..d18f1c3197d5 100644
> --- a/rules/rauc-udev.in
> +++ b/rules/rauc-udev.in
> @@ -1,4 +1,4 @@
> -## SECTION=project_specific
> +## SECTION=distrokit
>  
>  config RAUC_UDEV
>  	tristate
> diff --git a/rules/repart-config.in b/rules/repart-config.in
> index db9828f669f2..ea803c015978 100644
> --- a/rules/repart-config.in
> +++ b/rules/repart-config.in
> @@ -1,4 +1,4 @@
> -## SECTION=project_specific
> +## SECTION=distrokit
>  
>  config REPART_CONFIG
>  	tristate "repart-config"

Looks good overall, so …

Reviewed-by: Alexander Dahl <ada@thorsis.com>

Greets
Alex