[Qemu-commits] [qemu/qemu] 581031: qemu.py: Call logging.basicConfig() automatically

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 5810314e98c6e83b3334c451c7999fff7a27ef78
      
https://github.com/qemu/qemu/commit/5810314e98c6e83b3334c451c7999fff7a27ef78
  Author: Eduardo Habkost <address@hidden>
  Date:   2017-09-22 (Fri, 22 Sep 2017)

  Changed paths:
    M scripts/qemu.py

  Log Message:
  -----------
  qemu.py: Call logging.basicConfig() automatically

Not all scripts using qemu.py configure the Python logging
module, and end up generating a "No handlers could be found for
logger" message instead of actual log messages.

To avoid requiring every script using qemu.py to configure
logging manually, call basicConfig() when creating a QEMUMachine
object.  This won't affect scripts that already set up logging,
but will ensure that scripts that don't configure logging keep
working.

Reported-by: Kevin Wolf <address@hidden>
Fixes: 4738b0a85a0c2031fddc71b51cccebce0c4ba6b1
Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Reviewed-by: Cleber Rosa <address@hidden>
Acked-by: Lukáš Doktor <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>


  Commit: ad904f6689f1ced282b413cb904c166f294a73ee
      
https://github.com/qemu/qemu/commit/ad904f6689f1ced282b413cb904c166f294a73ee
  Author: Eduardo Habkost <address@hidden>
  Date:   2017-09-22 (Fri, 22 Sep 2017)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Add Python scripts

Cleber and I are volunteering to review and queue patches for the
Python scripts and modules in scripts/.

I'm setting "M: Odd fixes" because not all scripts are actively
maintained.

Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
[ehabkost: add tests/*.py too]
Signed-off-by: Eduardo Habkost <address@hidden>


  Commit: c348b54ab5c3e6c80fbf365b671974fd92f39113
      
https://github.com/qemu/qemu/commit/c348b54ab5c3e6c80fbf365b671974fd92f39113
  Author: Peter Maydell <address@hidden>
  Date:   2017-09-22 (Fri, 22 Sep 2017)

  Changed paths:
    M MAINTAINERS
    M scripts/qemu.py

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

Python queue, 2017-09-22

* MAINTAINERS update
* Fix logging issue on test scripts using qemu.py

# gpg: Signature made Fri 22 Sep 2017 15:41:43 BST
# gpg:                using RSA key 0x2807936F984DC5A6
# gpg: Good signature from "Eduardo Habkost <address@hidden>"
# Primary key fingerprint: 5A32 2FD5 ABC4 D3DB ACCF  D1AA 2807 936F 984D C5A6

* remotes/ehabkost/tags/python-next-pull-request:
  MAINTAINERS: Add Python scripts
  qemu.py: Call logging.basicConfig() automatically

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/bef81b3eb521...c348b54ab5c3