[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[pkg] brutespray - review



On Thu, 20 Jul 2017, Lukas Schwaighofer wrote:
> * Is there a reason this package has the debian/source/options file? (I
>   still don't know what these options are used for and I remember you
>   dropped them in curvedns after I asked?)

FYI debian/source/options are options for dpkg-source.

On Fri, 21 Jul 2017, Lukas Schwaighofer wrote:
> * debian/control: Priority standard is too high (that's basically a
>   statement, that you expect this package to be installed on a "normal"
>   installation). It should probably be either optional or extra (I do
>   lean towards extra).  Check the Debian Policy [1] for an explanation.

Please use optional. "extra" is going to disappear soon (check bugs
against debian-policy if you want to find out the proposal)
and is generally mis-used.

Cheers,
-- 
Rapha?l Hertzog ? Debian Developer

Support Debian LTS: https://www.freexian.com/services/debian-lts.html
Learn to master Debian: https://debian-handbook.info/get/



Reply to: