On Mon, Mar 30, 2015 at 03:48:23PM +0200, Petr Šabata wrote:
> On Mon, Mar 30, 2015 at 09:39:31AM -0400, Paul Wouters wrote:
> > On Mon, 30 Mar 2015, Petr Šabata wrote:
> > 
> > >>  Bug 815617: PATCH: properly deal with crypt() returning NULL (1043 days 
> > >> old)
> > >>    https://bugzilla.redhat.com/show_bug.cgi?id=815617
> > >>    https://bugzilla.redhat.com/attachment.cgi?id=585827&action=edit
> > >>
> > >>This bug is already closed. And has no "flags" set. In the past someone
> > >>set the review flag, which i think is why this is showing up for me. But
> > >>I cannot get rid of it.
> > >>
> > >>I have a few more of these in the review category. How can I get rid of
> > >>these?
> > >>
> > >>Paul
> > >
> > >msekleta has asked you, specifically, to review the patch.
> > >Perhaps setting the patch review flag to `+' would help.
> > 
> > Yes, on  2012-05-21 the review flag was set to me.
> > On 2013-07-04, Tomas Hozza set the review flag to +
> > I closed the bug on 2013-07-08.
> > 
> > There are currently no flags set at all.
> > 
> > Paul
> 
> As I understand it you were asked for input on the patch which
> you haven't provided yet.  The review(pwouters) flag is still
> set to `?'.  thozza's review+ didn't change that.
> 
> P

Shouldn't the bug being closed superseed the flags though?  Whats the point of
reviewing something thats already been resolved?

I ask because I get these on occasion too, and don't really want to waste time
with busywork setting flags on closed bugs.
Neil



> -- 
> devel mailing list
> devel@lists.fedoraproject.org
> https://admin.fedoraproject.org/mailman/listinfo/devel
> Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

-- 
devel mailing list
devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/devel
Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct

Reply via email to