cemmanouilidis / naturalscrolling

Natural Scrolling for Linux
GNU General Public License v3.0
69 stars 11 forks source link

Crashes every time I connect my blue tooth headset #41

Closed jc00ke closed 11 years ago

jc00ke commented 11 years ago

So I'll have naturalscrolling running (love it!) but when I connect my Motorola S305 blue tooth headset, naturalscrolling crashes. I can't seem to copy the output from the app crash program so here are some screenshots. Crash report_001

I select Leave Closed and naturalscrolling continues to work correctly. o_O

zedtux commented 11 years ago

Thank you @jc00ke for having reported that issue.

It will be hard to fix it as I don't have any bluethooth things, but based on your screenshot I know how to fix it. So the only way to fix it is that you help me:

If that's okay for you I will fix it :-)

jc00ke commented 11 years ago

Yes, I will be happy to help. Can you provide instructions on updating, or if you update the ppa I can just update via aptitude.

zedtux commented 11 years ago

Hey @jc00ke, I need the traceback section sorry.

jc00ke commented 11 years ago

@zedtux how can I get it to you? I don't see it in the screenshot above. Is there a crash report somewhere else?

zedtux commented 11 years ago

@jc00ke you can quit naturalscrolling and re-start it from a terminal or in your screenshot, there is a collapsed traceback section.

jc00ke commented 11 years ago

Yeah, duh Jesse :wink:

∴ naturalscrolling 
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.7/naturalscrolling_lib/udevobservator.py", line 87, in on_device_added
    gconf_key = GConfSettings().key(xid, bool)
  File "/usr/lib/pymodules/python2.7/naturalscrolling_lib/gconfsettings.py", line 239, in key
    return GConfKey(key, self.python_type_to_gconf_type(type))
  File "/usr/lib/pymodules/python2.7/naturalscrolling_lib/gconfsettings.py", line 125, in __init__
    if key.startswith(GCONF_ROOT_DIR):
AttributeError: 'NoneType' object has no attribute 'startswith'
zedtux commented 11 years ago

Great! That's enough for me to do the fix! :smile:

jc00ke commented 11 years ago

:+1: :shipit:

zedtux commented 11 years ago

@jc00ke could you please test my fix by following the step Fetch code sources from our wiki page ?

jc00ke commented 11 years ago

So this time naturalscrolling did not crash when my headphones connected but it did when they disconnected

∴ ./bin/naturalscrolling 
Traceback (most recent call last):
  File "/tmp/naturalscrolling/naturalscrolling_lib/udevobservator.py", line 106, in on_device_removed
    GConfSettings().key(xid).remove()
  File "/tmp/naturalscrolling/naturalscrolling_lib/gconfsettings.py", line 240, in key
    return GConfKey(key, self.python_type_to_gconf_type(type))
  File "/tmp/naturalscrolling/naturalscrolling_lib/gconfsettings.py", line 125, in __init__
    if key.startswith(GCONF_ROOT_DIR):
AttributeError: 'NoneType' object has no attribute 'startswith'
zedtux commented 11 years ago

Alright try again please :smiley: .

This time you just need to do a git pull instead of git clone.

jc00ke commented 11 years ago

No crash! :shipit:

∴ ./bin/naturalscrolling 

(naturalscrolling:10567): LIBDBUSMENU-GLIB-WARNING **: Trying to remove a child that doesn't believe we're it's parent.
jc00ke commented 11 years ago

Did this get pushed out to Launchpad? I can clone and run locally, but I would prefer to stick with the ppa.

jc00ke commented 11 years ago

PPA never updated for me. Was it pushed?

zedtux commented 11 years ago

You're right I forgotten to produce the package in the PPA.

The code is automatically imported to Launchpad, I just have to update the debian/changelog and start the production of the package.

As soon as it is done, you'll have to install the naturalscrolling-testing instead of naturalscrolling. When you'll have tested it then I'll update the naturalscrolling package.

zedtux commented 11 years ago

The package production is ongoing...

jc00ke commented 11 years ago

Sweet! I couldn't wait, so I run it off HEAD now, but I can install -testing and give it a whirl.

zedtux commented 11 years ago

Packages are now ready. Please test and let me know :-)

zedtux commented 11 years ago

Any news?

jc00ke commented 11 years ago

Sorry, been slammed at work. I'll test it as soon as I can.

zedtux commented 11 years ago

Don't worry we have time :smile:

jc00ke commented 11 years ago

naturalscrolling-testing works!

zedtux commented 11 years ago

Nice. I will then do a merge of the production branch and produce packages.

zedtux commented 11 years ago

Packages have been built and published :smile:

jc00ke commented 11 years ago

I just got a crash when turning off my bluetooth headphones :frowning:

Selection_001

Selection_002

zedtux commented 11 years ago

Hi @jc00ke,

Is this happening often ? Is it happening since a long time ago ?

jc00ke commented 11 years ago

I don't think I connected my bluetooth headphones for a while, and the first time I disconnected them I got that error.

zedtux commented 11 years ago

Is it still raising error or working ? (I will handle this case I just want to know if it's a really issue or maybe just wrong saved data)

jc00ke commented 11 years ago

I packed up my bluetooth headset while I moved, so I won't be able to test it. I don't quite understand what you mean by "if it's really a issue or maybe just wrong saved data" but I'll try to help out where I can.

zedtux commented 11 years ago

My bad... I didn't read carefully enough the error message. Anyway I will handle this case in order to no more have crashing.