| Message ID | 1523451771-29353-1-git-send-email-guillaume.bressaix@gmail.com |
|---|---|
| State | Changes Requested |
| Headers | show |
| Series | [1/1] adding new package: reaver | expand |
Hello, Thanks for this contribution. See below for some comments. First, the commit title should be "reaver: new package" On Wed, 11 Apr 2018 15:02:51 +0200, Guillaume W. Bres wrote: > Generic autotools package, easy to cross-compile but it is mandatory > to use the t6x github fork (latest Makefiles set). What is the "t6x github fork" ? > package/Config.in | 1 + > package/reaver/Config.in | 8 ++++++++ > package/reaver/reaver.mk | 15 +++++++++++++++ Addition of an entry in the DEVELOPERS file is missing, and also the hash file is missing. Could you add these ? > diff --git a/package/reaver/Config.in b/package/reaver/Config.in > new file mode 100644 > index 0000000..703fa28 > --- /dev/null > +++ b/package/reaver/Config.in > @@ -0,0 +1,8 @@ > +config BR2_PACKAGE_REAVER > + bool "reaver" > + Unneeded empty new line. > + select BR2_PACKAGE_LIBPCAP > + Ditto. Also, could you test your package with ./utils/test-pkg, to make sure it builds properly with some basic toolchain configurations ? > + help > + Reaver performs brute force attacks against WPS pin numbers. > + It is recommended to use "Aircrack-ng" along reaver, but not necessary. Lines are too long, they should be wrapped at 72 characters. If you run ./utils/check-package package/reaver/* you will see a number of warnings about your package. They should be fixed before submission. Also, please add the URL of the upstream project in the Config.in help text. See how it's done in other packages. > diff --git a/package/reaver/reaver.mk b/package/reaver/reaver.mk > new file mode 100644 > index 0000000..8e4e3c6 > --- /dev/null > +++ b/package/reaver/reaver.mk > @@ -0,0 +1,15 @@ > +################################################################################ > +# > +# reaver > +# > +################################################################################ > + > +REAVER_VERSION = 9bae55bd30b6d46b42da3a09dc23c8b0f9341996 > +REAVER_SITE = $(call github,t6x,reaver-wps-fork-t6x,$(REAVER_VERSION)) > +REAVER_LICENSE = GPL-2.0+ > +REAVER_LICENSE_FILES = docs/LICENSE > + > +REAVER_SUBDIR = src > +REAVER_INSTALL_TARGET = YES This last line is not needed. If you could fix those issues and submit an updated version, it would be nice. Don't forget to use "git format-patch -v2" for the new version, and add a changelog of what has been changed. Thanks! Thomas Petazzoni
diff --git a/package/Config.in b/package/Config.in index e48d0b4..2da8605 100644 --- a/package/Config.in +++ b/package/Config.in @@ -1823,6 +1823,7 @@ endif source "package/quagga/Config.in" source "package/rabbitmq-server/Config.in" source "package/radvd/Config.in" + source "package/reaver/Config.in" source "package/rp-pppoe/Config.in" source "package/rpcbind/Config.in" source "package/rsh-redone/Config.in" diff --git a/package/reaver/Config.in b/package/reaver/Config.in new file mode 100644 index 0000000..703fa28 --- /dev/null +++ b/package/reaver/Config.in @@ -0,0 +1,8 @@ +config BR2_PACKAGE_REAVER + bool "reaver" + + select BR2_PACKAGE_LIBPCAP + + help + Reaver performs brute force attacks against WPS pin numbers. + It is recommended to use "Aircrack-ng" along reaver, but not necessary. diff --git a/package/reaver/reaver.mk b/package/reaver/reaver.mk new file mode 100644 index 0000000..8e4e3c6 --- /dev/null +++ b/package/reaver/reaver.mk @@ -0,0 +1,15 @@ +################################################################################ +# +# reaver +# +################################################################################ + +REAVER_VERSION = 9bae55bd30b6d46b42da3a09dc23c8b0f9341996 +REAVER_SITE = $(call github,t6x,reaver-wps-fork-t6x,$(REAVER_VERSION)) +REAVER_LICENSE = GPL-2.0+ +REAVER_LICENSE_FILES = docs/LICENSE + +REAVER_SUBDIR = src +REAVER_INSTALL_TARGET = YES + +$(eval $(autotools-package))
Generic autotools package, easy to cross-compile but it is mandatory to use the t6x github fork (latest Makefiles set). Tested and used on RPI1 & RPI3. Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com> --- package/Config.in | 1 + package/reaver/Config.in | 8 ++++++++ package/reaver/reaver.mk | 15 +++++++++++++++ 3 files changed, 24 insertions(+) create mode 100644 package/reaver/Config.in create mode 100644 package/reaver/reaver.mk