From: | Zac Medico <zmedico@g.o> | ||
---|---|---|---|
To: | "Michał Górny" <mgorny@g.o>, gentoo-portage-dev@l.g.o, Alexander Berntsen <bernalex@g.o> | ||
Subject: | Re: [gentoo-portage-dev] How do we want to deal with Arfrever's bunch of patches? | ||
Date: | Sun, 13 Dec 2015 21:35:01 | ||
Message-Id: | 566DE479.9060102@gentoo.org | ||
In Reply to: | Re: [gentoo-portage-dev] How do we want to deal with Arfrever's bunch of patches? by Zac Medico |
1 | On 12/13/2015 08:58 AM, Zac Medico wrote: |
2 | > On 12/13/2015 04:58 AM, Michał Górny wrote: |
3 | >>>> 39d81c5 portage.package.ebuild.config.config.__getitem__(): Partially drop backward compatibility for nonexistent keys. |
4 | >>>> |
5 | >>>> Maybe keep it but needs someone smarter than me to review. |
6 | >>> |
7 | >>> Looks good, except the last hunk seems redundant because "for x in self" |
8 | >>> should only yield valid keys: |
9 | >>> |
10 | >>> @@ -2697,7 +2714,9 @@ class config(object): |
11 | >>> for x in self: |
12 | >>> if x in environ_filter: |
13 | >>> continue |
14 | >>> - myvalue = self[x] |
15 | >>> + myvalue = self.get(x) |
16 | >>> + if myvalue is None: |
17 | >>> + continue |
18 | >> |
19 | >> Could you fix it then, please? |
20 | > |
21 | > Fixed: |
22 | > |
23 | > https://gitweb.gentoo.org/proj/portage.git/commit/?id=6ba56ad7be84b18dcbf15e8c6b283f5a9a559123 |
24 | > |
25 | |
26 | I've also fixed a blanket except clause from the same commit, so it will |
27 | raise BaseException (like SystemExit and KeyboardInterrupt): |
28 | |
29 | https://gitweb.gentoo.org/proj/portage.git/commit/?id=ee7978914f27c6a48cd1d6ee2667470aed25687f |
30 | |
31 | -- |
32 | Thanks, |
33 | Zac |
Subject | Author |
---|---|
Re: [gentoo-portage-dev] How do we want to deal with Arfrever's bunch of patches? | Alexander Berntsen <bernalex@g.o> |