adding new package: reaver
PATCH 1/1 adding new package: reaver
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.
--- 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
package/Config.in +1 -0
@@ -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"
package/reaver/Config.in new file +8 -0
@@ -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.
package/reaver/reaver.mk new file +15 -0
@@ -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))
Signature · 1 line
2.7.4
PATCH 1/1 adding new package: reaver
Hello,
Thanks for this contribution. See below for some comments.
First, the commit title should be "reaver: new package"
Quoted 2 lines · Guillaume W. Bres
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" ?
Quoted 3 lines
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 ?
package/reaver/Config.in line 3
Quoted 9 lines
package/reaver/Config.in new file +3
-0@@ -0,0 +1,8 @@ +config BR2_PACKAGE_REAVER + bool "reaver" +
Unneeded empty new line.
package/reaver/Config.in line 5
Quoted 2 lines
package/reaver/Config.in new file +2
-0+ 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 ?
package/reaver/Config.in line 8
Quoted 3 lines
package/reaver/Config.in new file +3
-0+ 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.
Quoted 9 lines
package/reaver/reaver.mk new file +3
-0@@ -0,0 +1,15 @@ +################################################################################ +# +# reaver
package/reaver/reaver.mk line 13
Quoted 10 lines
package/reaver/reaver.mk new file +10
-0+# +################################################################################ + +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
Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com