Code review comment for lp:~mabac/svammel/auto-close-retried

Revision history for this message
Steve Langasek (vorlon) wrote :

> By "closing" I mean setting the status of all connected
> bug_tasks to "Invalid". There may well be a better
> implementation of closing bugs, please let me know.

Optimally, a bug that's being closed because the package has subsequently built successfully in the same package version would be closed as "invalid" and one closed in a later package version would be "Fix released"; but that's nitpicking on my part.

More importantly, we should only be closing bug tasks corresponding to the package the bug was originally reported on. If a bug report gets addressed in the current package by applying a workaround, but a new task has been opened on, for example, gcc-linaro to flag that the compiler needs a fix for the underlying issue, we shouldn't touch that task.

review: Needs Fixing

« Back to merge proposal