On Mon, Feb 17, 2014 at 4:21 PM,  <billc...@free.fr> wrote:
> Subject: [PATCH] [packages] ddns services :Please Add dyndns service providers

"please" should not be in the subject. The package name is
ddns-scripts, not "ddns services".

> Adding dyndns service providers (dyndns.fr, dyndnspro.com and 
> dynamidomain.net).

This line is okay, but check the spelling of the domain names. Also
you are adding four providers, not three; webdomotic.com is missing
from this list.

> Hi, could add those please. We need.  Give chance for all. Thanks

This should not be part of the commit message.

>
>
> Signed-off-by: nabilion (N. Chaniour) <billc...@free.fr>

This is not a proper Signed-off-by; it should include your full name
and no nicknames.

>
> Index: packages/net/ddns-scripts/files/usr/lib/ddns/services
> ===================================================================
> --- packages/net/ddns-scripts/files/usr/lib/ddns/services       (revision 
> 39585)
> +++ packages/net/ddns-scripts/files/usr/lib/ddns/services       (copie de 
> travail)
> @@ -8,11 +8,22 @@
>  # if you need to enter the same text in the url (which seems very unlikely) 
> put
>  # that text in lowercase, while the variables should remain in uppercase
>
> +###Dyndns.fr
> +"dyndns.fr"            
> "http://[DOMAIN]:[PASSWORD]@dyndns.dyndns.fr/update.php?hostname=[DOMAIN]&myip=[IP]";
> +##Dynamicdomain.net
> +"dynamicdomain.net"     
> "http://[DOMAIN]:[PASSWORD]@dyndns.dynamicdomain.net/update.php?hostname=[DOMAIN]&myip=[IP]";
> +####Dyndnspro.com
> +"dyndnspro.com"                
> "http://[DOMAIN]:[PASSWORD]@dyndns.dyndnspro.com/update.php?hostname=[DOMAIN]&myip=[IP]";
> +####"webdomotic"
> +"webdomotic.com"                
> "http://[DOMAIN]:[PASSWORD]@dyndns.webdomotic.com/update.php?hostname=[DOMAIN]&myip=[IP]";

What's up with all these random amount of #? Please use only one.

Also, this patch does not apply:

patching file net/ddns-scripts/files/usr/lib/ddns/services
patch: **** malformed patch at line 38:
_______________________________________________

Please fix and resubmit. And don't spam the list with resubmitting the
same patch over and over again, sending it once is enough.


Regards
Jonas
_______________________________________________
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel

Reply via email to