Skip to contentSkip to conversation

package/aircrack-ng: fix static build

PATCH New Yann E. MORIN 2 participants 4 messages Nov 16, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 package/aircrack-ng: fix static build

Two fixes in one:

- openssl's libcrypto needs -lz, but because aircrack-ng does not use pkg-config, it misses this dependency. The proper fix would be to use pkg-config, like is done for pcre with patch 0001-Makefile-use-pkg-config-to-find-libpcre-it-s-more-cros.patch but adding -lz is much easier.

- sqlite needs -lpthread, so -lpthread must be added _after_ -lsqlite, but we currently add it before. So, force -lpthread after -lsqlite when we are doing a static build.

Fixes:

    http://autobuild.buildroot.net/results/1ea/1ea877851e9d2aeeaf9d320bef12321ec2925b5b/
    http://autobuild.buildroot.net/results/cdf/cdf0203fc99d5f7e41e508f6d9edb78a0f0ea732/
Cc Gustavo Zacarias <[email protected]>
---
 package/aircrack-ng/aircrack-ng.mk | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
package/aircrack-ng/aircrack-ng.mk +3 -2
index 30298f5..c7fcb63 100644
@@ -14,7 +14,7 @@ AIRCRACK_NG_MAKE_OPTS = unstable=true
 
 # Account for libpthread in static
 AIRCRACK_NG_LDFLAGS = $(TARGET_LDFLAGS) \
-	$(if $(BR2_PREFER_STATIC_LIB),-lpthread)
+	$(if $(BR2_PREFER_STATIC_LIB),-lpthread -lz)
 
 # libnl support has issues when building static
 ifeq ($(BR2_PREFER_STATIC_LIB),y)
@@ -39,9 +39,10 @@ else
 	AIRCRACK_NG_MAKE_OPTS += pcre=false
 endif
 
+# Duplicate -lpthread, because it is also needed by sqlite
 ifeq ($(BR2_PACKAGE_SQLITE),y)
 	AIRCRACK_NG_DEPENDENCIES += sqlite
-	AIRCRACK_NG_MAKE_OPTS += sqlite=true LIBSQL="-lsqlite3"
+	AIRCRACK_NG_MAKE_OPTS += sqlite=true LIBSQL="-lsqlite3 $(if $(BR2_PREFER_STATIC_LIB),-lpthread)"
 else
 	AIRCRACK_NG_MAKE_OPTS += sqlite=false
 endif
Signature · 1 line
1.9.1
#1 lore raw

PATCH 1/1 package/aircrack-ng: fix static build

All,

On 2014-11-16 16:15 +0100, Yann E. MORIN spake thusly:

Quoted 1 line

Two fixes in one:

So, why did I not submit two patches?

Because both fixes are needed to fix the autobuild failures mentioned below...

Quoted 3 lines

Fixes:

    http://autobuild.buildroot.net/results/1ea/1ea877851e9d2aeeaf9d320bef12321ec2925b5b/
    http://autobuild.buildroot.net/results/cdf/cdf0203fc99d5f7e41e508f6d9edb78a0f0ea732/

Regards,
Yann E. MORIN.

Signature · 6 lines
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
#2 lore raw

PATCH 1/1 package/aircrack-ng: fix static build

Quoted 10 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Yann" == Yann E MORIN <[email protected]> writes:

All,
On 2014-11-16 16:15 +0100, Yann E. MORIN spake thusly:

Quoted 1 line

Two fixes in one:

So, why did I not submit two patches?

Because both fixes are needed to fix the autobuild failures mentioned below...

Talking to yourself? ;)

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard
#3 lore raw

PATCH 1/1 package/aircrack-ng: fix static build

Peter, All,

On 2014-11-16 22:46 +0100, Peter Korsgaard spake thusly:

Quoted 4 lines
Quoted 4 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Yann" == Yann E MORIN <[email protected]> writes:

All,
On 2014-11-16 16:15 +0100, Yann E. MORIN spake thusly:

Quoted 8 lines
Quoted 6 lines
Quoted 1 line

Two fixes in one:

So, why did I not submit two patches?

Because both fixes are needed to fix the autobuild failures mentioned below...

Talking to yourself? ;)

No, it was Alter talking to Ego, not me.

Muhahaha! :-)

Quoted 1 line

Committed, thanks.

Thanks!

Regards,
Yann E. MORIN.

Signature · 6 lines
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'