Re: [OpenWrt-Devel] Enable POSIX Threads and Disable Debug in wget

2013-07-20 Thread alpha sparc
IIRC the wget in busybox does not support SSL, but certain Dynamic DNS
services requires it.
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] Enable POSIX Threads and Disable Debug in wget

2013-07-20 Thread alpha sparc
For the multithread part I was thinking since new Routers are going
multicore and openwrt is already using pthread
and the resultant binary is just slightly bigger so must as well add
it. Not that important though.
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] Enable POSIX Threads and Disable Debug in wget

2013-07-19 Thread Catalin Patulea
Indeed, --disable-debug alone shaves 5% off ar71xx ipkg size:
176,652 bytes before
167,026 bytes with --disable-debug

But I'm not sure about --enable-threads=posix. By default my
config.log shows Thread model: posix. Looking at lib/glthread/lock.c
it's not immediately obvious to me which code is active under what
conditions.

My ipkg seems to *grow* when adding threads=posix:
167,055 --disable-debug --enable-threads=posix

but this must mean my test is inconclusive. What led you to pass
--enable-threads=posix?

On Thu, Jul 18, 2013 at 4:13 PM, alpha sparc alphasp...@gmail.com wrote:
 This patch enables POSIX threading in wget and disables debug hence
 making a smaller binary.
 Tested and working.


 Index : feeds/packages/net/wget/Makefile
 ===
 --- feeds/packages/net/wget/Makefile(revision 37381)
 +++ feeds/packages/net/wget/Makefile(working copy)
 @@ -1,5 +1,5 @@
  #
 -# Copyright (C) 2007-2011 OpenWrt.org
 +# Copyright (C) 2007-2013 OpenWrt.org
  #
  # This is free software, licensed under the GNU General Public License v2.
  # See /LICENSE for more information.
 @@ -63,7 +63,9 @@ endef

  CONFIGURE_ARGS+= \
 --disable-rpath \
 -   --disable-iri
 +   --disable-iri \
 +   --disable-debug \
 +   --enable-threads=posix  \

  CONFIGURE_VARS += \
 ac_cv_header_uuid_uuid_h=no
 ___
 openwrt-devel mailing list
 openwrt-devel@lists.openwrt.org
 https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


Re: [OpenWrt-Devel] Enable POSIX Threads and Disable Debug in wget

2013-07-19 Thread Catalin Patulea
ar71xx binary size profile:

472,738 bytes baseline
452,623 (-4.2%) --disable-debug
452,623 --disable-debug --enable-threads=posix

BTW, any reason not to use busybox wget (BUSYBOX_CONFIG_WGET)?

On Fri, Jul 19, 2013 at 7:44 PM, Catalin Patulea c...@vv.carleton.ca wrote:
 Indeed, --disable-debug alone shaves 5% off ar71xx ipkg size:
 176,652 bytes before
 167,026 bytes with --disable-debug

 But I'm not sure about --enable-threads=posix. By default my
 config.log shows Thread model: posix. Looking at lib/glthread/lock.c
 it's not immediately obvious to me which code is active under what
 conditions.

 My ipkg seems to *grow* when adding threads=posix:
 167,055 --disable-debug --enable-threads=posix

 but this must mean my test is inconclusive. What led you to pass
 --enable-threads=posix?

 On Thu, Jul 18, 2013 at 4:13 PM, alpha sparc alphasp...@gmail.com wrote:
 This patch enables POSIX threading in wget and disables debug hence
 making a smaller binary.
 Tested and working.


 Index : feeds/packages/net/wget/Makefile
 ===
 --- feeds/packages/net/wget/Makefile(revision 37381)
 +++ feeds/packages/net/wget/Makefile(working copy)
 @@ -1,5 +1,5 @@
  #
 -# Copyright (C) 2007-2011 OpenWrt.org
 +# Copyright (C) 2007-2013 OpenWrt.org
  #
  # This is free software, licensed under the GNU General Public License v2.
  # See /LICENSE for more information.
 @@ -63,7 +63,9 @@ endef

  CONFIGURE_ARGS+= \
 --disable-rpath \
 -   --disable-iri
 +   --disable-iri \
 +   --disable-debug \
 +   --enable-threads=posix  \

  CONFIGURE_VARS += \
 ac_cv_header_uuid_uuid_h=no
 ___
 openwrt-devel mailing list
 openwrt-devel@lists.openwrt.org
 https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


[OpenWrt-Devel] Enable POSIX Threads and Disable Debug in wget

2013-07-18 Thread alpha sparc
This patch enables POSIX threading in wget and disables debug hence
making a smaller binary.
Tested and working.


Index : feeds/packages/net/wget/Makefile
===
--- feeds/packages/net/wget/Makefile(revision 37381)
+++ feeds/packages/net/wget/Makefile(working copy)
@@ -1,5 +1,5 @@
 #
-# Copyright (C) 2007-2011 OpenWrt.org
+# Copyright (C) 2007-2013 OpenWrt.org
 #
 # This is free software, licensed under the GNU General Public License v2.
 # See /LICENSE for more information.
@@ -63,7 +63,9 @@ endef

 CONFIGURE_ARGS+= \
--disable-rpath \
-   --disable-iri
+   --disable-iri \
+   --disable-debug \
+   --enable-threads=posix  \

 CONFIGURE_VARS += \
ac_cv_header_uuid_uuid_h=no
___
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel