Fabioamd87 commented 7 years ago

The script says:

fabio@a514:~$ bingwallpaper 
Pinging Bing API...
Bing Image of the day: http://www.bing.com/az/hprichbg/rb/DivingGondola_EN-US11480704756_1920x1200.jpg
Saving image to /home/fabio/Pictures/Bing/DivingGondola_EN-US11480704756_1920x1200.jpg
New wallpaper set successfully for LXDE.

but the wallpaper didn't get changed

Fabioamd87 commented 7 years ago

solved by editing

.config/lxsession/Lubuntu/autostart

(that was empty) and adding:

@lxpanel --profile Lubuntu
@pcmanfm --desktop --profile Lubuntu
@xscreensaver -no-splash

I think is the second line was necessary

Fabioamd87 commented 7 years ago

After the update of tomorrow of Lubuntu 17.04 it seems to not work anymore. The output isn't changed but the wallpaper doesn't change, I've checked and the options are still there in the autostart file

dzmanto commented 7 years ago

Dear Fabioamd87

do you want to try

pcmanfm-qt -w "/path/to/your/background"

and let me know the result?

Cheers,

Daniela

From: Fabioamd87 notifications@github.com Sent: Friday, 14 April 2017 6:11 PM To: marguerite/linux-bing-wallpaper Cc: Subscribed Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

After the update of tomorrow of Lubuntu 17.04 it seems to not work anymore. The output isn't changed but the wallpaper doesn't change, I've checked and the options are still there in the autostart file

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-294113106, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSMwmr48w0reIIGECpuZF5hj9ePMtyks5rvyo6gaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

fabio@a514:~/Pictures/Bing$ pcmanfm -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg 
fabio@a514:~/Pictures/Bing$ pcmanfm-qt -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg 
** Message: x-terminal-emulator has very limited support, consider choose another terminal
isPrimaryInstance
fabio@a514:~/Pictures/Bing$

I'm using lxde and not lxqt BTW with the first command I don't get results. with the second I get the error in terminal.

Thanks

dzmanto commented 7 years ago

Hi Fabio,

does the wallpaper change with any of pcmanfm or pcmanfm-qt?

Cheers,

Daniela

From: Fabioamd87 notifications@github.com Sent: Friday, 21 April 2017 2:49 AM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

fabio@a514:~/Pictures/Bing$ pcmanfm -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg fabio@a514:~/Pictures/Bing$ pcmanfm-qt -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg ** Message: x-terminal-emulator has very limited support, consider choose another terminal isPrimaryInstance fabio@a514:~/Pictures/Bing$

I'm using lxde and not lxqt BTW with the first command I don't get results. with the second I get the error in terminal.

Thanks

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-295809976, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM3S41UX3T1wigBRB42qH3458MIuCks5rx4yGgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

Hi Daniela, With both commands the wallpaper remains the same.

dzmanto commented 7 years ago

Dear Fabio:

Can you try

pcmanfm --set-wallpaper=YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg pcmanfm --wallpaper-mode=center pcmanfm --wallpaper-mode=stretch

and let me know if it works, please?

Also, you should ascertain that YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg is the full path to your .jpg image file.

Cheers,

Daniela

From: Fabioamd87 [notifications@github.com] Sent: Friday, 21 April 2017 2:49 AM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

fabio@a514:~/Pictures/Bing$ pcmanfm -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg fabio@a514:~/Pictures/Bing$ pcmanfm-qt -w YellowFrontedWoodpecker_EN-US12671581596_1920x1080.jpg ** Message: x-terminal-emulator has very limited support, consider choose another terminal isPrimaryInstance fabio@a514:~/Pictures/Bing$

I'm using lxde and not lxqt BTW with the first command I don't get results. with the second I get the error in terminal.

Thanks

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-295809976, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM3S41UX3T1wigBRB42qH3458MIuCks5rx4yGgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

I am sorry Daniela but I think I didn't understand correctly your first comment.

using

pcmanfm -w "/path/to/your/background"

without quotes works. Relative path in your last comment not.

but I still have problems with using pcmanfm-qt, that gives me the error that I've posted in my last message "x-terminal-emulator has very limited support" it's the same errore that I get if I start "pcmanfm-qt" from terminal, it's not related with the script.

if the script would use pcmanfm instead of pcmanfm-qt it would be ok I suppose

dzmanto commented 7 years ago

Dear Fabio:

Do you want to try the enclosed script

chmod +x bing_wallpaper.sh

./bing_wallpaper.sh

and let me know if it works, please?

Cheers,

Daniela

From: Fabioamd87 notifications@github.com Sent: Friday, 21 April 2017 7:11 PM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

I am sorry Daniela but I think I didn't understand correctly your first comment.

using

pcmanfm -w "/path/to/your/background"

without quotes works. Relative path in your last comment not.

but I still have problems with using pcmanfm-qt, that gives me the error that I've posted in my last message "x-terminal-emulator has very limited support"

if the script would use pcmanfm instead of pcmanfm-qt it would be ok I suppose

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-296137168, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM7NuoJc3MB-Zxg7E9HljDQ_HerwTks5ryHLMgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

> fabio@a514:~/bin/linux-bing-wallpaper$ chmod +x bing_wallpaper.sh 
> fabio@a514:~/bin/linux-bing-wallpaper$ sudo ./bing_wallpaper.sh 
> [sudo] password for fabio: 
> ./bing_wallpaper.sh: line 231: feh: command not found

but also after installing feh, the script, given as sudo, doesn't give any output and the wallpaper doesn't change.

also given as normal user.

dzmanto commented 7 years ago

Alright, can you send me the output of

echo ${XDG_CURRENT_DESKTOP}

please?

Cheers,

Daniela

From: Fabioamd87 [notifications@github.com] Sent: Monday, 24 April 2017 10:08 PM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

fabio@a514:/bin/linux-bing-wallpaper$ chmod +x bing_wallpaper.sh fabio@a514:/bin/linux-bing-wallpaper$ sudo ./bing_wallpaper.sh [sudo] password for fabio: ./bing_wallpaper.sh: line 231: feh: command not found

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-296641105, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM50Kipx6Idkf8GFWRAiSo5T6a11Yks5rzJCmgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

fabio@a514:~/bin/linux-bing-wallpaper$ echo ${XDG_CURRENT_DESKTOP}
LXDE
fabio@a514:~/bin/linux-bing-wallpaper$

thanks again

dzmanto commented 7 years ago

Hi Fabio:

Does your answer mean bing_wallpaper.sh finally works?

Also, may I ask you to download and try the enclosed shell script bing_wallpaper.sh? I would like ascertain that bing_wallpaper.sh works before I publish an update of my script on github. Your feedback is appreciated.

Cheers,

Daniela

From: Fabioamd87 notifications@github.com Sent: Tuesday, 25 April 2017 12:37 AM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

fabio@a514:/bin/linux-bing-wallpaper$ echo ${XDG_CURRENT_DESKTOP} LXDE fabio@a514:/bin/linux-bing-wallpaper$

thanks again

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-296688685, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM8vQmP7Sx5Sn-UkaER-dsELhf2ILks5rzLOmgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

Hi Daniela, unfortunately still no result but thanks for following my problem. what do you mean with "enclosed"? I started the script as root after installed feh and it doesn't work with no errors now. as I said using pcmanfm instead of pcmanfm-qt works

dzmanto commented 7 years ago

Dear Fabio:

I enclosed a script bing_wallpaper.sh as an ATTACHMENT. Can you save the ATTACHMENT to your computer and run the ./bing_wallpaper.sh, please? Your feedback in regard to success or failure is highly appreciated.

Cheers,

Daniela

From: Fabioamd87 [notifications@github.com] Sent: Tuesday, 25 April 2017 7:15 AM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Hi Daniela, unfortunately still no result but thanks for following my problem. what do you mean with "enclosed"? I started the script as root after installed feh and it doesn't work with no errors now.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-296824542, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM6Ht2ubhZf4HhL_GbqIdEUwfYBqwks5rzRECgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

Hi Daniela, I am sorry but I cannot see the ATTACHMENT. Im reading the conversation on github support website.

dzmanto commented 7 years ago

Hi Fabio:

No worries. Can you provide me with an email address, please? I will then forward the attachment. Alternatively, you may try and run the script as pasted below.

Cheers,

Daniela

!/bin/sh

Author: Marguerite Su i@marguerite.su, dzmanto dzmanto@hotmail.com

License: GPL-3.0

Description: Download Bing Wallpaper of the Day and set it as your Linux Desktop.

https://github.com/marguerite/linux-bing-wallpaper

global options

$bing is needed to form the fully qualified URL for

the Bing pic of the day

bing="www.bing.com"

The idx parameter determines where to start from. 0 is the current day,

1 the previous day, etc.

idx="0"

$xmlURL is needed to get the xml data from which

the relative URL for the Bing pic of the day is extracted

xmlURL="http://www.bing.com/HPImageArchive.aspx?format=xml&idx=$idx&n=1&mkt=$mkt"

Set picture options

Valid options are: none,wallpaper,centered,scaled,stretched,zoom,spanned

picOpts="stretched"

The file extension for the Bing pic

picExt=".jpg"

contains() { local value=$(eval "echo \$$#") count=1 for i in $* do if [ "$i" = "$value" -a $count -lt $# ]; then echo "y" return 0 fi count=$(expr $count + 1) done echo "n" return 1 }

checkdep() { tool=$(which $1) if [ ! -x "$tool" ]; then echo "Linux-bing-wallpaper depends on $1." echo "Install $1, please." echo "Exit." exit 1 fi }

ctfn () { tfnns=$(mktemp /tmp/bing_wallpaper_XXXXXX) tfn="$tfnns$picExt" mv "$tfnns" "$tfn" echo "$tfn" }

detectDE() {

see https://bugs.freedesktop.org/show_bug.cgi?id=34164

unset GREP_OPTIONS

uname -a | grep -i darwin 2>&1 >/dev/null && DE="mac";

if [ -n "${XDG_CURRENT_DESKTOP}" ]; then
  case "${XDG_CURRENT_DESKTOP}" in
     'GNOME'|'gnome')
       DE="gnome"
       ;;
     'KDE'|'kde')
       DE="kde"
       ;;
     'LXDE'|'lxde')
       DE="lxde"
       ;;
     'LXQt'|'lxqt')
       DE="lxqt"
       ;;
     'MATE'|'mate')
       DE="mate"
       ;;
     'XFCE'|'xfce')
       DE="xfce"
       ;;
     'X-Cinnamon'|'x-cinnamon')
       DE="cinnamon"
       ;; 
  esac
fi

if [ -z "$DE" ]; then
  # classic fallbacks
  if [ x"$KDE_FULL_SESSION" = x"true" ]; then DE="kde";
  elif [ x"$GNOME_DESKTOP_SESSION_ID" != x"" ]; then DE="gnome";
  elif [ x"$MATE_DESKTOP_SESSION_ID" != x"" ]; then DE="mate";
  elif `dbus-send --print-reply --dest=org.freedesktop.DBus /org/freedesktop/DBus org.freedesktop.DBus.GetNameOwner string:org.gnome.SessionManager > /dev/null 2>&1` ; then DE="gnome";
  elif xprop -root _DT_SAVE_MODE 2> /dev/null | grep ' = \"xfce4\"$' >/dev/null 2>&1; then DE="xfce";
  elif xprop -root 2> /dev/null | grep -i '^xfce_desktop_window' >/dev/null 2>&1; then DE="xfce";
  fi
fi

if [ -z "$DE" ]; then
  # fallback to checking $DESKTOP_SESSION
 case "$DESKTOP_SESSION" in
     'gnome')
       DE="gnome"
       ;;
     'LXDE'|'Lubuntu'|'lxde'|'lubuntu')
       DE="lxde"
       ;;
     'MATE'|'mate')
       DE="mate"
       ;;
     'xfce'|'xfce4'|'Xfce Session'|'xfce session')
       DE="xfce"
       ;;
  esac
fi

if [ x"$DE" = x"gnome" ]; then
  # gnome-default-applications-properties is only available in GNOME 2.x
  # but not in GNOME 3.x
  which gnome-default-applications-properties > /dev/null 2>&1  || DE="gnome3"
fi

# DE not found, maybe used WM
if [ -z "$DE" ]; then
    DE="WM"
fi

echo $DE

}

helpme() { echo "Usage: bing_wallpaper.sh [market,runonce]" echo "Generic option:" echo " --help show this help" echo "Wallpaper parameters:" echo " market: de_CH, de_DE, en_AU, en_CA, en_NZ, en_UK, en-US, ja_JP, or zh_CN" echo " runonce: true or false" echo " Note that market and runonce must be specified together or not at all." }

for j do if [ "$j" = "--help" -o "$j" = "-h" ]; then helpme exit fi done

checkdep "curl" checkdep "egrep"

if [ $# -eq 0 ]; then

The mkt parameter determines which Bing market you would like to

obtain your images from.

mkt="zh-CN"

Try and guess language

ML=$(echo $LANG | cut -f 1 -d .) case $ML in 'en_US') mkt="en-US" ;; 'zh_CN') mkt="zh-CN" ;; 'ja_JP') mkt="ja-JP" ;; 'en_AU') mkt="en-AU" ;; 'en_UK') mkt="en-UK" ;; 'de_CH') mkt="de-DE" ;; 'de_DE') mkt="de-DE" ;; 'en_NZ') mkt="en-NZ" ;; 'en_CA') mkt="en-CA" ;; esac exitAfterRunning=false

elif [ $# -eq 2 ]; then list="de-DE en-AU en-CA en-NZ en-UK en-US ja-JP zh-CN"

Valid values are:

firstpar="$1"

inhibit code injection

firstpar=$(echo "$firstpar" | sed s/[^a-zA-Z-]// ) if [ "$(contains $list $firstpar)" = "y" ]; then mkt=$firstpar else echo "mkt must be one of the following:" printf '%s\n' "$list" exit 1 fi

if [ "$2" = true ]; then exitAfterRunning=true else exitAfterRunning=false fi

else echo "Usage: basename $0 mkt[en-US,zh-CN,ja-JP,en-AU,en-UK,de-DE,en-NZ,en-CA] exitAfterRunning[true,false]" exit 1 fi

set target filename

tfn=$(ctfn)

Download the highest resolution

while true; do

picName=""
picURL=""

for picRes in _1920x1200 _1920x1080 _1366x768 _1280x720 _1024x768; do

# Extract the relative URL of the Bing pic of the day from
# the XML data retrieved from xmlURL, form the fully qualified
# URL for the pic of the day, and store it in $picURL
picURL=$bing$(echo $(curl -H "Content-Type: text/html; charset=UTF-8" -L -s $xmlURL) | egrep -o "<urlBase>(.*)</urlBase>" | cut -d ">" -f 2 | cut -d "<" -f 1)$picRes$picExt

# Download the Bing pic of the day
curl -H "Content-Type: text/html; charset=UTF-8" -s -o "$tfn" -L "$picURL"

# Test if download was successful.
downloadResult=$?
if [ $downloadResult -ge 1 ]; then
rm -f "$tfn" && continue
elif [ ! -s "$tfn" ]; then
rm -f "$tfn" && continue   
fi

if [ -x "/usr/bin/convert" -a -x "/usr/bin/mogrify" ]; then
title=$(echo $(curl -H "Content-Type: text/html; charset=UTF-8" -L -s $xmlURL) | egrep -o "<copyright>(.*)</copyright>" | cut -d ">" -f 2 | cut -d "<" -f 1 )
convert "$tfn" -resize 1920x1200 "$tfn"
convert -background "#00000080" -fill white -gravity center -size 1024 -font "Droid Sans" -pointsize 22 caption:"${title}" "$tfn" +swap -gravity south -composite "$tfn"
fi
# Test if it's a pic
file --mime-type -b "$tfn" | grep "^image/" > /dev/null && break

rm -f "$tfn"
done

if [ $downloadResult -ge 1 ]; then
      echo "Failed to download any picture."
  echo "Try again in 60 seconds."
      sleep 60
      continue
fi

DE=$(detectDE)

if [ "$DE" = "cinnamon" ]; then
      # Set the Cinnamon wallpaper
      DISPLAY=:0 GSETTINGS_BACKEND=dconf gsettings set org.cinnamon.desktop.background picture-uri '"file://'$tfn'"'

      # Set the Cinnamon wallpaper picture options
      DISPLAY=:0 GSETTINGS_BACKEND=dconf gsettings set org.cinnamon.desktop.background picture-options $picOpts
elif [ "$DE" = "gnome" ]; then
checkdep "gconftool"
# Set the GNOME 2 wallpaper
gconftool-2 -s -t string /desktop/gnome/background/picture_filename "$tfn"

# Set the GNOME 2 wallpaper picture options
gconftool-2 -s -t string /desktop/gnome/background/picture_options "$picOpts"
elif [ "$DE" = "gnome3" ]; then
checkdep "gsettings"
# Set the GNOME3 wallpaper
DISPLAY=:0 GSETTINGS_BACKEND=dconf gsettings set org.gnome.desktop.background picture-uri '"file://'$tfn'"'

# Set the GNOME 3 wallpaper picture options
DISPLAY=:0 GSETTINGS_BACKEND=dconf gsettings set org.gnome.desktop.background picture-options $picOpts
gsettings set org.gnome.desktop.background picture-uri '"file://'$tfn'"'
elif [ "$DE" = "kde" ]; then
checkdep "xdotool"
checkdep "gettext"
LOCALE=$(echo $LANG | sed 's/\..*$//')

EN_CONSOLE1="Desktop Shell Scripting Console"
EN_CONSOLE2="Plasma Desktop Shell"

if [ -n $LOCALE ]; then
    JS_CONSOLE1=$(LANGUAGE=$LOCALE gettext -d plasma-desktop -s "$EN_CONSOLE1")
    JS_CONSOLE2=$(LANGUAGE=$LOCALE gettext -d plasma-desktop -s "$EN_CONSOLE2")
    JS_CONSOLE="$JS_CONSOLE1 – $JS_CONSOLE2"
else
    JS_CONSOLE="$EN_CONSOLE1 – $EN_CONSOLE2"
fi

js=$(mktemp)
cat << _EOF > $js

var wallpaper = "$tfn"; var activity = activities()[0]; activity.currentConfigGroup = new Array("Wallpaper", "image"); activity.writeConfig("wallpaper", wallpaper); activity.writeConfig("userswallpaper", wallpaper); activity.reloadConfig(); _EOF qdbus org.kde.plasma-desktop /App local.PlasmaApp.loadScriptInInteractiveConsole "$js" > /dev/null xdotool search --name "$JS_CONSOLE" windowactivate key ctrl+e key ctrl+w rm -f "$js"

elif [ "$DE" = "lxde" ]; then
  checkdep "pcmanfm"
  pcmanfm -w "$tfn"
  pcmanfm --wallpaper-mode=center
  pcmanfm --wallpaper-mode=stretch
elif [ "$DE" = "lxqt" ]; then
  checkdep "pcmanfm-qt"
  pcmanfm-qt -w "$tfn"
elif [ "$DE" = "mac" ]; then
# set target filename 4 mac
tfnnew=$(ctfn)

mv $tfn $tfnnew
rm -f $tfnold

osascript -e 'tell application "Finder" to set desktop picture to POSIX file "'"$tfnnew"'"'
osafirstResult=$?
osascript -e 'tell application "System Events" to set picture of every desktop to "'"$tfnnew"'"'
osasecondResult=$?
sqlite3 ~/Library/Application\ Support/Dock/desktoppicture.db "update data set value = '$tfnnew'" 2>&1 >/dev/null
sqliteResult=$?

tfnold=$tfnnew
elif [ "$DE" = "mate" ]; then
  checkdep "dconf"
  dconf write /org/mate/desktop/background/picture-filename '"'$tfn'"'
elif [ "$DE" = "xfce" ]; then
checkdep "xfconf-query"
# set to every monitor that contains image-path/last-image
properties=$(xfconf-query -c xfce4-desktop -p /backdrop -l | grep -e "screen.*/monitor.*image-path$" -e "screen.*/monitor.*/last-image$")

for property in $properties; do
    xfconf-query -c xfce4-desktop -p $property -s "$tfn"
done
 elif [ $DE = "WM" ]; then
checkdep "feh"
feh --bg-tile "$tfn"
fi

if [ "$exitAfterRunning" = true ] ; then
  # Exit the script
  exit 0
fi

# sleep for half a day
DIFF_TIME=0
LAST_RUN=$(date +%s)
LAST_DAY=$(date +%A)
LAST_HOUR=$(date +%I)
while [ $DIFF_TIME -lt 43199 ]; do
NOW=$(date +%s)
DIFF_TIME=$(expr $NOW - $LAST_RUN)
sleep 60
done

done

Fabioamd87 commented 7 years ago

Hi daniela, The script you've posted works correctly.

PS: I've noticed however, that the script never ends, also after changed the wallpaper, it would be nice to change the wallpaper with one command line.

dzmanto commented 7 years ago

Dear Fabio:

Thank you so much for your feedback. I really appreciate it. I am going to make my script available on https://github.com/dzmanto/linux-bing-wallpaper asap.

Cheers,

Daniela

From: Fabioamd87 [notifications@github.com] Sent: Wednesday, 26 April 2017 10:22 PM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Hi daniela, The script you've posted works correctly.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-297388328, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM3oe5klmCKmKCqpLwnMBq0PKxT9eks5rzzcBgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

Hi Daniela, Let me understand, it is a different repo?

dzmanto commented 7 years ago

Dear Fabio:

https://github.com/dzmanto/linux-bing-wallpaper is a fork from the main repo. My fork enables me to be more agile when patching linux-bing-wallpaper.

Cheers,

Daniela

-- That's me in the corner ... that's me in the spotlight ... shooting my computer

From: Fabioamd87 [notifications@github.com] Sent: Thursday, 27 April 2017 5:33 PM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Hi Daniela, Let me understand, it is a different repo?

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-297635166, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSMzpxHwQn5pWWwUDIKAmYSbkV419tks5r0ES4gaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

Thanks Daniela, As a suggest, do I need to use your repo or I can wait the main repo to be updetd? I saw that last updates on the main repo are months ago.

dzmanto commented 7 years ago

Dear Fabio:

The main repo is no longer updated on a regular basis. I suggest you use my repo.

Cheers,

Daniela

-- The quieter you become, the more you are able to hear.

From: Fabioamd87 [notifications@github.com] Sent: Thursday, 27 April 2017 10:33 PM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Thanks Daniela, As a suggest, do I need to use your repo or I can wait the main repo to be updetd? I saw that last updates on the main repo are months ago.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-297700226, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSMwTIGMf6W9-aXwVl84yTorMmBcUzks5r0IsUgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

Hi Daniela, I have noticed that the image file are no longer saved in the Bing folder, do you confirm this?

thanks, Fabio

dzmanto commented 7 years ago

Dear Fabio:

This is indeed confirmed. I moved the files to the /tmp folder. Linux distributions as well as Mac OS may clean up the /tmp folder on a regular basis. That is why I consider the /tmp directory a more appropriate folder. The bing folder should by now be obsolete.

Cheers,

Daniela

From: Fabioamd87 [notifications@github.com] Sent: Wednesday, 3 May 2017 12:57 AM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Hi Daniela, I have noticed that the image file are no longer saved in the Bing folder, do you confirm this?

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-298660708, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM_C9duZM4VbIKQj-JWuES8t-fklFks5r10RXgaJpZM4Mxi4Z.

Fabioamd87 commented 7 years ago

Hi Daniela, I think this is a correct way of thinking, but it is also nice to have save this beautiful wallpaper somewhere. What do you think about that? Maybe I can work on that.

dzmanto commented 7 years ago

Dear Fabio:

Sure you can work on saving wallpapers. Your contribution will be more than welcome.

Actually, by storing wallpapers in the /tmp folder I try and minimise disk usage.

Cheers,

Daniela

From: Fabioamd87 [notifications@github.com] Sent: Wednesday, 3 May 2017 6:22 PM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Comment Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Hi Daniela, I think this is a correct way of thinking, but it is also nice to have save this beautiful wallpaper somewhere. What do you think about that? Maybe I can work on that.

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-298849859, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSMw1FuYW3pUpIJAwkZifUgkVqiqWQks5r2DkqgaJpZM4Mxi4Z.

bugz8unny69 commented 5 years ago

Hi @dzmanto, I checked out your forked repo, 1. I see you're using ba(sh) instead of golang and 2. unfortunately you don't have issues enable for that repo. Should we be reporting issues on this ticket?

dzmanto commented 5 years ago

Hi @lhorace, how are issues enabled for this repository? Also, you are more than welcome to report issues. Cheers, Dani

bugz8unny69 commented 5 years ago

Hi @dzmanto, check out https://help.github.com/articles/disabling-issues/, simply do the reverse?

You script in it's current implementation right now, attempts to download whatever image BING offers, than later resize image to higher resolution, however unfortunately, convert here is returning a black image on convert -resize. I would rather prefer attempting to download the highest resolution image possible, than fallback to whatever image BING gives, than resize. This would avoid image quality lost by up-scaling a image in some cases?

dzmanto commented 5 years ago

Hi Horace,

Thank you for your feedback.

Do you want to propose code that downloads the highest resolution image possible?

Cheers,

Dani

From: Horace notifications@github.com Sent: Friday, 26 October 2018 7:33 AM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Mention Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Hi @dzmantohttps://github.com/dzmanto, check out https://help.github.com/articles/disabling-issues/, simply do the reverse?

You script in it's current implementation right now, attempts to download whatever image BING offers, than later resize image to higher resolution, however unfortunately, convert here is returning a black image on convert -resize. I would rather prefer attempting to download the highest resolution image possible, than fallback to whatever image BING gives, than resize. This would avoid image quality lost by up-scaling a image ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-433195721, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM9NaNj9BceK7skcx8cHzKitZuTjBks5uoiAbgaJpZM4Mxi4Z.

dzmanto commented 5 years ago

Hi Horace,

I managed to enable issues. Thanks for your hint.

If you want to propose code to overcome the resolution issue, please go ahead.

Cheers,

Daniela

From: Horace notifications@github.com Sent: Friday, 26 October 2018 7:33 AM To: marguerite/linux-bing-wallpaper Cc: dzmanto; Mention Subject: Re: [marguerite/linux-bing-wallpaper] LXDE Support? (#17)

Hi @dzmantohttps://github.com/dzmanto, check out https://help.github.com/articles/disabling-issues/, simply do the reverse?

You script in it's current implementation right now, attempts to download whatever image BING offers, than later resize image to higher resolution, however unfortunately, convert here is returning a black image on convert -resize. I would rather prefer attempting to download the highest resolution image possible, than fallback to whatever image BING gives, than resize. This would avoid image quality lost by up-scaling a image ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/marguerite/linux-bing-wallpaper/issues/17#issuecomment-433195721, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALJSM9NaNj9BceK7skcx8cHzKitZuTjBks5uoiAbgaJpZM4Mxi4Z.

bugz8unny69 commented 5 years ago

Sure, when I get time to do so! It's just rearranging your code, not to much work!

marguerite / linux-bing-wallpaper

LXDE Support? #17

chmod +x bing_wallpaper.sh

./bing_wallpaper.sh

!/bin/sh

Author: Marguerite Su i@marguerite.su, dzmanto dzmanto@hotmail.com

License: GPL-3.0

Description: Download Bing Wallpaper of the Day and set it as your Linux Desktop.

https://github.com/marguerite/linux-bing-wallpaper

global options

$bing is needed to form the fully qualified URL for

the Bing pic of the day

The idx parameter determines where to start from. 0 is the current day,

1 the previous day, etc.

$xmlURL is needed to get the xml data from which

the relative URL for the Bing pic of the day is extracted

Set picture options

Valid options are: none,wallpaper,centered,scaled,stretched,zoom,spanned

The file extension for the Bing pic

see https://bugs.freedesktop.org/show_bug.cgi?id=34164

The mkt parameter determines which Bing market you would like to

obtain your images from.

Try and guess language

Valid values are:

inhibit code injection

set target filename

Download the highest resolution