[Buildroot] [PATCH 1/1] reaver: new package
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Thu Apr 12 11:32:21 UTC 2018
Hello,
Thanks for this new version, looks much better. A few more comments
below, nothing major.
On Thu, 12 Apr 2018 11:16:59 +0200, Guillaume W. Bres wrote:
> Reaver is a tool to audit networks against brute WPS pins attacks.
>
> It is a generic autotools package, easy to build. Tests passed for the 7 default
> toolchains.
>
> It is mandantory to use the github.com/t6x fork that I set in reaver.mk
> at the moment because other versions/repos of this project won't cross-compile.
> Their makefiles are pretty outdated. It is the only updated and regularly maintained
> repo that I found for this project at the moment.
I think it would make sense to have this explanation before the
REAVER_SITE variable in the .mk file.
> -N: Guillaume William Brs <guillaume.bressaix at gmail.com>
> +N: Guillaume W. Bres <guillaume.bressaix at gmail.com>
This change is not really related. Is it an intended change? It should
preferably be part of a separate patch.
> F: package/liquid-dsp/
> +F: package/reaver/
Indentation is wrong.
> diff --git a/package/reaver/Config.in b/package/reaver/Config.in
> new file mode 100644
> index 0000000..fed433b
> --- /dev/null
> +++ b/package/reaver/Config.in
> @@ -0,0 +1,10 @@
> +config BR2_PACKAGE_REAVER
> + bool "reaver"
> + select BR2_PACKAGE_LIBPCAP
> +
This empty line is not needed.
Best regards,
Thomas Petazzoni
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list