Juan Hernandez has posted comments on this change.
Change subject: Use vncviewer passwordFile instead of passwdInput
......................................................................
Patch Set 2: (1 inline comment)
....................................................
File src/ovirtcli/platform/posix/vnc.py
Line 33: cmd_passwd = util.which('vncpasswd')
Line 34: if cmd_passwd is None:
Line 35: raise Error, Messages.Error.NO_CONSOLE_FOUND % ('vncpasswd',
'vncpasswd')
Line 36: p = Popen([cmd_passwd, "-f"], shell=False, stdin=PIPE, stdout=PIPE)
Line 37: password = p.communicate(input=ticket)[0]
You are right, no need for "p.wait()".
Line 38: args = ['vncviewer', '%s::%s' % (host, port), '-passwordFile',
'/dev/stdin' ]
Line 39: pid, pstdin = util.spawn(cmd, args, debug)
Line 40: os.write(pstdin, password)
--
To view, visit http://gerrit.ovirt.org/11857
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I76178f15217c421d88879c88534d7cb3a7583426
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine-cli
Gerrit-Branch: master
Gerrit-Owner: Sander Hoentjen <[email protected]>
Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Sander Hoentjen <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
