Bug 5088

Summary: [OE-Core]Buildhistory fails to report error if PR of some recipes go backwards
Product: [Build System, Metadata & Runtime] OE-Core Reporter: Alexandru Palalau <alexandrux.palalau>
Component: coreAssignee: Paul Eggleton <bluelightning>
Status: VERIFIED FIXED QA Contact:
Severity: normal    
Priority: Medium CC: meta.mr.watcher, meta.watcher
Version: 1.5   
Target Milestone: 1.5 M5   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: Regression (Used to work)
Verified: Documentation change: Yes (doc changes required)

Description Alexandru Palalau 2013-08-30 10:53:03 UTC
STATUS: FAILED
BUILD: 1.5: 5745e45b18e5099e94b4d5a73bc97dc6d4cdc91f
ENVIRONMENT: Multiple environments
STEPS TO REPRODUCE: 1. build some recipes and get the buildhistory log(for example, recipe "man")
2. change the PR of man backwards, for example from "r1" to "r0"
3. re-build the recipe

EXPECTED OUTCOME: pkghistory reports error if PR of some recipes go backwards .

Build Configuration:
BB_VERSION        = "1.19.1"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "Fedora-18"
TARGET_SYS        = "x86_64-poky-linux"
MACHINE           = "qemux86-64"
DISTRO            = "poky"
DISTRO_VERSION    = "1.4+snapshot-20130830"
TUNE_FEATURES     = "m64"
TARGET_FPU        = ""
meta              
meta-yocto        
meta-yocto-bsp    = "master:5745e45b18e5099e94b4d5a73bc97dc6d4cdc91f"

The actual outcome is that the package is successfully built without any warning or error on PR change.
Comment 1 Paul Eggleton 2013-09-03 10:13:17 UTC
This is because this check was changed to use "version-going-backwards" in WARN_QA or ERROR_QA in order to enable the check, and this has not been added to poky.conf. I've sent a patch to do this.
Comment 2 Paul Eggleton 2013-09-04 10:18:40 UTC
Fix merged to master:

http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=8318816e27040dabe1db18aba50075128648df19

In case it's not clear, this issue only affected master, previous release versions are unaffected.
Comment 3 Alexandru Palalau 2013-09-04 13:58:25 UTC
Verified on master branch.
Commit ID:f1c74a33835f97c6b42235358d80851882b4687e