[PATCH] samhain: rework due to changed oe cache handling


Armin Kuster
 

Max,

On 1/27/23 3:58 PM, Max Krummenacher wrote:
Am Freitag, dem 27.01.2023 um 16:14 +0000 schrieb Richard Purdie:
On Fri, 2023-01-27 at 13:30 +0100, Max Krummenacher wrote:
From: Max Krummenacher <max.krummenacher@...>

OE-Core changed the cache handling which made the use of ${BPN} fail.

Parsing recipes...WARNING: .../samhain-standalone.bb: Exception during build_dependencies for
do_configure
WARNING: .../samhain-standalone.bb: Error during finalise of .../samhain-standalone.bb
ERROR: ExpansionError during parsing .../samhain-standalone.bb
Traceback (most recent call last):
  File "Var <MODE_NAME>", line 1, in <module>
bb.data_smart.ExpansionError: Failure expanding variable MODE_NAME, expression was
${@d.getVar('BPN').split('-')[1]} which triggered exception IndexError: list index out of
range
The variable dependency chain for the failure is: MODE_NAME -> SAMHAIN_MODE -> do_configure
Simplify the setting of MODE_NAME and SAMHAIN_MODE by moving them from
the inc file to the recipe files where we know their values.

Relevant commits to produce the error:
bitbake:
ee89ade5 cache/codeparser: Switch to a new BB_CACHEDIR variable for cache location
oe-core:
7c15e03dd3 bitbake.conf: Add BB_HASH_CODEPARSER_VALS

Signed-off-by: Max Krummenacher <max.krummenacher@...>
---
 recipes-ids/samhain/samhain-client.bb     | 3 +++
 recipes-ids/samhain/samhain-server.bb     | 3 +++
 recipes-ids/samhain/samhain-standalone.bb | 3 +++
 recipes-ids/samhain/samhain.inc           | 8 --------
 4 files changed, 9 insertions(+), 8 deletions(-)
This is the second report I've seen of this. I'm wondering about
changing the default PN dummy value in bitbake.conf to include a "-"...

Cheers,

Richard
Richards patch [1] fixes the parsing error too. So we can drop this
one unless the simpler logic is worth taking it.

[1] https://lore.kernel.org/all/20230127163308.2649311-1-richard.purdie@linuxfoundation.org/
I am going the simpler logic so I will be taking your patch.

thanks,
Armin

Thanks Richard.

Regards
Max




Max Krummenacher <max.oss.09@...>
 

Am Freitag, dem 27.01.2023 um 16:14 +0000 schrieb Richard Purdie:
On Fri, 2023-01-27 at 13:30 +0100, Max Krummenacher wrote:
From: Max Krummenacher <max.krummenacher@...>

OE-Core changed the cache handling which made the use of ${BPN} fail.

Parsing recipes...WARNING: .../samhain-standalone.bb: Exception during build_dependencies for
do_configure
WARNING: .../samhain-standalone.bb: Error during finalise of .../samhain-standalone.bb
ERROR: ExpansionError during parsing .../samhain-standalone.bb
Traceback (most recent call last):
  File "Var <MODE_NAME>", line 1, in <module>
bb.data_smart.ExpansionError: Failure expanding variable MODE_NAME, expression was
${@d.getVar('BPN').split('-')[1]} which triggered exception IndexError: list index out of
range
The variable dependency chain for the failure is: MODE_NAME -> SAMHAIN_MODE -> do_configure
Simplify the setting of MODE_NAME and SAMHAIN_MODE by moving them from
the inc file to the recipe files where we know their values.

Relevant commits to produce the error:
bitbake:
ee89ade5 cache/codeparser: Switch to a new BB_CACHEDIR variable for cache location
oe-core:
7c15e03dd3 bitbake.conf: Add BB_HASH_CODEPARSER_VALS

Signed-off-by: Max Krummenacher <max.krummenacher@...>
---
 recipes-ids/samhain/samhain-client.bb     | 3 +++
 recipes-ids/samhain/samhain-server.bb     | 3 +++
 recipes-ids/samhain/samhain-standalone.bb | 3 +++
 recipes-ids/samhain/samhain.inc           | 8 --------
 4 files changed, 9 insertions(+), 8 deletions(-)
This is the second report I've seen of this. I'm wondering about
changing the default PN dummy value in bitbake.conf to include a "-"...

Cheers,

Richard
Richards patch [1] fixes the parsing error too. So we can drop this
one unless the simpler logic is worth taking it.

[1] https://lore.kernel.org/all/20230127163308.2649311-1-richard.purdie@linuxfoundation.org/

Thanks Richard.

Regards
Max


Richard Purdie
 

On Fri, 2023-01-27 at 13:30 +0100, Max Krummenacher wrote:
From: Max Krummenacher <max.krummenacher@...>

OE-Core changed the cache handling which made the use of ${BPN} fail.

Parsing recipes...WARNING: .../samhain-standalone.bb: Exception during build_dependencies for do_configure
WARNING: .../samhain-standalone.bb: Error during finalise of .../samhain-standalone.bb
ERROR: ExpansionError during parsing .../samhain-standalone.bb
Traceback (most recent call last):
File "Var <MODE_NAME>", line 1, in <module>
bb.data_smart.ExpansionError: Failure expanding variable MODE_NAME, expression was ${@d.getVar('BPN').split('-')[1]} which triggered exception IndexError: list index out of range
The variable dependency chain for the failure is: MODE_NAME -> SAMHAIN_MODE -> do_configure
Simplify the setting of MODE_NAME and SAMHAIN_MODE by moving them from
the inc file to the recipe files where we know their values.

