Re: [PATCH] lts-cve-triage: allow to skip packages already in dla-needed.txt
On Mon, Jun 29, 2015 at 10:53:41PM +0200, Raphael Hertzog wrote:
> Hi,
>
> On Fri, 26 Jun 2015, Guido Günther wrote:
> > With lots of packages in dla-needed.txt it's easier to focus on CVEs of
> > packages that are not being worked on at all.
>
> Looks fine to me.
>
> > for pkg in tracker.iterate_packages():
> > + if args.skip_dla_needed and pkg in tracker.dla_needed.keys():
>
> Minor nitpick: The final .keys() is useless. "key in dict" does the same
> as "key in dict.keys()" in a more efficient way.
Old habits die hard. Applied with the nit fixed.
Cheers,
-- Guido
>
> Cheers,
> --
> Raphaël Hertzog ◈ Debian Developer
>
> Support Debian LTS: http://www.freexian.com/services/debian-lts.html
> Learn to master Debian: http://debian-handbook.info/get/
>
Reply to: