Subject: Re: [alpine-aports] [PATCH] main/dnsmasq: Added description to init.d
 script
To: Natanael Copa <ncopa@alpinelinux.org>
References: <1446226418-18028-1-git-send-email-danielsan.dani@gmail.com>
 <20151103085244.1080136d@ncopa-desktop.alpinelinux.org>
Cc: alpine-aports@lists.alpinelinux.org
From: Daniele <danielsan.dani@gmail.com>
Message-ID: <56389046.3060103@gmail.com>
Date: Tue, 3 Nov 2015 11:45:26 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.3.0
Precedence: list
MIME-Version: 1.0
In-Reply-To: <20151103085244.1080136d@ncopa-desktop.alpinelinux.org>
Content-Type: multipart/alternative;
 boundary="------------080407060507060803050607"

This is a multi-part message in MIME format.
--------------080407060507060803050607
Content-Type: text/plain; charset=iso-8859-15; format=flowed
Content-Transfer-Encoding: 7bit

Hi Natanael,
Sorry for my mistakes...
What I did was to modify init.d file and then to recalculate checksum 
using md5sum and sha256sum on file; I updated the APKBUILD manually with 
new values.

 From your previous mail I understood I must update APKBUILD by issuing 
command 'abuild checksum'. Is that right?
If so, I will correct and re-submit patches.

Sorry again, it's my first patches.

Regards,

Daniele Coli`

-=-=-=
Love is not blind, it is far-sighted: the proof
lies in the fact that it begins to see defects
as they gradually slip away .
		-- Oscar Wilde
--=-=-=

On 11/03/2015 08:52 AM, Natanael Copa wrote:
> Hi, thanks for cleaning up the init.d scripts.
>
> On Fri, 30 Oct 2015 18:33:38 +0100
> Daniele Coli <danielsan.dani@gmail.com> wrote:
>
>> Added description to init.d script in order to make it compliant
>> with rc-system and avoid annoying warning messages on syslog.
>> ---
>>   main/dnsmasq/APKBUILD      | 6 +++---
>>   main/dnsmasq/dnsmasq.initd | 3 +++
>>   2 files changed, 6 insertions(+), 3 deletions(-)
> The checksum for this failed but i fixed it for you. but this was the
> only i fixed for you.
>
> It would be nice if future patches actually build.
>
> Thanks.
>
> -nc
>


--------------080407060507060803050607
Content-Type: text/html; charset=iso-8859-15
Content-Transfer-Encoding: 7bit

<html>
  <head>
    <meta content="text/html; charset=iso-8859-15"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix"><font face="Helvetica, Arial,
        sans-serif">Hi Natanael,<br>
        Sorry for my mistakes...<br>
        What I did was to modify init.d file and then to recalculate
        checksum using md5sum and sha256sum on file; I updated the
        APKBUILD manually with new values.<br>
        <br>
        From your previous mail I understood I must update APKBUILD by
        issuing command 'abuild checksum'. Is that right?<br>
        If so, I will correct and re-submit patches.<br>
        <br>
        Sorry again, it's my first patches.<br>
        <br>
        Regards,</font><br>
      <pre class="moz-signature" cols="72">Daniele Coli`

-=-=-=
Love is not blind, it is far-sighted: the proof
lies in the fact that it begins to see defects
as they gradually slip away .
		-- Oscar Wilde
--=-=-=
</pre>
      On 11/03/2015 08:52 AM, Natanael Copa wrote:<br>
    </div>
    <blockquote
      cite="mid:20151103085244.1080136d@ncopa-desktop.alpinelinux.org"
      type="cite">
      <pre wrap="">Hi, thanks for cleaning up the init.d scripts.

On Fri, 30 Oct 2015 18:33:38 +0100
Daniele Coli <a class="moz-txt-link-rfc2396E" href="mailto:danielsan.dani@gmail.com">&lt;danielsan.dani@gmail.com&gt;</a> wrote:

</pre>
      <blockquote type="cite">
        <pre wrap="">Added description to init.d script in order to make it compliant
with rc-system and avoid annoying warning messages on syslog.
---
 main/dnsmasq/APKBUILD      | 6 +++---
 main/dnsmasq/dnsmasq.initd | 3 +++
 2 files changed, 6 insertions(+), 3 deletions(-)
</pre>
      </blockquote>
      <pre wrap="">
The checksum for this failed but i fixed it for you. but this was the
only i fixed for you.

It would be nice if future patches actually build.

Thanks.

-nc

</pre>
    </blockquote>
    <br>
  </body>
</html>

--------------080407060507060803050607--


---
Unsubscribe:  alpine-aports+unsubscribe@lists.alpinelinux.org
Help:         alpine-aports+help@lists.alpinelinux.org
---