15563 2024-07-24 15:06:34 +0000 Building a recipe that inherits cpan-base creates an image runtime dependency on perl 2024-08-05 14:37:20 +0000 1 1 7 Build System, Metadata & Runtime OE-Core oe-core other 5.0 x86 Multiple RESOLVED INVALID Medium+ minor 5.1 M3 1 mr.scada mr.scada randy.macleod ross.burton tim.orling --- --- No (bug/feature does not impact docs) oldest_to_newest 99429 0 5061 mr.scada 2024-07-24 15:06:34 +0000 Created attachment 5061 Patch to break hard runtime dependency on perl Hello, I build an image with Postgresql in it via the meta-openembedded recipe: https://github.com/openembedded/meta-openembedded/blob/scarthgap/meta-oe/recipes-dbs/postgresql/postgresql.inc#L40 said recipe inherits cpan-base. cpan-base.bbclass (https://git.yoctoproject.org/poky/tree/meta/classes-recipe/cpan-base.bbclass?h=scarthgap#n14) adds an RDEPEND for perl to the non-native package. This results in perl being installed in my rootfs, which is not desired for CVEs, image size, etc. I have 'PACKAGECONFIG:remove = "perl"' in my local.conf and I've verified that postgresql itself is actually configuring with the flag --without-perl, but postgresql.do_prepare_recipe_sysroot adds a dependency on perl. I've created an attached patch (0001-cpan-base-Only-RDEPEND-on-perl-if-using-perl.patch) that limits the perl runtime dependency based on the PACKAGECONFIG of the package that inherits cpan-base, but I'm not certain such action is wise. Is there a better way to have postgresql in my image and not have perl? Thanks! 99435 1 randy.macleod 2024-07-25 14:44:54 +0000 The patch is a good start, Ross is going to comment on how to proceed. This isn't a oe-core recipe so since the changes are likely only to the recipe, we're going to reject the bug for now. 99437 2 ross.burton 2024-07-25 14:48:17 +0000 You can't check a PACKAGECONFIG in a base class unless its a well known and defined PACKAGECONFIG, which 'perl' is not. The cpan class is designed for recipes that fetch perl modules from cpan, so that class depending on perl makes sense. I can't see why the postgresql recipe actually pulls that class in at all. Would you be able to test a build of postgresql with perl enabled, but change the cpan-base inherit to perl-version? 99442 3 tim.orling 2024-07-25 14:54:10 +0000 I'd suggest removing cpan-base from: https://git.openembedded.org/meta-openembedded/tree/meta-oe/recipes-dbs/postgresql/postgresql.inc#n40 And add a line where the inherit conditionally happens if the perl PACKAGECONFIG is set. And please send patches to the mailing list: https://git.openembedded.org/meta-openembedded/tree/meta-oe/README.md 99446 4 mr.scada 2024-07-25 15:01:44 +0000 (In reply to Ross Burton from comment #2) > You can't check a PACKAGECONFIG in a base class unless its a well known and > defined PACKAGECONFIG, which 'perl' is not. Thanks for the explanation. I figured patch was sketchy. > The cpan class is designed for recipes that fetch perl modules from cpan, so > that class depending on perl makes sense. > > I can't see why the postgresql recipe actually pulls that class in at all. > Would you be able to test a build of postgresql with perl enabled, but > change the cpan-base inherit to perl-version? Swapping cpan-base to perl-version in the inherit leads to: Traceback (most recent call last): File "Var <PERLCONFIGTARGET>", line 1, in <module> bb.data_smart.ExpansionError: Failure expanding variable PERLCONFIGTARGET, expression was ${@is_target(d)} which triggered exception NameError: name 'is_target' is not defined The variable dependency chain for the failure is: PERLCONFIGTARGET Since they call that is_target function on https://github.com/openembedded/meta-openembedded/blob/scarthgap/meta-oe/recipes-dbs/postgresql/postgresql.inc#L126 99493 5 mr.scada 2024-08-05 14:37:20 +0000 Just for administrative housekeeping; I submitted a patch to meta-openembedded: https://github.com/openembedded/meta-openembedded/commit/f311f1f01c19924d8cced61fa915f1387f92455b Thanks! 5061 2024-07-24 15:06:34 +0000 2024-07-24 15:06:34 +0000 Patch to break hard runtime dependency on perl 0001-cpan-base-Only-RDEPEND-on-perl-if-using-perl.patch text/plain 1158 mr.scada RnJvbSAwZmE1M2M5MzUyOWE3NGFjYTc2M2VkNzU4OGQ3NjRmOGU2Mzc5YjY3IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBCYXJyeSBHcnVzc2xpbmcgPG1yLnNjYWRhQGdtYWlsLmNvbT4K RGF0ZTogVHVlLCAyMyBKdWwgMjAyNCAwODoxMzoyNCAtMDcwMApTdWJqZWN0OiBbUEFUQ0hdIGNw YW4tYmFzZTogT25seSBSREVQRU5EIG9uIHBlcmwgaWYgdXNpbmcgcGVybAoKU29tZSByZWNpcGVz IGxpa2UgcG9zdGdyZXNxbCBpbmhlcml0IGNwYW4tYmFzZSBidXQgZG9uJ3QgbmVjZXNzYXJpbHkK d2FudCBhIHJ1bi10aW1lIGRlcGVuZGVuY3kgb24gcGVybC4gVGhpcyBjaGFuZ2VzZXQgbG9va3Mg YXQgdGhlClBBQ0tBR0VDT05GSUcgdG8gZGV0ZXJtaW5lIGlmIGEgcGVybCBSREVQRU5EIGlzIGFw cHJvcHJpYXRlLgoKU2lnbmVkLW9mZi1ieTogQmFycnkgR3J1c3NsaW5nIDxtci5zY2FkYUBnbWFp bC5jb20+Ci0tLQogbWV0YS9jbGFzc2VzLXJlY2lwZS9jcGFuLWJhc2UuYmJjbGFzcyB8IDIgKy0K IDEgZmlsZSBjaGFuZ2VkLCAxIGluc2VydGlvbigrKSwgMSBkZWxldGlvbigtKQoKZGlmZiAtLWdp dCBhL21ldGEvY2xhc3Nlcy1yZWNpcGUvY3Bhbi1iYXNlLmJiY2xhc3MgYi9tZXRhL2NsYXNzZXMt cmVjaXBlL2NwYW4tYmFzZS5iYmNsYXNzCmluZGV4IDFkYjBhNGRlZDYuLjRlN2NlOGQ4OGQgMTAw NjQ0Ci0tLSBhL21ldGEvY2xhc3Nlcy1yZWNpcGUvY3Bhbi1iYXNlLmJiY2xhc3MKKysrIGIvbWV0 YS9jbGFzc2VzLXJlY2lwZS9jcGFuLWJhc2UuYmJjbGFzcwpAQCAtMTEsNyArMTEsNyBAQAogRklM RVM6JHtQTn0gKz0gIiR7bGliZGlyfS9wZXJsNSAke2RhdGFkaXJ9L3Blcmw1IgogCiBERVBFTkRT ICArPSAiJHtAWyJwZXJsIiwgInBlcmwtbmF0aXZlIl1bKGJiLmRhdGEuaW5oZXJpdHNfY2xhc3Mo J25hdGl2ZScsIGQpKV19IgotUkRFUEVORFM6JHtQTn0gKz0gIiR7QFsicGVybCIsICIiXVsoYmIu ZGF0YS5pbmhlcml0c19jbGFzcygnbmF0aXZlJywgZCkpXX0iCitSREVQRU5EUzoke1BOfSArPSAi JHtAW2JiLnV0aWxzLmZpbHRlcignUEFDS0FHRUNPTkZJRycsICdwZXJsJywgZCksICIiXVsoYmIu ZGF0YS5pbmhlcml0c19jbGFzcygnbmF0aXZlJywgZCkpXX0iCiAKIGluaGVyaXQgcGVybC12ZXJz aW9uCiAKLS0gCjIuNDUuMgoK