Bug 5088 - [OE-Core]Buildhistory fails to report error if PR of some recipes go backwards
Summary: [OE-Core]Buildhistory fails to report error if PR of some recipes go backwards
Status: VERIFIED FIXED
Alias: None
Product: OE-Core
Classification: Build System, Metadata & Runtime
Component: core (show other bugs)
Version: 1.5
Hardware: x86 Multiple
: Medium normal
Target Milestone: 1.5 M5
Assignee: Paul Eggleton
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2013-08-30 10:53 UTC by Alexandru Palalau
Modified: 2013-09-04 13:58 UTC (History)
2 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: Regression (Used to work)
Verified:
Documentation change: Yes (doc changes required)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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