From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: ddns@lists.ipfire.org
Subject: Re: [PATCH] add new provider it's DNS (https://www.itsdns.de)
Date: Sun, 26 May 2019 10:30:35 +0100
Message-ID: <90D15E55-658A-480D-8284-0DDF581B131A@ipfire.org>
In-Reply-To: <80d9fc05-8163-88d5-e482-08b0a7f7c07e@stalhut.de>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============6096727049127510135=="
List-Id: <ddns.lists.ipfire.org>

--===============6096727049127510135==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hello Jan,

Thank you for submitting your patch. It looks good to me.

But unfortunately I wasn=E2=80=99t able to merge it.

I suppose that your email client replaced all tabs with spaces or something:

[root(a)ipfire ddns]# pwclient git-am -p ddns -s 2265
Applying patch #2265 using "git am -s"
Description: add new provider it's DNS (https://www.itsdns.de)
Applying: add new provider it's DNS (https://www.itsdns.de)
error: patch failed: README:67
error: README: patch does not apply
error: patch failed: src/ddns/providers.py:1108
error: src/ddns/providers.py: patch does not apply
Patch failed at 0001 add new provider it's DNS (https://www.itsdns.de)
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
'git am' failed with exit status 128
[root(a)ipfire ddns]# git apply -3 .git/rebase-apply/patch
error: patch failed: README:67
Falling back to three-way merge...
error: patch failed: README:67
error: README: patch does not apply
error: patch failed: src/ddns/providers.py:1108
Falling back to three-way merge...
error: patch failed: src/ddns/providers.py:1108
error: src/ddns/providers.py: patch does not apply

Could you check that and send again?

Best,
-Michael

> On 25 May 2019, at 22:48, Jan Stalhut <jan(a)stalhut.de> wrote:
>=20
> ---
> README                |  1 +
> src/ddns/providers.py | 10 ++++++++++
> 2 files changed, 11 insertions(+)
>=20
> diff --git a/README b/README
> index d170858..1c5ffd6 100644
> --- a/README
> +++ b/README
> @@ -67,6 +67,7 @@ SUPPORTED PROVIDERS:
>        enom.com
>        entrydns.net
>        freedns.afraid.org
> +       itsdns.de
>        joker.com
>        loopia.se
>        myonlineportal.net
> diff --git a/src/ddns/providers.py b/src/ddns/providers.py
> index ea723e5..a9a0b03 100644
> --- a/src/ddns/providers.py
> +++ b/src/ddns/providers.py
> @@ -1108,6 +1108,16 @@ class DDNSProviderFreeDNSAfraidOrg(DDNSProvider):
>                # If we got here, some other update error happened.
>                raise DDNSUpdateError
>=20
> +class DDNSProviderItsdns(DDNSProtocolDynDNS2, DDNSProvider):
> +        handle    =3D "itsdns.de"
> +        name      =3D "it's DNS"
> +        website   =3D "http://www.itsdns.de/"
> +        protocols =3D ("ipv6", "ipv4")
> +
> +        # Information about the format of the HTTP request is to be
> +        # found here: https://www.itsdns.de/dynupdatehelp.htm
> +
> +        url =3D "https://www.itsdns.de/update.php=E2=80=9D
>=20

And you could add another empty line here, because we usually have two empty =
lines after a class.

> class DDNSProviderJoker(DDNSProtocolDynDNS2, DDNSProvider):
>                handle  =3D "joker.com"
> --=20
> 2.17.1
> _______________________________________________
> ddns mailing list
> ddns(a)lists.ipfire.org
> https://lists.ipfire.org/mailman/listinfo/ddns


--===============6096727049127510135==--