[Buildroot] [PATCH 0 of 5 v2] Deprecation: clarify manual, remove ttcp and lzma

[Buildroot] [PATCH 0 of 5 v2] Deprecation: clarify manual, remove ttcp and lzma

[Thomas De Schampheleire]

Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

---

 Config.in                                             |   8 ----
 Config.in.legacy                                      |  29 ++++++++++----
 docs/manual/appendix.txt                              |   8 +++-
 package/Config.in                                     |   2 -
 package/flex/Config.in                                |   2 +-
 package/lzma/Config.in                                |  13 ------
 package/lzma/lzma.mk                                  |   2 -
 package/ttcp/Config.in                                |   7 ---
 package/ttcp/ttcp-0001-replace-deprecated-bcopy.patch |  15 -------
 package/ttcp/ttcp.mk                                  |  25 ------------
 10 files changed, 29 insertions(+), 82 deletions(-)

[Buildroot] [PATCH 1 of 5 v2] manual: clarify handling of deprecated features

[Thomas De Schampheleire]

Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

---
v2: resend

 docs/manual/appendix.txt |  8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

diff --git a/docs/manual/appendix.txt b/docs/manual/appendix.txt
--- a/docs/manual/appendix.txt
+++ b/docs/manual/appendix.txt
@@ -29,6 +29,12 @@ Deprecated features
 -------------------
 
 The following features are marked as _deprecated_ in Buildroot due to
-their status either too old or unmaintained.
+them being either too old or unmaintained. They will be removed at
+some point, so stop using them.
+Each deprecated symbol in kconfig depends on a symbol
++BR2_DEPRECATED_SINCE_xxxx_xx+, which provides an indication of when
+the feature can be removed: features will not be removed within the
+year following deprecation. For example, a symbol depending on
++BR2_DEPRECATED_SINCE_2013_05+ can be removed from 2014.05 onwards.
 
 include::deprecated-list.txt[]

[Buildroot] [PATCH 2 of 5 v2] ttcp: remove deprecated package

[Thomas De Schampheleire]

Since this is the only package depending on BR2_DEPRECATED_SINCE_2012_05,
remove that symbol as well.

Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

---
v2: also remove deprecation symbol.

 Config.in                                             |   4 --
 Config.in.legacy                                      |   7 ++++
 package/Config.in                                     |   1 -
 package/ttcp/Config.in                                |   7 ----
 package/ttcp/ttcp-0001-replace-deprecated-bcopy.patch |  15 ---------
 package/ttcp/ttcp.mk                                  |  25 ---------------
 6 files changed, 7 insertions(+), 52 deletions(-)

diff --git a/Config.in b/Config.in
--- a/Config.in
+++ b/Config.in
@@ -268,10 +268,6 @@ config BR2_DEPRECATED_SINCE_2010_11
 	bool
 	default y
 
-config BR2_DEPRECATED_SINCE_2012_05
-	bool
-	default y
-
 config BR2_DEPRECATED_SINCE_2012_11
 	bool
 	default y
diff --git a/Config.in.legacy b/Config.in.legacy
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -112,6 +112,13 @@ config BR2_PACKAGE_LIBNFC_LLCP
 ###############################################################################
 comment "Legacy options removed in 2014.02"
 
+config BR2_PACKAGE_TTCP
+	bool "ttcp has been removed"
+	select BR2_LEGACY
+	help
+	  The 'ttcp' package has been removed since it has been
+	  deprecated for more than four buildroot releases.
+
 config BR2_PACKAGE_SQUASHFS3
 	bool "squashfs3 has been removed"
 	select BR2_LEGACY
diff --git a/package/Config.in b/package/Config.in
--- a/package/Config.in
+++ b/package/Config.in
@@ -931,7 +931,6 @@ source "package/thttpd/Config.in"
 source "package/tinyhttpd/Config.in"
 source "package/tn5250/Config.in"
 source "package/transmission/Config.in"
