new version wifite
Lukas Schwaighofer
lukas at schwaighofer.name
Mon Sep 4 18:01:39 UTC 2017
Hi Daniel,
On Sat, 2 Sep 2017 13:12:50 -0500
Daniel Echeverry <epsilon77 at gmail.com> wrote:
> Some time ago, wifite was updated with minor changes[1]. I updated the
> git repo with theses changes[2], Could someone check it out?
I took a look and updating seems reasonable. However, you should think
about your versioning scheme. The Debian versions up to now were called
2.0r85-1
2.0.85~gitedbdedd14-1
and you want to call the new one
2.0.85+git918a499786-1
That doesn't look like there is a system to that. I'd suggest you
adopt a schema that includes the date of the commit as YYYYMMDD (before
a short version of the git sha1). As the REVISION variable has been
set to 87, an idea for the version could be:
2.0.87+git20170515.918a499-1
Other cleanup/nitpick suggestions:
* Remove version depends from python, and the X-Python-Version [1] from
d/control; there is only one python2 version left in the archive
(even oldoldstable has 2.7).
* There's policy 4.1.0 now
* Link to copyright format in debian/copyright can be https
And finally I have a request to make reviewing your changes easier:
Please don't make any changes to debian/* files as part of the merge
commit. If you make those changes in separate commits after the merge,
their diffs are a lot easier to read (at least for me).
Regards
Lukas
[1] https://www.debian.org/doc/packaging-manuals/python-policy/ch-module_packages.html#s-specifying_versions
More information about the Pkg-security-team
mailing list