[danoxide]

Wczoraj napisałem mały program w Pythonie, który pobiera aktualny obraz z Bing i ustawia jako tło pulpitu.
Sprawdzone na Ubuntu z Gnome 3 bo na reszcie distro nie testowałem, bo po prostu nie mam :)
Póki co trzeba ręcznie odpalać albo dodać plik run.sh do autostartu.

https://github.com/danoxide/Bing-Wallpaper

[morsik]

@danoxide: o Panie… ale za taby i camelCase to bijemy po łapach! ( ͡° ʖ̯ ͡°) PEP8!
Do tego __init__ nie powinien robić nic więcej niż inicjalizować klasy. A tu jeszcze ustawia tapetę, to trochę za daleko.
No ale masz plusa za chęci ;)

[danoxide]

@morsik: na co dzień siedzę w webdevie, a nie Pythonie, więc wybacz :) Zaraz poprawię ;)

[p.....3]

@danoxide: webdev to też python ( ͡° ͜ʖ ͡°)

[danoxide]

@peterek93: bardziej php, css, js :)

[p.....o]

@danoxide: File "bingbackground.py", line 29
self.setAsWallpaper(self.target)
^
IndentationError: expected an indented block

[danoxide]

@py_chi_wo: u mnie działa ( ͡° ͜ʖ ͡°)

[p.....o]

@danoxide: W Python2.7 pobiera tapety ale nie zmienia (Lubuntu) a w Python3.4: Traceback (most recent call last):
File "bingbackground.py", line 5, in
import json, urllib2, os
ImportError: No module named 'urllib2'

[danoxide]

@py_chi_wo: ale korzystasz z LXDE, tak?

[p.....o]

@danoxide: Tak z LXDE

[p.....o]

@danoxide: Jeśli chcesz uwagi dotyczące kodu to:
1. Funkcji nie nazywamy camelCasem (wspomniane wyżej)
2. Komentarze poprzedzające klasę i funkcje zamieniłbym na docstringi: https://www.python.org/dev/peps/pep-0257/
3. W liniach 37-38 prawdopodobnie masz anti-pattern: http://docs.quantifiedcode.com/python-code-patterns/maintainability/not_using_with_to_open_files.html

[Leinnan]

@danoxide: https://ubuntuincident.wordpress.com/2015/03/08/set-the-desktop-wallpaper-under-xfce4/ Tu masz jak ustawia się w XFCE.

[danoxide]

@Leinnan: @py_chi_wo: a co wam zwraca zmienna $DESKTOP_SESSION?

[Leinnan]

@danoxide: Ja tego nawet nie pobrałem :D

[danoxide]

@Leinnan: ale nie, chcę zobaczyć jak się nazywa konkretne środowisko, bo u mnie "gnome", gdzie indzej może "lxde" czy coś :D

[Leinnan]

@danoxide: Aaa, bo ja myślałem, że chodziło ci o jakąś zmienną z twojego kodu. No to u mnie proszę bardzo:

[piotr@pc ~]$ echo $DESKTOP_SESSION
xfce

[p.....o]

@danoxide: U mnie zwraca: Lubuntu

[InnyKtosek]

@danoxide: Taki hint, zamiast określać środowisko graficzne po nazwie dystrybucji lepiej byłoby je wykryć :)

https://github.com/revolunet/python-desktop

[danoxide]

@py_chi_wo: @InnyKtosek: @Leinnan: poprawione, można testować :)

[Leinnan]

@danoxide: Po pierwsze zaznacz, że w Manjaro(i Archu chyba też) trzeba podawać python2 zamiast python. Po drugie:

Właściwość „/backdrop/screen0/monitorLVDS-0/workspace0/last-image” nie istnieje na kanale „xfce4-desktop”. Proszę użyć opcji --create, w celu utworzenia nowej właściowości..

[danoxide]

@Leinnan: a mógłbyś sprawdzić czy poniższe polecenie zmienia tło? :)

xfconf-query --channel xfce4-desktop --property /backdrop/screen0/monitor0/image-path --set /usr/share/backgrounds/xfce/xfce-blue.jpg