[Buildroot] [PATCH] aircrack-ng: new package
arnout at mind.be
Fri Feb 8 20:26:22 UTC 2013
On 08/02/13 16:32, Thomas Petazzoni wrote:
> Dear Laurent Cans,
> Thanks, this looks pretty good; A few comments below.
Yep, don't be discouraged by the large amount of feedback :-)
> On Fri, 8 Feb 2013 17:16:29 +0100, Laurent Cans wrote:
>> Aircrack-ng is a set of tools for auditing wireless security.
>> Although the last version is 1.1, I choose to package last svn version due to a fixed bug which fix a failed compilation.
It would be good to repeat this comment in the .mk file. That helps to
see if a certain packaged upstream version is in fact more recent than
what is already there.
>> + AIRCRACK_NG_OPTS = sqlite=true
I prefer AIRCRACK_NG_MAKE_OPTS.
>> + AIRCRACK_NG_DEPENDENCIES += sqlite
I would add an else branch that explicitly sets sqlite=false, to make
sure it doesn't leak in through the environment.
> The src/Makefile contains:
> ifeq ($(subst TRUE,true,$(filter TRUE true,$(sqlite) $(SQLITE))),true)
> LIBSQL = -L/usr/local/lib -lsqlite3
> LIBSQL =
> This -L/usr/local/lib isn't really nice since this is not where sqlite
> will be installed, and it may cause issues when cross-compiling.
The solution is simple:
AIRCRACK_NG_MAKE_OPTS += LIBSQL="-lsqlite3"
(no -L is needed because libsqlite3.a is in the default library search path).
>> +define AIRCRACK_NG_BUILD_CMDS
>> + $(MAKE) CC="$(TARGET_CC)" LD="$(TARGET_LD)" -C $(@D) $(AIRCRACK_NG_OPTS) all
> Use $(TARGET_CONFIGURE_OPTS) :
> $(MAKE) $(TARGET_CONFIGURE_OPTS) -C $(@D) $(AIRCRACK_NG_OPTS) all
If it works, I prefer to put the TARGET_CONFIGURE_OPTS in the
$(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) $(AIRCRACK_NG_MAKE_OPTS) all
That gives the Makefile the possibility to append to CFLAGS and LDFLAGS,
if necessary. However, some Makefiles unconditionally set CFLAGS or
LDFLAGS (or even CC), in that case passing through the environment
> And ditto for the other commands.
> Best regards,
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F
More information about the buildroot