-source "package/ttcp/Config.in"
 source "package/tvheadend/Config.in"
 source "package/udpcast/Config.in"
 source "package/ulogd/Config.in"
diff --git a/package/ttcp/Config.in b/package/ttcp/Config.in
deleted file mode 100644
--- a/package/ttcp/Config.in
+++ /dev/null
@@ -1,7 +0,0 @@
-config BR2_PACKAGE_TTCP
-	bool "ttcp"
-	depends on BR2_DEPRECATED_SINCE_2012_05
-	help
-	  Benchmarking tool for determining TCP and UDP performance.
-
-	  ftp://ftp.sgi.com/sgi/src/ttcp/README
diff --git a/package/ttcp/ttcp-0001-replace-deprecated-bcopy.patch b/package/ttcp/ttcp-0001-replace-deprecated-bcopy.patch
deleted file mode 100644
--- a/package/ttcp/ttcp-0001-replace-deprecated-bcopy.patch
+++ /dev/null
@@ -1,15 +0,0 @@
-Replace deprecated bcopy call by using memmove instead.
-
-Index: b/ttcp.c
-===================================================================
---- a/ttcp.c
-+++ b/ttcp.c
-@@ -241,7 +241,7 @@
- 			if ((addr=gethostbyname(host)) == NULL)
- 				err("bad hostname");
- 			sinhim.sin_family = addr->h_addrtype;
--			bcopy(addr->h_addr,(char*)&addr_tmp, addr->h_length);
-+			memmove((char*)&addr_tmp, addr->h_addr, addr->h_length);
- #if defined(cray)
- 			sinhim.sin_addr = addr_tmp;
- #else
diff --git a/package/ttcp/ttcp.mk b/package/ttcp/ttcp.mk
deleted file mode 100644
--- a/package/ttcp/ttcp.mk
+++ /dev/null
@@ -1,25 +0,0 @@
-################################################################################
-#
-# ttcp
-#
-################################################################################
-
-TTCP_VERSION = 1.12
-TTCP_SITE    = ftp://ftp.sgi.com/sgi/src/ttcp/
-TTCP_SOURCE  = ttcp.c
-TTCP_LICENSE = public domain
-
-define TTCP_EXTRACT_CMDS
-	cp -f -t $(@D) $(DL_DIR)/$(TTCP_SOURCE)
-endef
-
-define TTCP_BUILD_CMDS
-	$(TARGET_CC) $(TARGET_CPPFLAGS) $(TARGET_CFLAGS) \
-		-o $(@D)/ttcp $(@D)/$(TTCP_SOURCE)
-endef
-
-define TTCP_INSTALL_TARGET_CMDS
-	$(INSTALL) -D -m 0755 $(@D)/ttcp $(TARGET_DIR)/usr/bin/ttcp
-endef
-
-$(eval $(generic-package))

[Buildroot] [PATCH 3 of 5 v2] flex: change config text of 'target binary' option

[Thomas De Schampheleire]

The buildroot manual contains a list of deprecated items, with their config
title text, and the location in the config menu. If the config text does not
mention a package name, this can be confusing.

For example, the symbol BR2_PACKAGE_FLEX_BINARY has as text:
'Install tool in the target', which outside of the flex context makes no
sense at all.

To make sure the deprecated packages list in the manual is understandable,
rename the (deprecated) flex 'target binary' config option.

Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

---
v2: new patch based on feedback from Arnout Vandecappelle

 package/flex/Config.in |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/package/flex/Config.in b/package/flex/Config.in
--- a/package/flex/Config.in
+++ b/package/flex/Config.in
@@ -10,7 +10,7 @@ config BR2_PACKAGE_FLEX
 if BR2_PACKAGE_FLEX
 
 config BR2_PACKAGE_FLEX_BINARY
-	bool "Install tool in the target"
+	bool "Install flex binary on target"
 	# needs fork()
 	depends on BR2_USE_MMU
 	# runtime dependency