Relevant commits to produce the error:
bitbake:
ee89ade5 cache/codeparser: Switch to a new BB_CACHEDIR variable for cache location
oe-core:
7c15e03dd3 bitbake.conf: Add BB_HASH_CODEPARSER_VALS

Signed-off-by: Max Krummenacher <max.krummenacher@...>
---
recipes-ids/samhain/samhain-client.bb | 3 +++
recipes-ids/samhain/samhain-server.bb | 3 +++
recipes-ids/samhain/samhain-standalone.bb | 3 +++
recipes-ids/samhain/samhain.inc | 8 --------
4 files changed, 9 insertions(+), 8 deletions(-)
This is the second report I've seen of this. I'm wondering about
changing the default PN dummy value in bitbake.conf to include a "-"...

Cheers,

Richard


Max Krummenacher <max.oss.09@...>
 

Hi

Please disregard this duplicate. Forgot to delete a backup copy before
git send-email.
Sorry about the noise.

Max


Max Krummenacher <max.oss.09@...>
 

From: Max Krummenacher <max.krummenacher@...>

OE-Core changed the cache handling which made the use of ${BPN} fail.

| Parsing recipes...WARNING: .../samhain-standalone.bb: Exception during build_dependencies for do_configure
| WARNING: .../samhain-standalone.bb: Error during finalise of .../samhain-standalone.bb
| ERROR: ExpansionError during parsing .../samhain-standalone.bb
| Traceback (most recent call last):
| File "Var <MODE_NAME>", line 1, in <module>
| bb.data_smart.ExpansionError: Failure expanding variable MODE_NAME, expression was ${@d.getVar('BPN').split('-')[1]} which triggered exception IndexError: list index out of range
| The variable dependency chain for the failure is: MODE_NAME -> SAMHAIN_MODE -> do_configure

Simplify the setting of MODE_NAME and SAMHAIN_MODE by moving them from
the inc file to the recipe files where we know their values.

Relevant commits to produce the error:
bitbake:
ee89ade5 cache/codeparser: Switch to a new BB_CACHEDIR variable for cache location
oe-core:
7c15e03dd3 bitbake.conf: Add BB_HASH_CODEPARSER_VALS

Signed-off-by: Max Krummenacher <max.krummenacher@...>
---
recipes-ids/samhain/samhain-client.bb | 3 +++
recipes-ids/samhain/samhain-server.bb | 3 +++
recipes-ids/samhain/samhain-standalone.bb | 3 +++
recipes-ids/samhain/samhain.inc | 8 --------
4 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/recipes-ids/samhain/samhain-client.bb b/recipes-ids/samhain/samhain-client.bb
index 2b99e20..0de9c34 100644
--- a/recipes-ids/samhain/samhain-client.bb
+++ b/recipes-ids/samhain/samhain-client.bb
@@ -8,5 +8,8 @@ EXTRA_OECONF += " \
--with-port=${SAMHAIN_PORT} \
"

+MODE_NAME = "client"
+SAMHAIN_MODE = "client"
+
RDEPENDS:${PN} = "acl zlib attr bash"
RCONFLICTS:${PN} = "samhain-standalone"
diff --git a/recipes-ids/samhain/samhain-server.bb b/recipes-ids/samhain/samhain-server.bb
index 51bce07..8bae2d2 100644
--- a/recipes-ids/samhain/samhain-server.bb
+++ b/recipes-ids/samhain/samhain-server.bb
@@ -8,6 +8,9 @@ SRC_URI += "file://samhain-server-volatiles \
file://samhain-server-volatiles.conf \
"

+MODE_NAME = "server"
+SAMHAIN_MODE = "server"
+
TARGET_CC_ARCH += "${LDFLAGS}"

do_install:append() {
diff --git a/recipes-ids/samhain/samhain-standalone.bb b/recipes-ids/samhain/samhain-standalone.bb
index b832dc8..ae9ed2b 100644
--- a/recipes-ids/samhain/samhain-standalone.bb
+++ b/recipes-ids/samhain/samhain-standalone.bb
@@ -7,6 +7,9 @@ SRC_URI += "file://samhain-not-run-ptest-on-host.patch \

PROVIDES += "samhain"

+MODE_NAME = "standalone"
+SAMHAIN_MODE = "no"
+
SYSTEMD_SERVICE:${PN} = "samhain.service"

inherit ptest
diff --git a/recipes-ids/samhain/samhain.inc b/recipes-ids/samhain/samhain.inc
index eb8592d..8967d72 100644
--- a/recipes-ids/samhain/samhain.inc
+++ b/recipes-ids/samhain/samhain.inc
@@ -40,14 +40,6 @@ SYSTEMD_PACKAGES = "${PN}"
SYSTEMD_SERVICE:${PN} = "${INITSCRIPT_NAME}.service"
SYSTEMD_AUTO_ENABLE = "disable"

-# mode mapping:
-# BPN MODE_NAME SAMHAIN_MODE
-# samhain-standalone standalone no
-# samhain-client client client
-# samhain-server server server
-MODE_NAME = "${@d.getVar('BPN').split('-')[1]}"
-SAMHAIN_MODE = "${@oe.utils.ifelse(d.getVar('MODE_NAME') == 'standalone', 'no', '${MODE_NAME}')}"
-
# supports mysql|postgresql|oracle|odbc but postgresql is the only one available

PACKAGECONFIG ??= "postgresql ps \
--
2.35.3