Skip to contentSkip to conversation

axel: bump to version 2.15

PATCH Superseded Ismael Luceno 2 participants 3 messages Oct 16, 2017

v2 of 3 Superseded Latest is v3 →

lore
Share & tools
lore

Ismael Luceno

author
#0 lore raw

PATCH v2 1/1 axel: bump to version 2.15

- Change upstream URL.
- Add optional dependency on libressl/openssl.
- Switch to autotools-package.

Signed-off-by Ismael Luceno <[email protected]>
---
 package/axel/Config.in |  2 +-
 package/axel/axel.hash |  3 +--
 package/axel/axel.mk   | 41 +++++++++++++++++------------------------
 3 files changed, 19 insertions(+), 27 deletions(-)
package/axel/Config.in +1 -1
index 6e709cfb1351..4507855ff8e7 100644
@@ -4,7 +4,7 @@ config BR2_PACKAGE_AXEL
 	help
 	  HTTP/FTP download accelerator.
 
-	  http://axel.alioth.debian.org/
+	  https://github.com/axel-download-accelerator/axel/
 
 comment "axel needs a toolchain w/ threads"
 	depends on !BR2_TOOLCHAIN_HAS_THREADS
package/axel/axel.hash +1 -2
index 895767011892..a49de66b9497 100644
@@ -1,2 +1 @@
-# Locally calculated
-sha256	359a57ab4e354bcb6075430d977c59d33eb3e2f1415a811948fa8ae657ca8036  axel-2.4.tar.gz
+sha256 70daa27abca5a3f1abe4be833d8f64205ac422416f5bcf396856eb6dd0fe9194  axel-2.15.tar.xz
package/axel/axel.mk +17 -24

50 lines not on this page · raw message

Signature · 1 line
2.14.2
#1 lore raw

PATCH v2 1/1 axel: bump to version 2.15

Hello,

Quoted 6 lines · Ismael Luceno

- Change upstream URL.
- Add optional dependency on libressl/openssl.
- Switch to autotools-package.

Signed-off-by Ismael Luceno <[email protected]>

---

You forgot to add a changelog here, describing what changes since v1.

Quoted 16 lines
package/axel/axel.mk +3 -9
-AXEL_VERSION = 2.4
-AXEL_SITE = http://sources.buildroot.net
+AXEL_VERSION = 2.15
+AXEL_SITE = https://github.com/axel-download-accelerator/axel/releases/download/v$(AXEL_VERSION)
+AXEL_SOURCE = axel-$(AXEL_VERSION).tar.xz
 AXEL_LICENSE = GPL-2.0+
 AXEL_LICENSE_FILES = COPYING
 AXEL_DEPENDENCIES = $(TARGET_NLS_DEPENDENCIES)
-AXEL_LDFLAGS = -lpthread $(TARGET_NLS_LIBS)
 
-ifeq ($(BR2_SYSTEM_ENABLE_NLS),)
-AXEL_DISABLE_I18N = --i18n=0
-endif
-
-define AXEL_CONFIGURE_CMDS
-	(cd $(@D); \

package/axel/axel.mk line 17

Quoted 10 lines
package/axel/axel.mk +4 -6
-		./configure \
-			--prefix=/usr \
-			--debug=1 \
-			$(AXEL_DISABLE_I18N) \
-	)
-endef
+AXEL_CFLAGS = -D_POSIX_C_SOURCE=200112L
+AXEL_CONF_OPTS = \
+	ac_cv_prog_cc_c99=-std=c99 \
+	CFLAGS="$(TARGET_CFLAGS) $(AXEL_CFLAGS)"

Why do you need these? It should be explained in the commit log, or perhaps even better with a short comment.

Thanks!

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
author
#2 lore raw

PATCH v2 1/1 axel: bump to version 2.15

On 16/Oct/2017 21:23, Thomas Petazzoni wrote:
<...>

Quoted 7 lines

package/axel/axel.mk line 17

Quoted 4 lines
package/axel/axel.mk +4 -0
+AXEL_CFLAGS = -D_POSIX_C_SOURCE=200112L
+AXEL_CONF_OPTS = \
+	ac_cv_prog_cc_c99=-std=c99 \
+	CFLAGS="$(TARGET_CFLAGS) $(AXEL_CFLAGS)"

Why do you need these? It should be explained in the commit log, or perhaps even better with a short comment.

ac_cv_prog_cc_c99 is needed because autoconf's C99 check requires wchar_t.

The define is to build against uClibc, it will be fixed upstream with the next release.

I'll send another version of the patch.