Bug 3350

Summary: Ensure smart supports hinting for RRECOMMENDS
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Paul Eggleton <bluelightning>
Component: coreAssignee: Paul Eggleton <bluelightning>
Status: VERIFIED FIXED QA Contact:
Severity: enhancement    
Priority: Medium+ CC: corneliux.stoicescu, meta.mr.watcher, meta.watcher, sgw
Version: 1.4   
Target Milestone: 1.4 M2   
Hardware: All   
OS: Multiple   
Whiteboard: Patches sent for review (4 Dec)
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: ---
Bug Depends on:    
Bug Blocks: 3182, 3351    

Description Paul Eggleton 2012-10-29 17:10:35 UTC
Ensure that smart can support the hint flags on dependencies between RPM packages that indicates that the relationship is a recommendation (RRECOMMENDS as opposed to RDEPENDS).
Comment 1 Paul Eggleton 2012-12-11 09:49:24 UTC
Implementation now merged to master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=69b9c6bd0c6f9c9884fa2d92b4062037457d06ad
Comment 2 Stoicescu Cornel 2013-02-08 13:27:12 UTC
This is not working as intended:

1) Adding "gzip" to RRECOMMENDS_${PN} meta/recipes-core/packagegroups/packagegroup-core-boot.bb adds gzip to a core-image-minimal but adding "gzippp" will result in an error:

ERROR: Nothing RPROVIDES 'gzippp' (but /home/cornel/SSD/poky/meta/recipes-core/packagegroups/packagegroup-core-boot.bb RDEPENDS on or otherwise requires it)

If I am correct, this should generate a warning, not an error.

2) Setting RDEPENDS_udev += " gzip" in local.conf adds gzip to a core-image-minimal but setting RRECOMMENDS_udev += " gzip" does not work.


Tested in master: b07759a20221de575da98d221feb9ea9295f7e6f
Comment 3 Paul Eggleton 2013-02-08 13:34:36 UTC
No, invalid package names as RRECOMMENDS are not allowed - there has to be something that RPROVIDES the recommend value even if it doesn't actually end up being satisfied at runtime; this has always been the case and has nothing to do with Smart.

Also, I don't think you should expect to be able to meaningfully set R* variables from local.conf. Please do your testing within the recipe itself.
Comment 4 Stoicescu Cornel 2013-02-08 14:39:28 UTC
Considering the error is not smart related and that RRECOMMENDS should not be used from local.conf, this bug is verified.

I will investigate and open a new bug for the error if it's needed.

Verified in master:8d6e55bf2192524bda84138b5356a2791adbbe61
Comment 5 Stoicescu Cornel 2013-04-30 10:27:53 UTC
Test case created: https://bugzilla.yoctoproject.org/tr_show_case.cgi?case_id=755