[Buildroot] [PATCH 4 of 5 v2] lzma: remove deprecated target package

[Thomas De Schampheleire]

The on-target lzma package has been deprecated for a long time, so remove
it. This does not remove the support for generating lzma-compressed rootfs
images.

Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

---
v2: new patch

 Config.in              |   4 ----
 Config.in.legacy       |  13 ++++++++++---
 package/Config.in      |   1 -
 package/lzma/Config.in |  13 -------------
 package/lzma/lzma.mk   |   2 --
 5 files changed, 10 insertions(+), 23 deletions(-)

diff --git a/Config.in b/Config.in
--- a/Config.in
+++ b/Config.in
@@ -264,10 +264,6 @@ config BR2_DEPRECATED
 
 if BR2_DEPRECATED
 
-config BR2_DEPRECATED_SINCE_2010_11
-	bool
-	default y
-
 config BR2_DEPRECATED_SINCE_2012_11
 	bool
 	default y
diff --git a/Config.in.legacy b/Config.in.legacy
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -99,6 +99,16 @@ comment "-------------------------------
 endif
 
 ###############################################################################
+comment "Legacy options removed in 2014.02"
+
+config BR2_PACKAGE_LZMA
+	bool "lzma target package has been removed"
+	select BR2_LEGACY
+	help
+	  The 'lzma' target package has been removed since it has been
+	  deprecated for more than four buildroot releases.
+	  Note: generating lzma-compressed rootfs images is still supported.
+
 config BR2_PACKAGE_LIBNFC_LLCP
 	bool "libnfc-llcp has been removed"
 	depends on BR2_TOOLCHAIN_HAS_THREADS # libusb
@@ -109,9 +119,6 @@ config BR2_PACKAGE_LIBNFC_LLCP
           to 'libllcp'. We have added a new package for 'libllcp' and bumped
           the version at the same time.
 
-###############################################################################
-comment "Legacy options removed in 2014.02"
-
 config BR2_PACKAGE_TTCP
 	bool "ttcp has been removed"
 	select BR2_LEGACY
diff --git a/package/Config.in b/package/Config.in
--- a/package/Config.in
+++ b/package/Config.in
@@ -35,7 +35,6 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
 source "package/gzip/Config.in"
 endif
 source "package/infozip/Config.in"
-source "package/lzma/Config.in"
 source "package/lzop/Config.in"
 source "package/xz/Config.in"
 endmenu
diff --git a/package/lzma/Config.in b/package/lzma/Config.in
deleted file mode 100644
--- a/package/lzma/Config.in
+++ /dev/null
@@ -1,13 +0,0 @@
-config BR2_PACKAGE_LZMA
-	bool "lzma"
-	depends on BR2_INSTALL_LIBSTDCPP
-	depends on BR2_DEPRECATED_SINCE_2010_11
-	help
-	  Lempel Ziv compression method (LZMA) is a compression
-	  algorithm with high compression ratio.
-
-	  http://tukaani.org/lzma/
-
-comment "lzma needs a toolchain w/ C++"
-	depends on BR2_DEPRECATED_SINCE_2010_11
-	depends on !BR2_INSTALL_LIBSTDCPP
diff --git a/package/lzma/lzma.mk b/package/lzma/lzma.mk
--- a/package/lzma/lzma.mk
+++ b/package/lzma/lzma.mk
@@ -7,10 +7,8 @@
 LZMA_VERSION = 4.32.7
 LZMA_SOURCE = lzma-$(LZMA_VERSION).tar.xz
 LZMA_SITE = http://tukaani.org/lzma/
-LZMA_INSTALL_STAGING = YES
 LZMA_CONF_OPT = $(if $(BR2_ENABLE_DEBUG),--enable-debug,--disable-debug)
 
-$(eval $(autotools-package))
 $(eval $(host-autotools-package))
 
 LZMA = $(HOST_DIR)/usr/bin/lzma

