Skip to contentSkip to conversation

package/intltool: remove target variant

PATCH New Romain Naour 2 participants 2 messages Dec 31, 2016 – Jan 1, 2017

lore
Share & tools
lore

Romain Naour

author
#0 lore raw

PATCH 1/1 package/intltool: remove target variant

The target variant depends on BR2_HOST_ONLY which is just like BROKEN (i.e not defined anywere). BR2_HOST_ONLY was introduced by [1] back in 2010 and nobody seems to need it. So remove intltool for the target.

[1] 0b876d39776fdec69762b988216e5ed64dbe6ba8

Signed-off-by Romain Naour <[email protected]>
Cc Thomas Petazzoni <[email protected]>
---
 package/Config.in            | 1 -
 package/intltool/Config.in   | 9 ---------
 package/intltool/intltool.mk | 1 -
 3 files changed, 11 deletions(-)
 delete mode 100644 package/intltool/Config.in
package/Config.in +0 -1
index da9715c..8d7cddf 100644
@@ -138,7 +138,6 @@ menu "Development tools"
 	source "package/git/Config.in"
 	source "package/gperf/Config.in"
 	source "package/grep/Config.in"
-	source "package/intltool/Config.in"
 	source "package/jq/Config.in"
 	source "package/libtool/Config.in"
 	source "package/make/Config.in"
package/intltool/Config.in deleted +0 -9
deleted file mode 100644
index 4643a89..0000000
@@ -1,9 +0,0 @@
-config BR2_PACKAGE_INTLTOOL
-	bool "intltool"
-	# Hide from configuration as we only support the host package
-	# for the moment
-	depends on BR2_HOST_ONLY
-	help
-	  Utility scripts for internationalizing XML
-
-	  http://www.freedesktop.org/wiki/Software/intltool
package/intltool/intltool.mk +0 -1
index fb801a1..dbef80f 100644
@@ -11,5 +11,4 @@ INTLTOOL_LICENSE_FILES = COPYING
 
 HOST_INTLTOOL_DEPENDENCIES = host-gettext host-libxml-parser-perl
 
-$(eval $(autotools-package))
 $(eval $(host-autotools-package))
Signature · 1 line
2.9.3
Applied
#1 lore raw

PATCH 1/1 package/intltool: remove target variant

Hello,

Quoted 6 lines · Romain Naour

The target variant depends on BR2_HOST_ONLY which is just like BROKEN (i.e not defined anywere). BR2_HOST_ONLY was introduced by [1] back in 2010 and nobody seems to need it. So remove intltool for the target.

[1] 0b876d39776fdec69762b988216e5ed64dbe6ba8

Quoted 8 lines
Signed-off-by Romain Naour <[email protected]>
Cc Thomas Petazzoni <[email protected]>
---
 package/Config.in            | 1 -
 package/intltool/Config.in   | 9 ---------
 package/intltool/intltool.mk | 1 -
 3 files changed, 11 deletions(-)
 delete mode 100644 package/intltool/Config.in

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com