Xpra: Ticket #1239: HTML Server spewing name error tracebacks

I have a feeling it has to do with the fixes due to #1218.

With a trunk built r12890 server and connecting with Google Chrome:

NameError: global name 'excluded' is not defined
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/xpra/server/window/window_source.py", line 1112, in expire_delayed_region
    self.may_send_delayed()
  File "/usr/lib64/python2.7/site-packages/xpra/server/window/window_source.py", line 1211, in may_send_delayed
    self.do_send_delayed()
  File "/usr/lib64/python2.7/site-packages/xpra/server/window/window_source.py", line 1223, in do_send_delayed
    self.send_delayed_regions(*delayed)
  File "/usr/lib64/python2.7/site-packages/xpra/server/window/window_source.py", line 1239, in send_delayed_regions
    self.do_send_delayed_regions(damage_time, regions, coding, options)
  File "/usr/lib64/python2.7/site-packages/xpra/server/window/window_video_source.py", line 579, in do_send_delayed_regions
    return send_nonvideo(get_best_encoding=self.get_best_encoding)
  File "/usr/lib64/python2.7/site-packages/xpra/server/window/window_video_source.py", line 570, in send_nonvideo
    WindowSource.do_send_delayed_regions(self, damage_time, regions, encoding, options, exclude_region=exclude_region, get_best_encoding=get_best_encoding)
  File "/usr/lib64/python2.7/site-packages/xpra/server/window/window_source.py", line 1332, in do_send_delayed_regions
    return excluded
NameError: global name 'excluded' is not defined

The html client does appear to work okay, if a little slow.



Wed, 22 Jun 2016 16:49:27 GMT - J. Max Mena: description, summary changed

Update:

I see this even with connecting with a regular Python client, and even with HTML disabled. Updating the ticket to reflect this.


Wed, 22 Jun 2016 16:52:14 GMT - Antoine Martin: owner, description, summary changed

Hmmm, are you sure of the state of your source tree? I can find no such code anywhere:

$ grep -r "excluded" xpra/ | wc -l
0

Not even in older branches. And line 1332 doesn't have it either: browser/xpra/trunk/src/xpra/server/window/window_source.py


Wed, 22 Jun 2016 17:01:31 GMT - J. Max Mena:

Hmmmm.

I turfed my source tree and re-downloaded trunk to be sure, before I turfed it it found 13, and now it says 8.

For what it's worth, after rebuilding, I no longer get that error, but I do see:

Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/xpra/server/server_base.py", line 2328, in _process_sound_control
    ss.sound_control(*packet[1:])
  File "/usr/lib64/python2.7/site-packages/xpra/server/source.py", line 1124, in sound_control
    self.set_av_sync_delay(int(args[0]))
  File "/usr/lib64/python2.7/site-packages/xpra/server/source.py", line 1187, in set_av_sync_delay
    self.update_av_sync_delay_total()
  File "/usr/lib64/python2.7/site-packages/xpra/server/source.py", line 1210, in update_av_sync_delay_total
    ws.set_av_sync_delay(self.av_sync_delay_total)
TypeError: set_av_sync_delay() takes exactly 1 argument (2 given)

a couple times on connect, but it seems to run fine now.

Weird.


Wed, 22 Jun 2016 17:09:22 GMT - Antoine Martin:

set_av_sync_delay() takes exactly 1 argument (2 given)


This one is real, caused by r12868 and should be fixed in r12891.


Wed, 22 Jun 2016 17:10:32 GMT - J. Max Mena: status changed; resolution set

Okay, then it was my source tree. I'm gonna go ahead and close this as invalid.

Thanks for the help.


Tue, 12 Jul 2016 16:52:22 GMT - Antoine Martin: milestone changed

Milestone renamed


Sat, 23 Jan 2021 05:18:48 GMT - migration script:

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