Re: [PATCH] Eliminate the most of explicit py3 conditionals

Zac Medico <[email protected]>
Newsgroups gmane.linux.gentoo.portage.devel
Message-ID <[email protected]>
On 7/16/20 5:18 AM, Michał Górny wrote:
> Eliminate the most of py2/py3 conditions in the code.  Leave a few
> where the relevant code is unclear, they will be addressed later.
> 
> Closes: https://github.com/gentoo/portage/pull/574
> Signed-off-by: Michał Górny <[email protected]>
> ---
>  bin/check-implicit-pointer-usage.py           | 25 ++--------
>  bin/chmod-lite.py                             | 11 ++---
>  bin/dohtml.py                                 | 11 ++---
>  bin/doins.py                                  |  7 ++-
>  bin/ebuild                                    | 15 +-----
>  bin/filter-bash-environment.py                | 11 ++---
>  bin/install.py                                | 19 ++++---
>  bin/pid-ns-init                               | 10 ++--
>  bin/xattr-helper.py                           | 32 +++---------
>  lib/_emerge/DependencyArg.py                  | 10 ----
>  lib/_emerge/JobStatusDisplay.py               |  4 +-
>  lib/_emerge/Package.py                        |  8 ---
>  lib/_emerge/PackageVirtualDbapi.py            |  3 --
>  lib/_emerge/SequentialTaskQueue.py            |  3 --
>  lib/_emerge/TaskSequence.py                   |  3 --
>  lib/_emerge/UseFlagDisplay.py                 |  8 ---
>  lib/_emerge/UserQuery.py                      | 17 +++----
>  lib/_emerge/actions.py                        |  2 -
>  lib/_emerge/resolver/DbapiProvidesIndex.py    |  3 --
>  lib/_emerge/resolver/output_helpers.py        | 16 ------
>  lib/_emerge/resolver/slot_collision.py        |  8 ---
>  lib/portage/__init__.py                       | 49 +++++++------------
>  lib/portage/_emirrordist/Config.py            |  4 --
>  lib/portage/_selinux.py                       | 12 -----
>  lib/portage/_sets/base.py                     |  3 --
>  lib/portage/dbapi/porttree.py                 |  7 +--
>  lib/portage/dep/__init__.py                   | 18 +------
>  lib/portage/dep/soname/SonameAtom.py          |  8 ---
>  lib/portage/elog/mod_save_summary.py          |  2 -
>  lib/portage/elog/mod_syslog.py                |  4 --
>  lib/portage/exception.py                      | 45 ++++-------------
>  lib/portage/mail.py                           | 49 +++----------------
>  lib/portage/manifest.py                       |  7 ---
>  lib/portage/output.py                         |  4 +-
>  lib/portage/process.py                        | 16 +-----
>  lib/portage/proxy/objectproxy.py              |  6 ---
>  lib/portage/repository/config.py              |  6 ---
>  .../tests/unicode/test_string_format.py       | 44 ++++-------------
>  lib/portage/util/_ShelveUnicodeWrapper.py     | 45 -----------------
>  39 files changed, 102 insertions(+), 453 deletions(-)
>  delete mode 100644 lib/portage/util/_ShelveUnicodeWrapper.py

Looks good. Please merge.
-- 
Thanks,
Zac
signature.asc (application/pgp-signature, 981 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQKTBAEBCgB9FiEE8OgXaltWzqgSupCu0HX7jBBKPSAFAl8QkXtfFIAAAAAALgAo
aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEYw
RTgxNzZBNUI1NkNFQTgxMkJBOTBBRUQwNzVGQjhDMTA0QTNEMjAACgkQ0HX7jBBK
PSCRRxAAm/hB+nB6bHSlG2XLgFNRSl15gseWalmpbmU5mnpw21ccSuYXJprcyo6g
Yk0IqJd3F6MUe6FJitrAc3a0Bq9FuK0H8ItGyQzexCYTtLOLWcz0tMoTMYWHdd7h
8WB63xzLidAg/YJAbWmo7aiT6jkzuWGhJKFftgxgBJ6Pk15lmcoZqd17HPmBgKCp
4r5bR944a9jlpDEzpa/K6jVbIbFMCAujY8Pm5ZqXwPR0Zh7R6QNMzFZXFvV5BXNl
bW9XvbfkxKYl8xDU7J5HA7wNkCU95SRFayTKyXyznfI/hQo2IvTA9zMF4WdM/htW
/TNj6xYXJwmm0Ah8wTXzIseVBS74HDa75E2OJOM8Ra/9/PmyIQClo94rDC+K/PP8
jdikI9S7wmTCZ+7UdaRs0sojcukEP27UIsPG6itaFs/1Jw1cTxCNXBLzoA4qFXwQ
6XUvkQLK0JULWjIu1VOH+AB2Ud3gOcxD7jVk1pi17cGj15afucLB3K+MG1IAKkK8
qOGDI5Sh6T3JmvnEFNGB6ERcogRCGijFQI++bgec/GvPOTFR/pVOQelYIQKDSsHW
0AZ1YFyUACQVNhMKe9SXpqxqjSkGPMUlZV2s4QD+Ph1kHj+SNebI9w96FKl9Ergt
I7aq+rvOKbMczeDWP3WBXMtIQTDQtfM8afLn1pz8dRcpAq155T4=
=xAwT
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.