[pkg] CurveDNS - review

Stéphane Neveu stefneveu at gmail.com
Mon Jun 26 06:57:19 UTC 2017


Hi Lukas,

2017-06-26 0:04 GMT+02:00 Lukas Schwaighofer <lukas at schwaighofer.name>:
> Hi Stéphane,
>
> I'm planning to do a full review on Tuesday evening.  I've done a test
> built right now, ran lintian and noticed:
>
> * Your Makefile does not apply the C proprocessor flags $(CPPFLAGS).
>   In your case that means each of the compile commands starting with
>       $(CC) $(CFLAGS)
>   should instead start with
>       $(CC) $(CPPFLAGS) $(CFLAGS)
> * debian/control
>   - Depends: debonf can be dropped (inserted into ${misc:Depends} by
>     dh_installdebconf)
>   - The most current Standards version is now 4.0.0.  You can use the
>     update checklist [1] to help you identify if any changes are
>     necessary.
> * rename templates to curvedns.templates for consistency (missed that
>   one previously)
> * run `debconf-updatepo` and commit after you change curvedns.templates
>   (it's currently outdated)
>
> Good evening
> Lukas
>
> [1] https://www.debian.org/doc/debian-policy/upgrading-checklist.html#s-4.0.0


All done... I hope !

Stephane



More information about the Pkg-security-team mailing list