Hi Andreas,

Thanks for the patches, and real fast too!  I have applied them, all three.

For fastrpz, I removed the trailing whitespace, both from the source and
from the patch file.  The patch already applied for me, before your fix.
 With the fix it applies without offsets.  The fix uses -p1, not -p0.

Also, I don't agree that the spelling is improved by lintian.  But to
remove the warning, the patch is applied.

Best regards, Wouter

On 06/03/18 23:32, A. Schulze via Unbound-users wrote:
> Am 06.03.2018 um 11:02 schrieb W.C.A. Wijngaards via Unbound-users:
>> Unbound 1.7.0rc1 maintainers prerelease is available
> Hello Wouter,
> the feature list sounds promising!
> 3 points:
> 1.
> contrib/fastrpz.patch apply but not without additional help from patch
> I have a "fixed" version that apply without warnings. -> attached
> also I received 2 warnings from quilt, the tool to manage patch stacks in 
> Debian,
> while preparing this fastrpz.patch
> Warning: trailing whitespace in line 1390 of daemon/worker.c
> Warning: trailing whitespace in line 879 of services/cache/dns.c
> 2.
> since some months I use a patch that log keytag queries generated by unbound 
> to support trustanchor signaling.
> It's a simple change but it helped us a lot to identify hosts not using the 
> new KSK-2017
> -> log_trust-anchor-signaling.patch
> (I wish, NSD could log such queries, too)
> 3.
> Debian lintian warn about "allow to" should be written as "allow one to"
> -> spelling-error-in-manpage.patch
> That's only from packaging, tommorow I'll /install/ the new version...
> Andreas

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to