* [Buildroot] [git commit] dropbear: bump to version 2011.54
@ 2011-12-01 8:58 Peter Korsgaard
2011-12-01 13:22 ` Michael S. Zick
0 siblings, 1 reply; 8+ messages in thread
From: Peter Korsgaard @ 2011-12-01 8:58 UTC (permalink / raw)
To: buildroot
commit: http://git.buildroot.net/buildroot/commit/?id=8b56bfe8385826c6417dbc8dc731ad535b0de3b5
branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
.../dropbear-0.53.1-static_build_fix.patch | 26 --------------------
package/dropbear/dropbear-2011.54-no-ipv6.patch | 18 +++++++++++++
package/dropbear/dropbear.mk | 3 +-
3 files changed, 19 insertions(+), 28 deletions(-)
diff --git a/package/dropbear/dropbear-0.53.1-static_build_fix.patch b/package/dropbear/dropbear-0.53.1-static_build_fix.patch
deleted file mode 100644
index 0e0a91a..0000000
--- a/package/dropbear/dropbear-0.53.1-static_build_fix.patch
+++ /dev/null
@@ -1,26 +0,0 @@
-dropbear: fix static build
-
-the -lcrypt is missing during the link
-
-svr-authpasswd.o: In function `svr_auth_password':
-svr-authpasswd.c:(.text+0xfc): undefined reference to `crypt'
-collect2: ld returned 1 exit status
-
-Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
----
- Makefile.in | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
-
-Index: b/Makefile.in
-===================================================================
---- a/Makefile.in
-+++ b/Makefile.in
-@@ -77,7 +77,7 @@ STRIP=@STRIP@
- INSTALL=@INSTALL@
- CPPFLAGS=@CPPFLAGS@
- CFLAGS+=-I. -I$(srcdir) $(CPPFLAGS) @CFLAGS@
--LIBS+=@LIBS@
-+LIBS+=@CRYPTLIB@ @LIBS@
- LDFLAGS=@LDFLAGS@
-
- EXEEXT=@EXEEXT@
diff --git a/package/dropbear/dropbear-2011.54-no-ipv6.patch b/package/dropbear/dropbear-2011.54-no-ipv6.patch
new file mode 100644
index 0000000..4ee9aaa
--- /dev/null
+++ b/package/dropbear/dropbear-2011.54-no-ipv6.patch
@@ -0,0 +1,18 @@
+Check for IPV6_TCLASS instead of IPPROTO_IPV6 since
+it's present on non-IPv6 enabled toolchains too.
+
+Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
+---
+
+diff -Nura dropbear-2011.54.orig/dbutil.c dropbear-2011.54/dbutil.c
+--- dropbear-2011.54.orig/dbutil.c 2011-11-08 09:48:15.000000000 -0300
++++ dropbear-2011.54/dbutil.c 2011-11-09 12:14:59.430074138 -0300
+@@ -164,7 +164,7 @@
+ /* set the TOS bit for either ipv4 or ipv6 */
+ #ifdef IPTOS_LOWDELAY
+ val = IPTOS_LOWDELAY;
+-#ifdef IPPROTO_IPV6
++#ifdef IPV6_TCLASS
+ setsockopt(sock, IPPROTO_IPV6, IPV6_TCLASS, (void*)&val, sizeof(val));
+ #endif
+ setsockopt(sock, IPPROTO_IP, IP_TOS, (void*)&val, sizeof(val));
diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk
index bced0ef..5fa50ba 100644
--- a/package/dropbear/dropbear.mk
+++ b/package/dropbear/dropbear.mk
@@ -4,8 +4,7 @@
#
#############################################################
-DROPBEAR_VERSION = 0.53.1
-DROPBEAR_SOURCE = dropbear-$(DROPBEAR_VERSION).tar.gz
+DROPBEAR_VERSION = 2011.54
DROPBEAR_SITE = http://matt.ucc.asn.au/dropbear/releases
DROPBEAR_TARGET_BINS = dbclient dropbearkey dropbearconvert scp ssh
DROPBEAR_MAKE = $(MAKE) MULTI=1 SCPPROGRESS=1 \
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Buildroot] [git commit] dropbear: bump to version 2011.54
2011-12-01 8:58 [Buildroot] [git commit] dropbear: bump to version 2011.54 Peter Korsgaard
@ 2011-12-01 13:22 ` Michael S. Zick
2011-12-01 13:52 ` Thomas Petazzoni
0 siblings, 1 reply; 8+ messages in thread
From: Michael S. Zick @ 2011-12-01 13:22 UTC (permalink / raw)
To: buildroot
On Fri November 11 2011, Peter Korsgaard wrote:
> commit: http://git.buildroot.net/buildroot/commit/?id=8b56bfe8385826c6417dbc8dc731ad535b0de3b5
> branch: http://git.buildroot.net/buildroot/commit/?id=refs/heads/master
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
> ---
> .../dropbear-0.53.1-static_build_fix.patch | 26 --------------------
> package/dropbear/dropbear-2011.54-no-ipv6.patch | 18 +++++++++++++
> package/dropbear/dropbear.mk | 3 +-
> 3 files changed, 19 insertions(+), 28 deletions(-)
>
> diff --git a/package/dropbear/dropbear-0.53.1-static_build_fix.patch b/package/dropbear/dropbear-0.53.1-static_build_fix.patch
> deleted file mode 100644
> index 0e0a91a..0000000
> --- a/package/dropbear/dropbear-0.53.1-static_build_fix.patch
> +++ /dev/null
> @@ -1,26 +0,0 @@
> -dropbear: fix static build
> -
> -the -lcrypt is missing during the link
> -
> -svr-authpasswd.o: In function `svr_auth_password':
> -svr-authpasswd.c:(.text+0xfc): undefined reference to `crypt'
> -collect2: ld returned 1 exit status
> -
> -Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> ----
> - Makefile.in | 2 +-
> - 1 file changed, 1 insertion(+), 1 deletion(-)
> -
> -Index: b/Makefile.in
> -===================================================================
> ---- a/Makefile.in
> -+++ b/Makefile.in
> -@@ -77,7 +77,7 @@ STRIP=@STRIP@
> - INSTALL=@INSTALL@
> - CPPFLAGS=@CPPFLAGS@
> - CFLAGS+=-I. -I$(srcdir) $(CPPFLAGS) @CFLAGS@
> --LIBS+=@LIBS@
> -+LIBS+=@CRYPTLIB@ @LIBS@
> - LDFLAGS=@LDFLAGS@
> -
> - EXEEXT=@EXEEXT@
> diff --git a/package/dropbear/dropbear-2011.54-no-ipv6.patch b/package/dropbear/dropbear-2011.54-no-ipv6.patch
> new file mode 100644
> index 0000000..4ee9aaa
> --- /dev/null
> +++ b/package/dropbear/dropbear-2011.54-no-ipv6.patch
> @@ -0,0 +1,18 @@
> +Check for IPV6_TCLASS instead of IPPROTO_IPV6 since
> +it's present on non-IPv6 enabled toolchains too.
> +
> +Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> +---
> +
> +diff -Nura dropbear-2011.54.orig/dbutil.c dropbear-2011.54/dbutil.c
> +--- dropbear-2011.54.orig/dbutil.c 2011-11-08 09:48:15.000000000 -0300
> ++++ dropbear-2011.54/dbutil.c 2011-11-09 12:14:59.430074138 -0300
> +@@ -164,7 +164,7 @@
> + /* set the TOS bit for either ipv4 or ipv6 */
> + #ifdef IPTOS_LOWDELAY
> + val = IPTOS_LOWDELAY;
> +-#ifdef IPPROTO_IPV6
> ++#ifdef IPV6_TCLASS
> + setsockopt(sock, IPPROTO_IPV6, IPV6_TCLASS, (void*)&val, sizeof(val));
> + #endif
> + setsockopt(sock, IPPROTO_IP, IP_TOS, (void*)&val, sizeof(val));
> diff --git a/package/dropbear/dropbear.mk b/package/dropbear/dropbear.mk
> index bced0ef..5fa50ba 100644
> --- a/package/dropbear/dropbear.mk
> +++ b/package/dropbear/dropbear.mk
> @@ -4,8 +4,7 @@
> #
> #############################################################
>
> -DROPBEAR_VERSION = 0.53.1
> -DROPBEAR_SOURCE = dropbear-$(DROPBEAR_VERSION).tar.gz
>
Has the "DROPBEAR_SOURCE" line been accidentally dropped here?
At "DROPBEAR_SITE" the package is named: dropbear-2011.54.tar.gz
(or .tar.bz2)
Mike
> +DROPBEAR_VERSION = 2011.54
> DROPBEAR_SITE = http://matt.ucc.asn.au/dropbear/releases
> DROPBEAR_TARGET_BINS = dbclient dropbearkey dropbearconvert scp ssh
> DROPBEAR_MAKE = $(MAKE) MULTI=1 SCPPROGRESS=1 \
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [git commit] dropbear: bump to version 2011.54
2011-12-01 13:22 ` Michael S. Zick
@ 2011-12-01 13:52 ` Thomas Petazzoni
2011-12-01 14:00 ` Michael S. Zick
2011-12-01 14:35 ` Thomas De Schampheleire
0 siblings, 2 replies; 8+ messages in thread
From: Thomas Petazzoni @ 2011-12-01 13:52 UTC (permalink / raw)
To: buildroot
Le Thu, 1 Dec 2011 07:22:33 -0600,
"Michael S. Zick" <minimod@morethan.org> a ?crit :
> Has the "DROPBEAR_SOURCE" line been accidentally dropped here?
> At "DROPBEAR_SITE" the package is named: dropbear-2011.54.tar.gz
> (or .tar.bz2)
This <pkg>_SOURCE value is the default one:
$(2)_BASE_NAME = $(1)-$$($(2)_VERSION)
$(2)_SOURCE ?= $$($(2)_BASE_NAME).tar.gz
(from package/Makefile.package.in)
Regards,
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [git commit] dropbear: bump to version 2011.54
2011-12-01 13:52 ` Thomas Petazzoni
@ 2011-12-01 14:00 ` Michael S. Zick
2011-12-01 14:08 ` Thomas Petazzoni
2011-12-01 14:35 ` Thomas De Schampheleire
1 sibling, 1 reply; 8+ messages in thread
From: Michael S. Zick @ 2011-12-01 14:00 UTC (permalink / raw)
To: buildroot
On Thu December 1 2011, Thomas Petazzoni wrote:
> Le Thu, 1 Dec 2011 07:22:33 -0600,
> "Michael S. Zick" <minimod@morethan.org> a ?crit :
>
> > Has the "DROPBEAR_SOURCE" line been accidentally dropped here?
> > At "DROPBEAR_SITE" the package is named: dropbear-2011.54.tar.gz
> > (or .tar.bz2)
>
> This <pkg>_SOURCE value is the default one:
>
> $(2)_BASE_NAME = $(1)-$$($(2)_VERSION)
> $(2)_SOURCE ?= $$($(2)_BASE_NAME).tar.gz
>
Ah, so -
The infrastructure builds the correct name and
the deleted line was not required.
Sorry, was looking only at the patch out of context.
Mike
> (from package/Makefile.package.in)
>
> Regards,
>
> Thomas
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [git commit] dropbear: bump to version 2011.54
2011-12-01 14:00 ` Michael S. Zick
@ 2011-12-01 14:08 ` Thomas Petazzoni
0 siblings, 0 replies; 8+ messages in thread
From: Thomas Petazzoni @ 2011-12-01 14:08 UTC (permalink / raw)
To: buildroot
Le Thu, 1 Dec 2011 08:00:11 -0600,
"Michael S. Zick" <minimod@morethan.org> a ?crit :
> Ah, so -
> The infrastructure builds the correct name and
> the deleted line was not required.
Yep.
> Sorry, was looking only at the patch out of context.
No problem. It's always good to have people reviewing the changes being
merged!
Regards,
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [git commit] dropbear: bump to version 2011.54
2011-12-01 13:52 ` Thomas Petazzoni
2011-12-01 14:00 ` Michael S. Zick
@ 2011-12-01 14:35 ` Thomas De Schampheleire
2011-12-01 14:46 ` Michael S. Zick
2011-12-01 15:49 ` Peter Korsgaard
1 sibling, 2 replies; 8+ messages in thread
From: Thomas De Schampheleire @ 2011-12-01 14:35 UTC (permalink / raw)
To: buildroot
On Thu, Dec 1, 2011 at 2:52 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Le Thu, 1 Dec 2011 07:22:33 -0600,
> "Michael S. Zick" <minimod@morethan.org> a ?crit :
>
>> Has the "DROPBEAR_SOURCE" line been accidentally dropped here?
>> At "DROPBEAR_SITE" the package is named: dropbear-2011.54.tar.gz
>> (or .tar.bz2)
>
> This <pkg>_SOURCE value is the default one:
>
> $(2)_BASE_NAME ?= ?$(1)-$$($(2)_VERSION)
> $(2)_SOURCE ? ? ?= $$($(2)_BASE_NAME).tar.gz
>
> (from package/Makefile.package.in)
Just throwing the question out: should we adapt existing .mk files to
remove lines that correspond to the default? This is such an example,
and FOO_INSTALL_TARGET=YES is another one.
Best regards,
Thomas
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [git commit] dropbear: bump to version 2011.54
2011-12-01 14:35 ` Thomas De Schampheleire
@ 2011-12-01 14:46 ` Michael S. Zick
2011-12-01 15:49 ` Peter Korsgaard
1 sibling, 0 replies; 8+ messages in thread
From: Michael S. Zick @ 2011-12-01 14:46 UTC (permalink / raw)
To: buildroot
On Thu December 1 2011, Thomas De Schampheleire wrote:
> On Thu, Dec 1, 2011 at 2:52 PM, Thomas Petazzoni
> <thomas.petazzoni@free-electrons.com> wrote:
> > Le Thu, 1 Dec 2011 07:22:33 -0600,
> > "Michael S. Zick" <minimod@morethan.org> a ?crit :
> >
> >> Has the "DROPBEAR_SOURCE" line been accidentally dropped here?
> >> At "DROPBEAR_SITE" the package is named: dropbear-2011.54.tar.gz
> >> (or .tar.bz2)
> >
> > This <pkg>_SOURCE value is the default one:
> >
> > $(2)_BASE_NAME ?= ?$(1)-$$($(2)_VERSION)
> > $(2)_SOURCE ? ? ?= $$($(2)_BASE_NAME).tar.gz
> >
> > (from package/Makefile.package.in)
>
> Just throwing the question out: should we adapt existing .mk files to
> remove lines that correspond to the default? This is such an example,
> and FOO_INSTALL_TARGET=YES is another one.
>
I would say yes, since people who are adding packages
might pick one of the ones with useless assignments as
their model and then useless assignments would never go
away (or Peter would have to tweak the new patches).
Mike
> Best regards,
> Thomas
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [git commit] dropbear: bump to version 2011.54
2011-12-01 14:35 ` Thomas De Schampheleire
2011-12-01 14:46 ` Michael S. Zick
@ 2011-12-01 15:49 ` Peter Korsgaard
1 sibling, 0 replies; 8+ messages in thread
From: Peter Korsgaard @ 2011-12-01 15:49 UTC (permalink / raw)
To: buildroot
>>>>> "Thomas" == Thomas De Schampheleire <patrickdepinguin+buildroot@gmail.com> writes:
Hi,
Thomas> Just throwing the question out: should we adapt existing .mk files to
Thomas> remove lines that correspond to the default? This is such an example,
Thomas> and FOO_INSTALL_TARGET=YES is another one.
For _INSTALL_TARGET definately, yes.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2011-12-01 15:49 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-01 8:58 [Buildroot] [git commit] dropbear: bump to version 2011.54 Peter Korsgaard
2011-12-01 13:22 ` Michael S. Zick
2011-12-01 13:52 ` Thomas Petazzoni
2011-12-01 14:00 ` Michael S. Zick
2011-12-01 14:08 ` Thomas Petazzoni
2011-12-01 14:35 ` Thomas De Schampheleire
2011-12-01 14:46 ` Michael S. Zick
2011-12-01 15:49 ` Peter Korsgaard
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox