Skip to content

Aggiunte opzioni notify e help, settata la creazione della variabile $MANNAGGIAURL solo se l'audioflag è vera #16

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Nhoya wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Aggiunte opzioni notify e help, settata la creazione della variabile $MANNAGGIAURL solo se l'audioflag è vera #16

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8b573ea
Aggiunta opzione per mannaggiare via notifica desktop (--notify) e la…
Feb 13, 2016
fd8ea8d
Update mannaggia.sh
Nhoya Feb 13, 2016
6aaa0ef
la variabile MANNAGGIAURL deve esistere solo se l'audio flag è attiva
Nhoya Feb 14, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 24 additions & 3 deletions mannaggia.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,10 +27,18 @@ DELSTRING1="</FONT>"
DELSTRING2="</b>"
DEFPLAYER="mplayer -really-quiet -ao alsa"
PLAYER="${PLAYER:-$DEFPLAYER}"

notify=0
help=0
# lettura parametri da riga comando
for parm in "$@"
do
if [ "$parm" = "--help" ];then
help=1
fi
if [ "$parm" = "--notify" ]
then
notify=1
fi
# leggi dai parametri se c'e' l'audio
if [ "$parm" = "--audio" ]
then
Expand All @@ -44,7 +52,7 @@ for parm in "$@"

# leggi dai parametri se c'e' da mandare i commenti su wall
if [ "$parm" = "--wall" ]
then
then
wallflag=true
fi

Expand Down Expand Up @@ -90,7 +98,7 @@ while [ "$nds" != 0 ]
# shellcheck disable=SC2019
MANNAGGIA="Mannaggia $(curl -s "www.santiebeati.it/$(</dev/urandom tr -dc A-Z|head -c1)/"|grep -a tit|cut -d'>' -f 4-9|shuf -n1 |awk -F "$DELSTRING1" '{print$1$2}'|awk -F "$DELSTRING2" '{print$1}')"
MANNAGGIAURL="http://www.ispeech.org/p/generic/getaudio?text=$MANNAGGIA%2C&voice=euritalianmale&speed=0&action=convert"

if [ "$wallflag" = true ]
then
pot=$(( nds % 50 ))
Expand All @@ -101,7 +109,20 @@ while [ "$nds" != 0 ]
# attenzione: se non siete root o sudoers dovete togliere dalla riga successiva "sudo" e "-n"
echo "$MANNAGGIA" | sudo wall -n
fi
#mannaggia via notifica desktop
elif [ "$notify" = 1 ]; then
if ( ! type "notify-send" >/dev/null 2>&1 ); then
echo "Ti serve notify-send"
exit 1
else
notify-send "$MANNAGGIA" 2>/dev/null
fi
elif [ "$help" = 1 ]; then
Copy link
Copy Markdown
Contributor

@rfc1459 rfc1459 Feb 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Da un punto di vista puramente logico, se l'utente chiede l'help ogni azione dovrebbe essere soppressa, quindi il check dovrebbe essere in cima

echo "Uso: ./mannaggiatore.sh [opzioni]\nQuesto script provvede a nominare tutti i santi per voi quando vi sentite depressi.\n\nOpizoni:\n --audio: attiva mplayer per fargli pronunciare i santi
Copy link
Copy Markdown
Contributor

@rfc1459 rfc1459 Feb 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Diversi problemi:

  • nome dello script hardcodato (trivial)
  • SC2028: echo won't expand escape sequences. Consider printf. (critical)
  • mplayer non è necessariamente il player di default (minor)

Copy link
Copy Markdown
Author

@Nhoya Nhoya Feb 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per quanto riguarda l'usage, ho preso le descrizioni che avete dato voi alle opzioni
Provvederò al più preso a sostituire echo con printf

--spm: numero di santi per minuto\n --wall: invia l'output a tute le console: atenzione, se non siete root o sudoers disattivare il flag -n\n --notify: invia una notifica desktop\n --help: mostra questa pagina\n\n Buona managgiata a tutti e ricordate: $MANNAGGIA systemd merda 0/"
Copy link
Copy Markdown
Contributor

@rfc1459 rfc1459 Feb 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/atenzione/attenzione/

Copy link
Copy Markdown
Author

@Nhoya Nhoya Feb 13, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Refuso

exit 0

else
echo "$MANNAGGIA" > /dev/stdout
fi

Expand Down