diff mbox series

[meta-openembedded,2/2] snort3: fix snort.pc

Message ID 20240224084239.121470-2-martin.jansa@gmail.com
State Accepted
Headers show
Series [meta-openembedded,1/2] snort3: drop SRCPV from PV | expand

Commit Message

Martin Jansa Feb. 24, 2024, 8:42 a.m. UTC
* fixes:
  ERROR: lib32-snort3-3+git-r0 do_populate_sysroot: QA Issue: snort.pc failed sanity test (tmpdir) in path lib32-snort3/3+git/sysroot-destdir/usr/lib/pkg

* it's broken for non-multilib builds as well, the issue is that
  FLEX_CPPFLAGS points to native include dir, e.g.
  FLEX_CPPFLAGS=-I/OE/../lib32-snort3/3+git/recipe-sysroot-native/usr/include
  and the work around from:
  https://github.com/openembedded/meta-openembedded/commit/973647848049abda6f2adbee0189720109863731
  sed -i "s#${RECIPE_SYSROOT}##g" ${D}${libdir}/pkgconfig/snort.pc
  strips the "/OE/../lib32-snort3/3+git/recipe-sysroot" part in non-multilib
  case, but leaves:
  FLEX_CPPFLAGS=-I-native/usr/include
  which is still wrong, but not detected by buildpaths QA check anymore
  and in multilib case, this didn't strip the first part because the
  target sysroot is:
  "/OE/../lib32-snort3/3+git/lib32-recipe-sysroot"
  so it didn't strip anything from native sysroot:
  "/OE/../lib32-snort3/3+git/recipe-sysroot-native"

Signed-off-by: Martin Jansa <martin.jansa@gmail.com>
---
 meta-networking/recipes-connectivity/snort/snort3_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/meta-networking/recipes-connectivity/snort/snort3_git.bb b/meta-networking/recipes-connectivity/snort/snort3_git.bb
index e9c37c8346..b8826b0682 100644
--- a/meta-networking/recipes-connectivity/snort/snort3_git.bb
+++ b/meta-networking/recipes-connectivity/snort/snort3_git.bb
@@ -21,7 +21,7 @@  PACKAGES =+ "${PN}-scripts"
 inherit cmake pkgconfig
 
 do_install:append() {
-    sed -i "s#${RECIPE_SYSROOT}##g" ${D}${libdir}/pkgconfig/snort.pc
+    sed -i "s#${RECIPE_SYSROOT_NATIVE}##g; s#${RECIPE_SYSROOT}##g" ${D}${libdir}/pkgconfig/snort.pc
 }
 
 FILES:${PN} += "${libdir}/snort/daq/*.so"