https://bugzilla.novell.com/show_bug.cgi?id=697643 https://bugzilla.novell.com/show_bug.cgi?id=697643#c1 Juergen Weigert changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO InfoProvider| |adrian@novell.com --- Comment #1 from Juergen Weigert 2011-06-07 20:49:48 UTC --- This is a designed, unfortunatly. The way osc rq show presents its output leads to several common misinterpretations. First, 'new' is not new at all, it means 'ready for checkin, reviews passed'. Second, the state next to the timestamp is what was requested, not what was done. E.g. you 'requested' a review by creating a new request. Then 4 minutes later licensedigger requested another manual review, which was finally done by myself. To add some more confusion, internal build service logic changes states and attributes the state to the person that triggered the logic: You did not explicitly request a review, but the build service logic decided that when creating the request for you. Also I did not explicitly set the state to new, the build service logic did that, when the last open review got approved, which happend to be me. Sufficiently confused now? The bug is not in the implementation. It correctly follows a design. Adrian, do you want to defend the design? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=697643 https://bugzilla.novell.com/show_bug.cgi?id=697643#c3 Sascha Peilicke changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED CC| |speilicke@novell.com InfoProvider|adrian@novell.com | Resolution| |WONTFIX --- Comment #3 from Sascha Peilicke 2011-06-17 10:43:27 UTC --- I'd say it's save to close this bug as WONTFIX :-) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.