[Buildroot] [PATCH 1/1] aircrack-ng: bump to version 1.2
Fabrice Fontaine
fontaine.fabrice at gmail.com
Tue May 29 20:33:20 UTC 2018
Dear Peter,
2018-05-29 21:34 GMT+02:00 Peter Korsgaard <peter at korsgaard.com>:
> >>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice at gmail.com> writes:
>
> > - Remove patch (already in version)
> > - Add three patches (first one sent and merged upstream, second and
> > third patchs retrieved upstream)
> > - Move to autotools-package
> > - Remove workaround for static handling of pthread as it isn't needed
> > anymore
> > - Remove deactivation of libnl in static build as it isn't needed
> > anymore
> > - Update workaround for static handling of libpcap
> > - Add hash for license file
>
> [snip]
>
> > +++ b/package/aircrack-ng/Config.in
> > @@ -3,9 +3,7 @@ config BR2_PACKAGE_AIRCRACK_NG
> > depends on BR2_USE_MMU # uses fork()
> > depends on BR2_TOOLCHAIN_HAS_THREADS
> > depends on BR2_INSTALL_LIBSTDCPP
> > - # libnl has issues when linking statically
> > - # they need fixing in libnl itself
> > - select BR2_PACKAGE_LIBNL if !BR2_STATIC_LIBS
> > + select BR2_PACKAGE_LIBNL
>
> Looking at the m4 files, it looks as if libnl is supposed to be
> optional. If so, then we shouldn't force it here.
>
Indeed, but libnl is set in the Requirements section of the Readme.md (and
not in the "Optional stuff" section) and it was selected since version
1.2-rc1 so I didn't know what to do.
But you're right, I will make it optional.
>
> > select BR2_PACKAGE_OPENSSL
>
> It also looks as if libgcrypt can be used instead of openssl.
>
> > select BR2_PACKAGE_ZLIB
>
> And zlib looks to be optional as well.
>
For libgcrypt and zlib, I have additional patches but I can put everything
in a single patch if you prefer.
>
> > +++ b/package/aircrack-ng/aircrack-ng.mk
> > @@ -4,69 +4,42 @@
> > #
> > ############################################################
> ####################
>
> > -AIRCRACK_NG_VERSION = 1.2-rc4
> > +AIRCRACK_NG_VERSION = 1.2
> > AIRCRACK_NG_SITE = http://download.aircrack-ng.org
> > AIRCRACK_NG_LICENSE = GPL-2.0+
> > AIRCRACK_NG_LICENSE_FILES = LICENSE
> > -AIRCRACK_NG_DEPENDENCIES = openssl zlib host-pkgconf
> > +AIRCRACK_NG_DEPENDENCIES = \
> > + libnl \
> > + openssl \
> > + $(if $(BR2_PACKAGE_PCRE),pcre) \
> > + zlib \
> > + host-pkgconf
> > +AIRCRACK_NG_AUTORECONF = YES
> > # Enable buddy-ng, easside-ng, tkiptun-ng, wesside-ng
> > -AIRCRACK_NG_MAKE_OPTS = unstable=true
> > -
> > -# Account for libpthread in static
> > -AIRCRACK_NG_LDFLAGS = $(TARGET_LDFLAGS) \
> > - $(if $(BR2_STATIC_LIBS),-lpthread -lz)
> > -
> > -# libnl support has issues when building static
> > -ifeq ($(BR2_STATIC_LIBS),y)
> > -AIRCRACK_NG_MAKE_OPTS += libnl=false
> > -else
> > -AIRCRACK_NG_MAKE_OPTS += libnl=true
> > -AIRCRACK_NG_DEPENDENCIES += libnl
> > -endif
> > +AIRCRACK_NG_CONF_OPTS = --with-experimental
>
> Is this the replacement for unstable=true? According to the m4 files,
> this defaults to off.
>
Yes, this is the replacement for unstable. What do you mean by it defaults
to off? Do you want me to disable experimental features?
As a side note, unstable has been replaced by experimental since 1.2-rc2 (
https://github.com/aircrack-ng/aircrack-ng/tree/1.2-rc2) so this is
currently "broken".
>
>
> > ifeq ($(BR2_TOOLCHAIN_HAS_SSP),y)
> > -AIRCRACK_NG_MAKE_OPTS += STACK_PROTECTOR=true
> > +AIRCRACK_NG_CONF_OPTS += --with-opt
> > else
> > -AIRCRACK_NG_MAKE_OPTS += STACK_PROTECTOR=false
> > +AIRCRACK_NG_CONF_OPTS += --without-opt
>
> Hmm, --with-opt seems to be related to adding -O3 to CFLAGS or not:
>
> AC_ARG_WITH(opt,
> [AS_HELP_STRING([--without-opt],
> [disable -O3 optimizations])])
>
> AS_IF([test "x$enable_code_coverage" = "xno"], [
> case $with_opt in
> yes | "")
> AX_CHECK_COMPILE_FLAG([-O3], [
> AX_APPEND_FLAG(-O3, [opt_[]_AC_LANG_ABBREV[]flags])
> ])
> ;;
> esac
> ])
>
Yes, --without-opt will disable O3 but also the stack protector (see
Readme.md and second patch).
>
> There also seems to be a check for ethtool. This by default looks if
> ethtool is available in the path on the host. We should detect if
> BR2_PACKAGE_ETHTOOL is enabled and point it to it.
>
Yes, ethtool is needed for airmon-ng, I will update my patch to manage it.
>
> --
> Bye, Peter Korsgaard
>
Best Regards,
Fabrice
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180529/1ff4c56e/attachment-0002.html>
More information about the buildroot
mailing list