Quoting Sascha Peilicke
Am 24. Januar 2013 01:51:49 schrieb Cristian Rodríguez
: El 23/01/13 10:09, Stephan Kulow escribió:
We're at 47 problems now -
I am trying to do my part getting rid of part of this 47 problems, but it seems insanity has taken over the review team, the only thing missing now is a grammar police with the cops knocking at my door.
So sad that this absurd nitpicking now takes precedence over getting stuff building and running.
I just give up and somebody else with patience can take over.
Christian, whats the issue here? I suspect you got a review declined for disputable reasons? Generally it is worth discussing this either with the reviewer or the review@ ML. Reviewers can't know everything and they (I should say we) make mistakes. And at times, they receive a lot of heat for that :-)
Very much assuming that the issue is #149730: this was actually declined BEFORE it reached the review team. It's the games maintainer which refused to accept a patch that hides much more inside than what is documented. It is not exactly the first time such patches are being submitted, claiming to do one thing and in the same patch doing much more.
But I assume the intent was to respect our policies and to keep the packaging quality high (and we achieved quite a bit with regards to that). Of course that can collide with the short term goal of fixing Factory build errors ASAP. So I would recommend solving this with the reviewer directly. Or you explain the issue here (neutrally) and leave the reviewer a chance to clarify, so that we all can learn something.
'Fixing' Factory should not go directly on cost of packaging quality... Those two are not mutually exclusive, if people respect them to start with... Dominique -- To unsubscribe, e-mail: opensuse-factory+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-factory+owner@opensuse.org