Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BatteryIcon: move timeout_add to _configure #531

Closed
wants to merge 1 commit into from

Conversation

aborilov
Copy link
Contributor

BatteryIcon don't work because there are no self.qtile.timeout_add in init method

@tych0
Copy link
Member

tych0 commented Oct 13, 2014

Hi,

Sorry about this, I have some patches to fix this in a more general way at https://github.com/tych0/qtile/tree/fix-google-calendar

Can you verify this patch series fixes this issue for you? I keep meaning to land it, but things keep coming up.

Thanks,

Tycho

@tych0
Copy link
Member

tych0 commented Oct 26, 2014

Can you try the current development branch and see if that works for you? I've pushed my patches. Thanks!

@aborilov
Copy link
Contributor Author

Will do tomorrow

@aborilov
Copy link
Contributor Author

on my arch machine I have no problem, with aur package qtile-python3-git
but on my old ubuntu 12.04, I have a lot of installation. Now I know what is cabal for and how to install hackell packages... :)
but still can't start qtile session with error

2014-10-30 09:49:41,005 ERROR make_qtile:123 Qtile crashed during startup
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/qtile-0.8.0-py2.7.egg/EGG-INFO/scripts/qtile", line 120, in make_qtile
state=options.state,
File "/usr/local/lib/python2.7/dist-packages/qtile-0.8.0-py2.7.egg/libqtile/manager.py", line 125, in init
self.supporting_wm_check_window.wid
File "/usr/local/lib/python2.7/dist-packages/qtile-0.8.0-py2.7.egg/libqtile/xcbq.py", line 617, in set_property
value
File "/usr/local/lib/python2.7/dist-packages/xcffib-0.1.8-py2.7-linux-i686.egg/xcffib/xproto.py", line 1756, in ChangeProperty
buf.write(xcffib.pack_list(data, "c"))
File "/usr/local/lib/python2.7/dist-packages/xcffib-0.1.8-py2.7-linux-i686.egg/xcffib/init.py", line 696, in pack_list
return struct.pack("=" + pack_type * len(from_), *from_)
error: char format require string of length 1
2014-10-30 09:52:03,347 WARNING init_log:70 Starting Qtile
2014-10-30 09:52:04,089 WARNING safe_import:27 Can't Import Widget: '.wlan.Wlan', No module named pythonwifi.iwlibs
2014-10-30 09:52:04,106 WARNING safe_import:27 Can't Import Widget: '.google_calendar.GoogleCalendar', No module named apiclient.discovery
2014-10-30 09:52:04,191 ERROR make_qtile:123 Qtile crashed during startup
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/qtile-0.8.0-py2.7.egg/EGG-INFO/scripts/qtile", line 120, in make_qtile
state=options.state,
File "/usr/local/lib/python2.7/dist-packages/qtile-0.8.0-py2.7.egg/libqtile/manager.py", line 125, in init
self.supporting_wm_check_window.wid
File "/usr/local/lib/python2.7/dist-packages/qtile-0.8.0-py2.7.egg/libqtile/xcbq.py", line 617, in set_property
value
File "/usr/local/lib/python2.7/dist-packages/xcffib-0.1.8-py2.7-linux-i686.egg/xcffib/xproto.py", line 1756, in ChangeProperty
buf.write(xcffib.pack_list(data, "c"))
File "/usr/local/lib/python2.7/dist-packages/xcffib-0.1.8-py2.7-linux-i686.egg/xcffib/init.py", line 696, in pack_list
return struct.pack("=" + pack_type * len(from_), *from_)
error: char format require string of length 1

@tych0
Copy link
Member

tych0 commented Oct 31, 2014

Ok, I just reverted the commit that was causing that issue, which was unrelated to the widget issue (we broke develop in the process :). Anyway, can you try again when you get a chance. Thanks!

@aborilov
Copy link
Contributor Author

Nop, the same error. https://gist.github.com/aborilov/8090452b6cd68cdd9a4d

@tych0
Copy link
Member

tych0 commented Nov 11, 2014

Ah ha, I think this is actually a problem with the underlying xcffib. Would you mind testing this patch? tych0/xcffib@ef7f575

@tych0
Copy link
Member

tych0 commented Dec 12, 2014

Ok, I think all is well here, as it seems the above patch is working for other folks. Closing this for now.

@tych0 tych0 closed this Dec 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants