submitting a patch

[Markus Wolf]

Salve palermo,

  forse qualcuno di voi sa della webradio che stiamo tentanto di
metter su in citta` sullo spazio gentilmente offertoci da indivia.net.
Marcello Orlando e Faboski Hustuniu hanno realizzato il sito web della
radio che si trova all'indirizzo http://www.radiofranca.org. Per
streammare abbiamo scelto mpd (Music Player Daemon,
http://www.musicpd.org ) che e' in grado di interfacciarsi con
icecast2. Uno dei pochi nei di questo bel software consiste nel fatto
che non pubblica l'url della homepage del sito di riferimento della
radio: per questa ragione il sito della radio non e` raggiungibile
dalla liste degli stream attivi ( http://webstream.indivia.net
http://radio.autistici.org). Strano perche' la cosa non e' difficile,
a mio avviso. Ho scaricato l'svn di mpd, gli ho dato un'occhiata e ho
scritto una microscopica patch per aggiungere questa funzionalita`.
Poiche` non l'ho mai fatto prima, prima di inviare la patch all'autore
di mpd volevo chiedervi un parere su quello che ho fatto:

[n4rc0s@maggie Devel]$ diff -Naur mpd/ new/mpd >show_url.diff
[n4rc0s@maggie Devel]$ cat shout_url.diff
diff -Naur mpd/src/audioOutputs/audioOutput_shout.c
new/mpd/src/audioOutputs/audioOutput_shout.c
--- mpd/src/audioOutputs/audioOutput_shout.c    2007-07-27
10:01:09.000000000 +0200
+++ new/mpd/src/audioOutputs/audioOutput_shout.c        2007-07-27
09:52:43.000000000 +0200
@@ -245,6 +245,13 @@
                      param->line, shout_get_error(sd->shoutConn));
        }

+       blockParam = getBlockParam(param, "url");
+       if (blockParam && shout_set_url(sd->shoutConn,
+                                       blockParam->value)) {
+               FATAL("error configuring shout defined at line %i: %s \n",
+                     param->line, shout_get_error(sd->shoutConn));
+       }
+
        {
                char temp[11];
                memset(temp, 0, sizeof(temp));

e la patch si applica nel solito modo:
[n4rc0s@maggie Devel]$ patch -p0 <shout_url.diff
patching file mpd/src/audioOutputs/audioOutput_shout.c

dopo aver patchato infatti:
[n4rc0s@maggie Devel]$ diff -s
mpd/src/audioOutputs/audioOutput_shout.c
new/mpd/src/audioOutputs/audioOutput_shout.c
Files mpd/src/audioOutputs/audioOutput_shout.c and
new/mpd/src/audioOutputs/audioOutput_shout.c are identical

sbaglio qualcosa?

Ciao,
  Markus