[Buildroot] [PATCH 4 of 5 v2] lzma: remove deprecated target package

[Yann E. MORIN]

Thomas, All,

On 2014-01-20 22:46 +0100, Thomas De Schampheleire spake thusly:
> The on-target lzma package has been deprecated for a long time, so remove
> it. This does not remove the support for generating lzma-compressed rootfs
> images.
> 
> Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
[--SNIP--]
> diff --git a/Config.in.legacy b/Config.in.legacy
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -99,6 +99,16 @@ comment "-------------------------------
>  endif
>  
>  ###############################################################################
> +comment "Legacy options removed in 2014.02"
> +
> +config BR2_PACKAGE_LZMA
> +	bool "lzma target package has been removed"
> +	select BR2_LEGACY
> +	help
> +	  The 'lzma' target package has been removed since it has been
> +	  deprecated for more than four buildroot releases.
> +	  Note: generating lzma-compressed rootfs images is still supported.
> +
>  config BR2_PACKAGE_LIBNFC_LLCP
>  	bool "libnfc-llcp has been removed"
>  	depends on BR2_TOOLCHAIN_HAS_THREADS # libusb
> @@ -109,9 +119,6 @@ config BR2_PACKAGE_LIBNFC_LLCP
>            to 'libllcp'. We have added a new package for 'libllcp' and bumped
>            the version at the same time.
>  
> -###############################################################################
> -comment "Legacy options removed in 2014.02"
> -

Moving the comment will gonna clash with my recent patch being applied
by Peter:
    http://git.buildroot.org/buildroot/commit/?id=f169e5e1055600fda1b3ff0c852ee84d2ec80460

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  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.  |
'------------------------------^-------^------------------^--------------------'

[Buildroot] [PATCH 4 of 5 v2] lzma: remove deprecated target package

[Thomas De Schampheleire]

On Mon, Jan 20, 2014 at 10:58 PM, Yann E. MORIN <yann.morin.1998@free.fr> wrote:
[..]
>> -
>
> Moving the comment will gonna clash with my recent patch being applied
> by Peter:
>     http://git.buildroot.org/buildroot/commit/?id=f169e5e1055600fda1b3ff0c852ee84d2ec80460
>

Grumbl... ;-)

Rebased with v3, thanks.

Best regards,
Thomas

[Buildroot] [PATCH 5 of 5 v2] Config.in.legacy: fix misc. problems with libnfc-llcp removal

[Thomas De Schampheleire]

The Config.in.legacy entry associated with the replacement of libnfc-llcp
was broken in multiple ways:

- it was not correctly added within the section for 2014.02 deprecation
  (incidentally fixed with the patch removing target lzma)
- it still had select/depend statement from the original symbol, which
  should not be added in Config.in.legacy.
- it did not select the replacement symbol as to make the transition for
  users more convenient.
- indentation of the help text was not tab+2spaces.

Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

---
v2: new patch

 Config.in.legacy |  9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/Config.in.legacy b/Config.in.legacy
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -110,14 +110,13 @@ config BR2_PACKAGE_LZMA
 	  Note: generating lzma-compressed rootfs images is still supported.
 
 config BR2_PACKAGE_LIBNFC_LLCP
-	bool "libnfc-llcp has been removed"
-	depends on BR2_TOOLCHAIN_HAS_THREADS # libusb
-	select BR2_PACKAGE_LIBNFC
+	bool "libnfc-llcp has been replaced by libllcp"
 	select BR2_LEGACY
+	select BR2_PACKAGE_LIBLLCP
 	help
 	  The 'libnfc-llcp' package has been removed since upstream renamed
-          to 'libllcp'. We have added a new package for 'libllcp' and bumped
-          the version at the same time.
+	  to 'libllcp'. We have added a new package for 'libllcp' and bumped
+	  the version at the same time.
 
 config BR2_PACKAGE_TTCP
 	bool "ttcp has been removed"