On 2014-08-31 08:58:49, Roman Valls Guimera wrote:
> All good points, fixed in my last two commits on GitHub. Let me know what you 
> think.

Thank you for those changes.

It seems you are patching upstream directly in the git
repo. Unfortunately, the Debian archive doesn't yet support the "git"
format for uploads, so we still need to create patches and ship them in
debian/patches when we change upstream. So those last two commits
changing the Makefile should probably be reverted.

*but* since those are only Makefile settings, it should be possible to
pass in the proper value in the Makefile.

I *think* you should be able to setup those variables with a target
like:

override_dh_auto_build:
        dh_auto_build -- -DWANT_ACCESSLIST_BLACK \
        -DWANT_IP_FROM_QUERY_STRING -DWANT_COMPRESSION_GZIP \
        -DWANT_IP_FROM_PROXY \
        -DWANT_LOG_NUMWANT -DWANT_MODEST_FULLSCRAPES \
        -DWANT_SPOT_WOODPECKER -DWANT_SYSLOGS

PREFIX should be automatically fed from the regular dh processes, CC as
well. The default LIBOWFAT settings seemed alright.

> If you think there needs to be further followup changes, sure, give me 
> credentials on Debian repo.

For this you would need to create an account on alioth, then I can ask
for an authorization:

https://lists.debian.org/debian-devel-announce/2012/01/msg00006.html

Thanks!

A.

-- 
Men often become what they believe themselves to be. If I believe I
cannot do something, it makes me incapable of doing it. But when I
believe I can, then I acquire the ability to do it even if I didn't
have it in the beginning.
                         - Mahatma Gandhi

Attachment: pgp0J2TkLq_op.pgp
Description: PGP signature

Reply via email to