[PATCH] meta/classes-recipe/dracut-module: drop redundant bb import in python function

[srinuv...@siemens.com]

From: Srinuvasan A <srinuv...@siemens.com>

The BitBake `bb` module is implicitly available in Python functions,
so the explicit import in `replace_marker_with_file_content()` is
unnecessary. Remove the redundant import to align with BitBake
Python function style guidelines.

Ref: https://docs.yoctoproject.org/bitbake/singleindex.html#bitbake-style-python-functions

Signed-off-by: Srinuvasan A <srinuv...@siemens.com>
---
meta/classes-recipe/dracut-module.bbclass | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes-recipe/dracut-module.bbclass b/meta/classes-recipe/dracut-module.bbclass
index e2ed52ac..77521309 100644
--- a/meta/classes-recipe/dracut-module.bbclass
+++ b/meta/classes-recipe/dracut-module.bbclass
@@ -37,7 +37,7 @@ def add_file_if_variable_is_set(d, variable_name, prefix):
return ''

def replace_marker_with_file_content(template_file, content_file, marker):
- import re, bb
+ import re

tmpl = open(template_file).read()
content = open(content_file).read().rstrip('\n')
--
2.34.1

[MOESSBAUER, Felix]

On Tue, 2026-01-06 at 12:39 +0530, srinuvasan.a via isar-users wrote:
> From: Srinuvasan A <srinuv...@siemens.com>
>
> The BitBake `bb` module is implicitly available in Python functions,
> so the explicit import in `replace_marker_with_file_content()` is
> unnecessary. Remove the redundant import to align with BitBake
> Python function style guidelines.

We need a linter to automatically find these cases :)

Reviewed-by: Felix Moessbauer <felix.mo...@siemens.com>

>
> Ref: https://docs.yoctoproject.org/bitbake/singleindex.html#bitbake-style-python-functions
>
> Signed-off-by: Srinuvasan A <srinuv...@siemens.com>
> ---
> meta/classes-recipe/dracut-module.bbclass | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/classes-recipe/dracut-module.bbclass b/meta/classes-recipe/dracut-module.bbclass
> index e2ed52ac..77521309 100644
> --- a/meta/classes-recipe/dracut-module.bbclass
> +++ b/meta/classes-recipe/dracut-module.bbclass
> @@ -37,7 +37,7 @@ def add_file_if_variable_is_set(d, variable_name, prefix):
> return ''
>
> def replace_marker_with_file_content(template_file, content_file, marker):
> - import re, bb
> + import re
>
> tmpl = open(template_file).read()
> content = open(content_file).read().rstrip('\n')
> --
> 2.34.1
>

> --
> You received this message because you are subscribed to the Google Groups "isar-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+...@googlegroups.com.
> To view this discussion visit https://groups.google.com/d/msgid/isar-users/20260106070939.50797-1-srinuvasan.a%40siemens.com.

--
Siemens AG
Linux Expert Center
Friedrich-Ludwig-Bauer-Str. 3
85748 Garching, Germany

[Anton Mikanovich]

06/01/2026 09:09, srinuvasan.a via isar-users wrote:
> From: Srinuvasan A <srinuv...@siemens.com>
>
> The BitBake `bb` module is implicitly available in Python functions,
> so the explicit import in `replace_marker_with_file_content()` is
> unnecessary. Remove the redundant import to align with BitBake
> Python function style guidelines.
>
> Ref: https://docs.yoctoproject.org/bitbake/singleindex.html#bitbake-style-python-functions
>
> Signed-off-by: Srinuvasan A <srinuv...@siemens.com>

Applied to next, thanks.