xpra icon
Bug tracker and wiki

This bug tracker and wiki are being discontinued
please use https://github.com/Xpra-org/xpra instead.

Opened 5 years ago

Closed 5 years ago

Last modified 17 months ago

#1612 closed defect (duplicate)

--dpi: client fails with "local variable 'xdpi' referenced before assignment"

Reported by: mviereck Owned by: Antoine Martin
Priority: minor Milestone:
Component: client Version: 2.1.x
Keywords: dpi Cc:


system: debian stretch with winswitch stable repository
bug in xpra 2.1, currently xpra v2.1-r16525
not a bug in 2.0.3

With option --dpi xpra 2.1 client fails with

error preparing connection: local variable 'xdpi' referenced before assignment
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/xpra/client/client_base.py", line 317, in send_hello
  File "/usr/lib/python2.7/dist-packages/xpra/client/gtk2/client.py", line 287, in make_hello
    capabilities = GTKXpraClient.make_hello(self)
  File "/usr/lib/python2.7/dist-packages/xpra/client/gtk_base/gtk_client_base.py", line 418, in make_hello
    capabilities = UIXpraClient.make_hello(self)
  File "/usr/lib/python2.7/dist-packages/xpra/client/ui_client_base.py", line 1424, in make_hello
    self._last_screen_settings = (root_w, root_h, sss, ndesktops, desktop_names, u_root_w, u_root_h, xdpi, ydpi)
UnboundLocalError: local variable 'xdpi' referenced before assignment

I provide --dpi to server and client, if that matters. The server starts up fine, the client crashes with above message.
(I did not test providing --dpi to only one of server or client.)
The error does not happen in xpra 2.0.3.

Change History (4)

comment:1 Changed 5 years ago by Antoine Martin

Resolution: duplicate
Status: newclosed

See #1605

comment:2 Changed 5 years ago by mviereck

Sorry for double-ticket, I could not find #1605 in active tickets (searching for "dpi"), probably because it is already closed.

The correct fix is in r16546.
Backport to older branches in r16547.

r16547 is currently not present in winswitch repo for debian stretch, likely you will provide it, soon.

comment:3 Changed 5 years ago by Antoine Martin

This will be included in 2.1.1 release, due in the next few days.

comment:4 Changed 17 months ago by migration script

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/1612

Note: See TracTickets for help on using tickets.