Ticket #3203 regression closed fixed

Opened 5 years ago

Last modified 5 years ago

PortableGtkReactor is broken

Reported by: jdahlin Owned by:
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/portablegtkreactor-3203-2
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

I get the following error when trying to use it on win32: AttributeError: 'PortableGtkReactor' object has no attribute '_simtag'

The workaround is to set self._simtag = None in the constructor.

Attachments

3203-win.diff Download (0.9 KB) - added by therve 5 years ago.

Change History

1

Changed 5 years ago by jdahlin

  • type changed from enhancement to regression

2

Changed 5 years ago by exarkun

  • branch set to branches/portablegtkreactor-3203
  • author set to exarkun

(In [23422]) Branching to 'portablegtkreactor-3203'

3

Changed 5 years ago by glyph

#3221 was a duplicate of this bug.

4

Changed 5 years ago by exarkun

The full test suite hangs when run against the branch. It gets to a test in flow and that's the end. Not sure what's up, and I don't have a Windows machine so debugging further will be more of a hassle for me than I'm likely to be willing to deal with.

5

Changed 5 years ago by therve

  • owner changed from glyph to therve

I'll try to look at it.

Changed 5 years ago by therve

6

Changed 5 years ago by therve

The attached patch fixes most problems I saw. There is still one problem with some tests succeeding after timeout (like LargeBufferTestCase.testWriter). But the rest looks good.

7

Changed 5 years ago by therve

(In [23528]) Apply patch

Refs #3203

8

Changed 5 years ago by therve

  • keywords review added
  • owner therve deleted

Please review.

9

Changed 5 years ago by jdahlin

gtk.mainquit() is a leftover since PyGTK 0.6.x (gtk 1.2.x) and all references to it should probably be removed, since the module is called gtk2reactor after all.

10

Changed 5 years ago by exarkun

  • branch changed from branches/portablegtkreactor-3203 to branches/portablegtkreactor-3203-2

(In [23689]) Branching to 'portablegtkreactor-3203-2'

11

Changed 5 years ago by exarkun

  • keywords review removed
  • owner set to therve

Merged forward to make sure it passed at least as many tests on trunk, and it seems to. Change seems sensible to me, mostly mirroring what was done to the other gtk2reactor I guess. I guess we don't have any test coverage showing that gtk.mainquit() is doing anything. I dunno what versions of stuff anyone has or would expect to work. I'd be inlined to leave that code alone for a while longer, since all you did was indent it.

So, seems good to merge.

12

Changed 5 years ago by therve

  • status changed from new to closed
  • resolution set to fixed

(In [23696]) Merge portablegtkreactor-3203-2

Authors: therve, exarkun Reviewer: exarkun Fixes #3203

Fix PortableGtkReactor: we're now able to run trial test suite with the gtk2 reactor under Windows.

13

Changed 2 years ago by <automation>

  • owner therve deleted
Note: See TracTickets for help on using tickets.