Univention Bugzilla – Bug 36635
python undefined name in UVMM
Last modified: 2015-04-16 10:51:32 CEST
./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:65: undefined name 'uvmm_p' ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:68: undefined name 'uvmm_c' ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:72: undefined name 'uvmm_c' → The lines where it is imported are uncommented. ./virtualization/univention-virtual-machine-manager-daemon/test/urlparser.py:83: undefined name 'urlparse' ./virtualization/univention-virtual-machine-manager-daemon/test/urlparser.py:84: undefined name 'urlparse'
(In reply to Florian Best from comment #0) > ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:65: > undefined name 'uvmm_p' > ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:68: > undefined name 'uvmm_c' > ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:72: > undefined name 'uvmm_c' > → The lines where it is imported are uncommented. This is dead code, which should be removed (it never worked because until the PEP366 work-around might have fixed it, but the current code calls the external CLI tool anyway, which does the job) > ./virtualization/univention-virtual-machine-manager-daemon/test/urlparser.py: > 83: undefined name 'urlparse' > ./virtualization/univention-virtual-machine-manager-daemon/test/urlparser.py: > 84: undefined name 'urlparse' This is a false positive, which is cause by how pyflakes/pylint works - see Bug #32091 comment 1 or <http://www.logilab.org/blogentry/78354>
(In reply to Philipp Hahn from comment #1) > (In reply to Florian Best from comment #0) > > ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:65: > > undefined name 'uvmm_p' > > ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:68: > > undefined name 'uvmm_c' > > ./virtualization/univention-virtual-machine-manager-daemon/uvmmd-ldap.py:72: > > undefined name 'uvmm_c' > > → The lines where it is imported are uncommented. > > This is dead code, which should be removed (it never worked because until > the PEP366 work-around might have fixed it, but the current code calls the > external CLI tool anyway, which does the job) That code has been removed: r59484 | Bug #36635 UVMM: Remove dead UDL code Package: univention-virtual-machine-manager-daemon Version: 4.0.24-2.592.201503291138 Branch: ucs_4.0-0 Scope: errata4.0-1 r59487 | Bug #36635,Bug #36640 UVMM: YAML 2015-03-29-univention-virtual-machine-manager-daemon.yaml
Verified
<http://errata.univention.de/ucs/4.0/147.html>