OpenELEC / service.openelec.settings

service.openelec.settings - OpenELEC settings addon
Other
12 stars 35 forks source link

move lcdproc to initscript #22

Closed stefansaraev closed 11 years ago

stefansaraev commented 11 years ago

now we start LCDd from the addon. in resources/lib/modules/system.py def set_lcd_driver

I think it would be a good idea to move this back to an initscript. as we did for bluez/avahi/samba/...

if we do this, we shoud re-use the good old /etc/init.d/63_lcdproc from openelec-3.0 branch

@sraue @lfiebach I don't know how important lcdproc is, I have no hardware to test so I don't even know how it works. I just noticed that we start it via settings addon while trying to fix a weird bug where LCDd was not restarted on driver change (see d1204b1a84).

if you decide to rework, consider this as very low prio. if not - feel free to close this ticket. in general I am ok with lcdproc as it is now if it works as intended. the only issue we had IMO was os.system() vs subprocess.Popen(), which is already fixed in d1204b1a84

stefansaraev commented 11 years ago

ignore it. I'll take care of this if still needed