qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 5690b4: python: Update for pylint 2.10


From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] 5690b4: python: Update for pylint 2.10
Date: Mon, 20 Sep 2021 06:20:31 -0700

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 5690b4370b868b339ee95f5330ad423b32e95e81
      
https://github.com/qemu/qemu/commit/5690b4370b868b339ee95f5330ad423b32e95e81
  Author: John Snow <jsnow@redhat.com>
  Date:   2021-09-16 (Thu, 16 Sep 2021)

  Changed paths:
    M python/qemu/machine/machine.py
    M python/setup.cfg

  Log Message:
  -----------
  python: Update for pylint 2.10

A few new annoyances. Of note is the new warning for an unspecified
encoding when opening a text file, which actually does indicate a
potentially real problem; see
https://www.python.org/dev/peps/pep-0597/#motivation

Use LC_CTYPE to determine an encoding to use for interpreting QEMU's
terminal output. Note that Python states: "language code and encoding
may be None if their values cannot be determined" -- use a platform
default as a backup.

Notes: Passing encoding=None will generate a suppressed warning on
Python 3.10+ that 'None' should not be passed as the encoding
argument. This behavior may be deprecated in the future and the default
switched to be a ubiquitous UTF-8. Opting in to the locale default will
be done by passing the encoding 'locale', but that isn't available in
3.6 through 3.9. Presumably this warning will be unsuppressed some time
prior to the actual switch and we can re-investigate these issues at
that time if necessary.

Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Message-id: 20210916182248.721529-2-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>


  Commit: eb8033f658e8b6f23ba9f4ef4a1b55894f7ea486
      
https://github.com/qemu/qemu/commit/eb8033f658e8b6f23ba9f4ef4a1b55894f7ea486
  Author: John Snow <jsnow@redhat.com>
  Date:   2021-09-16 (Thu, 16 Sep 2021)

  Changed paths:
    M python/setup.cfg

  Log Message:
  -----------
  python: pylint 2.11 support

We're not ready to enforce f-strings everywhere, so just silence this
new warning.

Signed-off-by: John Snow <jsnow@redhat.com>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
Message-id: 20210916182248.721529-3-jsnow@redhat.com
Signed-off-by: John Snow <jsnow@redhat.com>


  Commit: c3f76fbca6079c61c5452a77d5e517961aff1143
      
https://github.com/qemu/qemu/commit/c3f76fbca6079c61c5452a77d5e517961aff1143
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-09-20 (Mon, 20 Sep 2021)

  Changed paths:
    M python/qemu/machine/machine.py
    M python/setup.cfg

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/jsnow-gitlab/tags/python-pull-request' 
into staging

Python Pull request

This fixes the check-python-tox job.

CI including optional jobs is all green:
https://gitlab.com/jsnow/qemu/-/pipelines/372151147

# gpg: Signature made Thu 16 Sep 2021 23:05:35 BST
# gpg:                using RSA key F9B7ABDBBCACDF95BE76CBD07DEF8106AAFC390E
# gpg: Good signature from "John Snow (John Huston) <jsnow@redhat.com>" [full]
# Primary key fingerprint: FAEB 9711 A12C F475 812F  18F2 88A9 064D 1835 61EB
#      Subkey fingerprint: F9B7 ABDB BCAC DF95 BE76  CBD0 7DEF 8106 AAFC 390E

* remotes/jsnow-gitlab/tags/python-pull-request:
  python: pylint 2.11 support
  python: Update for pylint 2.10

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


Compare: https://github.com/qemu/qemu/compare/735c20bd4925...c3f76fbca607



reply via email to

[Prev in Thread] Current Thread [Next in Thread]