Re: master f25f4699d7 2/2: Support several new DEs in wallpaper.el

[Po Lu <luangruo@yahoo.com>]

Stefan Kangas <stefankangas@gmail.com> writes:

> branch: master
> commit f25f4699d760c5409a1571dcfdf4ed0fdac9e0ec
> Author: Stefan Kangas <stefankangas@gmail.com>
> Commit: Stefan Kangas <stefankangas@gmail.com>
>
>     Support several new DEs in wallpaper.el
>     
>     * lisp/image/wallpaper.el (wallpaper--default-commands)
>     (wallpaper--check-command, wallpaper-command): Add support for Unity,
>     Pantheon, Budgie, LXQt, and LXDE.  Also add support for Lubuntu,
>     Xubuntu, and Pop!_OS.
> ---
>  lisp/image/wallpaper.el | 43 +++++++++++++++++++++++++++++++++++++++++--
>  1 file changed, 41 insertions(+), 2 deletions(-)
>
> diff --git a/lisp/image/wallpaper.el b/lisp/image/wallpaper.el
> index ff47d37e3a..effa25b180 100644
> --- a/lisp/image/wallpaper.el
> +++ b/lisp/image/wallpaper.el
> @@ -62,6 +62,16 @@
>      ;; XFCE
>      ("xfconf-query" "-c" "xfce4-desktop" "-p"
>       "/backdrop/screen0/monitoreDP/workspace0/last-image" "-s" "%f")
> +    ;; LXDE
> +    ("pcmanfm" "--set-wallpaper=%f")
> +    ;; LXQt
> +    ("pcmanfm-qt" "--set-wallpaper=%f") ; "--wallpaper-mode=MODE"
> +    ;; ;; Mate
> +    ;; ("gsettings" "set" "org.mate.background" "picture-filename" "%f")
> +    ;; ;; Cinnamon
> +    ;; ("gsettings" "set" "org.cinnamon.desktop.background" "picture-uri" "file://%f")
> +    ;; ;; Deepin
> +    ;; ("gsettings" "set" "com.deepin.wrap.gnome.desktop.background" "picture-uri" "file://%f")

BTW, did you test the file://%f commands with file names containing
spaces and/or Unicode to make sure they indeed do not have to be
encoded?