[v2,07/11] Drop intltool add-on

Message ID c5111e76-9d5c-8017-a165-357cc3bc372b@ipfire.org
State Rejected
Headers
Series Drop unmaintained or orphaned add-ons and packages, first batch |

Commit Message

Peter Müller May 17, 2021, 9:57 p.m. UTC
  As discussed in https://wiki.ipfire.org/devel/telco/2021-05-03.

Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
---
 config/rootfiles/packages/intltool | 13 -----
 lfs/intltool                       | 85 ------------------------------
 make.sh                            |  1 -
 3 files changed, 99 deletions(-)
 delete mode 100644 config/rootfiles/packages/intltool
 delete mode 100644 lfs/intltool
  

Comments

Michael Tremer May 20, 2021, 9:36 a.m. UTC | #1
Hello,

We need this as a build dependency of probably a hundred packages :)

I will keep this and move it into the core system.

-Michael

> On 17 May 2021, at 22:57, Peter Müller <peter.mueller@ipfire.org> wrote:
> 
> As discussed in https://wiki.ipfire.org/devel/telco/2021-05-03.
> 
> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
> ---
> config/rootfiles/packages/intltool | 13 -----
> lfs/intltool                       | 85 ------------------------------
> make.sh                            |  1 -
> 3 files changed, 99 deletions(-)
> delete mode 100644 config/rootfiles/packages/intltool
> delete mode 100644 lfs/intltool
> 
> diff --git a/config/rootfiles/packages/intltool b/config/rootfiles/packages/intltool
> deleted file mode 100644
> index e1421fad7..000000000
> --- a/config/rootfiles/packages/intltool
> +++ /dev/null
> @@ -1,13 +0,0 @@
> -usr/bin/intltool-extract
> -usr/bin/intltool-merge
> -usr/bin/intltool-prepare
> -usr/bin/intltool-update
> -usr/bin/intltoolize
> -#usr/share/aclocal/intltool.m4
> -#usr/share/intltool
> -#usr/share/intltool/Makefile.in.in
> -#usr/share/man/man8/intltool-extract.8
> -#usr/share/man/man8/intltool-merge.8
> -#usr/share/man/man8/intltool-prepare.8
> -#usr/share/man/man8/intltool-update.8
> -#usr/share/man/man8/intltoolize.8
> diff --git a/lfs/intltool b/lfs/intltool
> deleted file mode 100644
> index 234262f62..000000000
> --- a/lfs/intltool
> +++ /dev/null
> @@ -1,85 +0,0 @@
> -###############################################################################
> -#                                                                             #
> -# IPFire.org - A linux based firewall                                         #
> -# Copyright (C) 2007-2018  IPFire Team  <info@ipfire.org>                     #
> -#                                                                             #
> -# This program is free software: you can redistribute it and/or modify        #
> -# it under the terms of the GNU General Public License as published by        #
> -# the Free Software Foundation, either version 3 of the License, or           #
> -# (at your option) any later version.                                         #
> -#                                                                             #
> -# This program is distributed in the hope that it will be useful,             #
> -# but WITHOUT ANY WARRANTY; without even the implied warranty of              #
> -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the               #
> -# GNU General Public License for more details.                                #
> -#                                                                             #
> -# You should have received a copy of the GNU General Public License           #
> -# along with this program.  If not, see <http://www.gnu.org/licenses/>.       #
> -#                                                                             #
> -###############################################################################
> -
> -###############################################################################
> -# Definitions
> -###############################################################################
> -
> -include Config
> -
> -VER        = 0.40.5
> -
> -THISAPP    = intltool-$(VER)
> -DL_FILE    = $(THISAPP).tar.bz2
> -DL_FROM    = $(URL_IPFIRE)
> -DIR_APP    = $(DIR_SRC)/$(THISAPP)
> -TARGET     = $(DIR_INFO)/$(THISAPP)
> -PROG       = intltool
> -PAK_VER    = 1
> -
> -DEPS       =
> -
> -###############################################################################
> -# Top-level Rules
> -###############################################################################
> -
> -objects = $(DL_FILE)
> -
> -$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
> -
> -$(DL_FILE)_MD5 = 3e7ded6f50d88cf8b8bb505eaf591775
> -
> -install : $(TARGET)
> -
> -check : $(patsubst %,$(DIR_CHK)/%,$(objects))
> -
> -download :$(patsubst %,$(DIR_DL)/%,$(objects))
> -
> -md5 : $(subst %,%_MD5,$(objects))
> -
> -
> -###############################################################################
> -# Downloading, checking, md5sum
> -###############################################################################
> -
> -$(patsubst %,$(DIR_CHK)/%,$(objects)) :
> -	@$(CHECK)
> -
> -$(patsubst %,$(DIR_DL)/%,$(objects)) :
> -	@$(LOAD)
> -
> -$(subst %,%_MD5,$(objects)) :
> -	@$(MD5)
> -
> -dist: 
> -	@$(PAK)
> -
> -###############################################################################
> -# Installation Details
> -###############################################################################
> -
> -$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
> -	@$(PREBUILD)
> -	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar jxf $(DIR_DL)/$(DL_FILE)
> -	cd $(DIR_APP) && ./configure --prefix=/usr
> -	cd $(DIR_APP) && make $(MAKETUNING) $(EXTRA_MAKE)
> -	cd $(DIR_APP) && make install
> -	@rm -rf $(DIR_APP)
> -	@$(POSTBUILD)
> diff --git a/make.sh b/make.sh
> index 3bca46d8f..56c93fb79 100755
> --- a/make.sh
> +++ b/make.sh
> @@ -1385,7 +1385,6 @@ buildipfire() {
>   lfsmake2 openvpn
>   lfsmake2 mpage
>   lfsmake2 dbus
> -  lfsmake2 intltool
>   lfsmake2 libdaemon
>   lfsmake2 avahi
>   lfsmake2 cups
> -- 
> 2.26.2
  
Peter Müller May 20, 2021, 2:13 p.m. UTC | #2
Hello Michael,

thanks for your reply.

I am a bit puzzled by it: Yes, I would have expected us to need intltool for building packages
- but only in the toolchain. If I understood you correctly, we used to rely on an add-on for
building core packages? That would be - um - interesting. :-)

If so, do we have more candidates such as intltool?

Thanks, and best regards,
Peter Müller


> Hello,
> 
> We need this as a build dependency of probably a hundred packages :)
> 
> I will keep this and move it into the core system.
> 
> -Michael
> 
>> On 17 May 2021, at 22:57, Peter Müller <peter.mueller@ipfire.org> wrote:
>>
>> As discussed in https://wiki.ipfire.org/devel/telco/2021-05-03.
>>
>> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
>> ---
>> config/rootfiles/packages/intltool | 13 -----
>> lfs/intltool                       | 85 ------------------------------
>> make.sh                            |  1 -
>> 3 files changed, 99 deletions(-)
>> delete mode 100644 config/rootfiles/packages/intltool
>> delete mode 100644 lfs/intltool
>>
>> diff --git a/config/rootfiles/packages/intltool b/config/rootfiles/packages/intltool
>> deleted file mode 100644
>> index e1421fad7..000000000
>> --- a/config/rootfiles/packages/intltool
>> +++ /dev/null
>> @@ -1,13 +0,0 @@
>> -usr/bin/intltool-extract
>> -usr/bin/intltool-merge
>> -usr/bin/intltool-prepare
>> -usr/bin/intltool-update
>> -usr/bin/intltoolize
>> -#usr/share/aclocal/intltool.m4
>> -#usr/share/intltool
>> -#usr/share/intltool/Makefile.in.in
>> -#usr/share/man/man8/intltool-extract.8
>> -#usr/share/man/man8/intltool-merge.8
>> -#usr/share/man/man8/intltool-prepare.8
>> -#usr/share/man/man8/intltool-update.8
>> -#usr/share/man/man8/intltoolize.8
>> diff --git a/lfs/intltool b/lfs/intltool
>> deleted file mode 100644
>> index 234262f62..000000000
>> --- a/lfs/intltool
>> +++ /dev/null
>> @@ -1,85 +0,0 @@
>> -###############################################################################
>> -#                                                                             #
>> -# IPFire.org - A linux based firewall                                         #
>> -# Copyright (C) 2007-2018  IPFire Team  <info@ipfire.org>                     #
>> -#                                                                             #
>> -# This program is free software: you can redistribute it and/or modify        #
>> -# it under the terms of the GNU General Public License as published by        #
>> -# the Free Software Foundation, either version 3 of the License, or           #
>> -# (at your option) any later version.                                         #
>> -#                                                                             #
>> -# This program is distributed in the hope that it will be useful,             #
>> -# but WITHOUT ANY WARRANTY; without even the implied warranty of              #
>> -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the               #
>> -# GNU General Public License for more details.                                #
>> -#                                                                             #
>> -# You should have received a copy of the GNU General Public License           #
>> -# along with this program.  If not, see <http://www.gnu.org/licenses/>.       #
>> -#                                                                             #
>> -###############################################################################
>> -
>> -###############################################################################
>> -# Definitions
>> -###############################################################################
>> -
>> -include Config
>> -
>> -VER        = 0.40.5
>> -
>> -THISAPP    = intltool-$(VER)
>> -DL_FILE    = $(THISAPP).tar.bz2
>> -DL_FROM    = $(URL_IPFIRE)
>> -DIR_APP    = $(DIR_SRC)/$(THISAPP)
>> -TARGET     = $(DIR_INFO)/$(THISAPP)
>> -PROG       = intltool
>> -PAK_VER    = 1
>> -
>> -DEPS       =
>> -
>> -###############################################################################
>> -# Top-level Rules
>> -###############################################################################
>> -
>> -objects = $(DL_FILE)
>> -
>> -$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
>> -
>> -$(DL_FILE)_MD5 = 3e7ded6f50d88cf8b8bb505eaf591775
>> -
>> -install : $(TARGET)
>> -
>> -check : $(patsubst %,$(DIR_CHK)/%,$(objects))
>> -
>> -download :$(patsubst %,$(DIR_DL)/%,$(objects))
>> -
>> -md5 : $(subst %,%_MD5,$(objects))
>> -
>> -
>> -###############################################################################
>> -# Downloading, checking, md5sum
>> -###############################################################################
>> -
>> -$(patsubst %,$(DIR_CHK)/%,$(objects)) :
>> -	@$(CHECK)
>> -
>> -$(patsubst %,$(DIR_DL)/%,$(objects)) :
>> -	@$(LOAD)
>> -
>> -$(subst %,%_MD5,$(objects)) :
>> -	@$(MD5)
>> -
>> -dist: 
>> -	@$(PAK)
>> -
>> -###############################################################################
>> -# Installation Details
>> -###############################################################################
>> -
>> -$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
>> -	@$(PREBUILD)
>> -	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar jxf $(DIR_DL)/$(DL_FILE)
>> -	cd $(DIR_APP) && ./configure --prefix=/usr
>> -	cd $(DIR_APP) && make $(MAKETUNING) $(EXTRA_MAKE)
>> -	cd $(DIR_APP) && make install
>> -	@rm -rf $(DIR_APP)
>> -	@$(POSTBUILD)
>> diff --git a/make.sh b/make.sh
>> index 3bca46d8f..56c93fb79 100755
>> --- a/make.sh
>> +++ b/make.sh
>> @@ -1385,7 +1385,6 @@ buildipfire() {
>>   lfsmake2 openvpn
>>   lfsmake2 mpage
>>   lfsmake2 dbus
>> -  lfsmake2 intltool
>>   lfsmake2 libdaemon
>>   lfsmake2 avahi
>>   lfsmake2 cups
>> -- 
>> 2.26.2
>
  
Michael Tremer May 20, 2021, 2:14 p.m. UTC | #3
Hello,

This package was obviously available in the build environment.

There was just no need to package it and ship it as an add-on.

-Michael

> On 20 May 2021, at 15:13, Peter Müller <peter.mueller@ipfire.org> wrote:
> 
> Hello Michael,
> 
> thanks for your reply.
> 
> I am a bit puzzled by it: Yes, I would have expected us to need intltool for building packages
> - but only in the toolchain. If I understood you correctly, we used to rely on an add-on for
> building core packages? That would be - um - interesting. :-)
> 
> If so, do we have more candidates such as intltool?

I don’t know. Do you want to have a look? :)

> 
> Thanks, and best regards,
> Peter Müller
> 
> 
>> Hello,
>> 
>> We need this as a build dependency of probably a hundred packages :)
>> 
>> I will keep this and move it into the core system.
>> 
>> -Michael
>> 
>>> On 17 May 2021, at 22:57, Peter Müller <peter.mueller@ipfire.org> wrote:
>>> 
>>> As discussed in https://wiki.ipfire.org/devel/telco/2021-05-03.
>>> 
>>> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
>>> ---
>>> config/rootfiles/packages/intltool | 13 -----
>>> lfs/intltool                       | 85 ------------------------------
>>> make.sh                            |  1 -
>>> 3 files changed, 99 deletions(-)
>>> delete mode 100644 config/rootfiles/packages/intltool
>>> delete mode 100644 lfs/intltool
>>> 
>>> diff --git a/config/rootfiles/packages/intltool b/config/rootfiles/packages/intltool
>>> deleted file mode 100644
>>> index e1421fad7..000000000
>>> --- a/config/rootfiles/packages/intltool
>>> +++ /dev/null
>>> @@ -1,13 +0,0 @@
>>> -usr/bin/intltool-extract
>>> -usr/bin/intltool-merge
>>> -usr/bin/intltool-prepare
>>> -usr/bin/intltool-update
>>> -usr/bin/intltoolize
>>> -#usr/share/aclocal/intltool.m4
>>> -#usr/share/intltool
>>> -#usr/share/intltool/Makefile.in.in
>>> -#usr/share/man/man8/intltool-extract.8
>>> -#usr/share/man/man8/intltool-merge.8
>>> -#usr/share/man/man8/intltool-prepare.8
>>> -#usr/share/man/man8/intltool-update.8
>>> -#usr/share/man/man8/intltoolize.8
>>> diff --git a/lfs/intltool b/lfs/intltool
>>> deleted file mode 100644
>>> index 234262f62..000000000
>>> --- a/lfs/intltool
>>> +++ /dev/null
>>> @@ -1,85 +0,0 @@
>>> -###############################################################################
>>> -#                                                                             #
>>> -# IPFire.org - A linux based firewall                                         #
>>> -# Copyright (C) 2007-2018  IPFire Team  <info@ipfire.org>                     #
>>> -#                                                                             #
>>> -# This program is free software: you can redistribute it and/or modify        #
>>> -# it under the terms of the GNU General Public License as published by        #
>>> -# the Free Software Foundation, either version 3 of the License, or           #
>>> -# (at your option) any later version.                                         #
>>> -#                                                                             #
>>> -# This program is distributed in the hope that it will be useful,             #
>>> -# but WITHOUT ANY WARRANTY; without even the implied warranty of              #
>>> -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the               #
>>> -# GNU General Public License for more details.                                #
>>> -#                                                                             #
>>> -# You should have received a copy of the GNU General Public License           #
>>> -# along with this program.  If not, see <http://www.gnu.org/licenses/>.       #
>>> -#                                                                             #
>>> -###############################################################################
>>> -
>>> -###############################################################################
>>> -# Definitions
>>> -###############################################################################
>>> -
>>> -include Config
>>> -
>>> -VER        = 0.40.5
>>> -
>>> -THISAPP    = intltool-$(VER)
>>> -DL_FILE    = $(THISAPP).tar.bz2
>>> -DL_FROM    = $(URL_IPFIRE)
>>> -DIR_APP    = $(DIR_SRC)/$(THISAPP)
>>> -TARGET     = $(DIR_INFO)/$(THISAPP)
>>> -PROG       = intltool
>>> -PAK_VER    = 1
>>> -
>>> -DEPS       =
>>> -
>>> -###############################################################################
>>> -# Top-level Rules
>>> -###############################################################################
>>> -
>>> -objects = $(DL_FILE)
>>> -
>>> -$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
>>> -
>>> -$(DL_FILE)_MD5 = 3e7ded6f50d88cf8b8bb505eaf591775
>>> -
>>> -install : $(TARGET)
>>> -
>>> -check : $(patsubst %,$(DIR_CHK)/%,$(objects))
>>> -
>>> -download :$(patsubst %,$(DIR_DL)/%,$(objects))
>>> -
>>> -md5 : $(subst %,%_MD5,$(objects))
>>> -
>>> -
>>> -###############################################################################
>>> -# Downloading, checking, md5sum
>>> -###############################################################################
>>> -
>>> -$(patsubst %,$(DIR_CHK)/%,$(objects)) :
>>> -	@$(CHECK)
>>> -
>>> -$(patsubst %,$(DIR_DL)/%,$(objects)) :
>>> -	@$(LOAD)
>>> -
>>> -$(subst %,%_MD5,$(objects)) :
>>> -	@$(MD5)
>>> -
>>> -dist: 
>>> -	@$(PAK)
>>> -
>>> -###############################################################################
>>> -# Installation Details
>>> -###############################################################################
>>> -
>>> -$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
>>> -	@$(PREBUILD)
>>> -	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar jxf $(DIR_DL)/$(DL_FILE)
>>> -	cd $(DIR_APP) && ./configure --prefix=/usr
>>> -	cd $(DIR_APP) && make $(MAKETUNING) $(EXTRA_MAKE)
>>> -	cd $(DIR_APP) && make install
>>> -	@rm -rf $(DIR_APP)
>>> -	@$(POSTBUILD)
>>> diff --git a/make.sh b/make.sh
>>> index 3bca46d8f..56c93fb79 100755
>>> --- a/make.sh
>>> +++ b/make.sh
>>> @@ -1385,7 +1385,6 @@ buildipfire() {
>>>  lfsmake2 openvpn
>>>  lfsmake2 mpage
>>>  lfsmake2 dbus
>>> -  lfsmake2 intltool
>>>  lfsmake2 libdaemon
>>>  lfsmake2 avahi
>>>  lfsmake2 cups
>>> -- 
>>> 2.26.2
>>
  

Patch

diff --git a/config/rootfiles/packages/intltool b/config/rootfiles/packages/intltool
deleted file mode 100644
index e1421fad7..000000000
--- a/config/rootfiles/packages/intltool
+++ /dev/null
@@ -1,13 +0,0 @@ 
-usr/bin/intltool-extract
-usr/bin/intltool-merge
-usr/bin/intltool-prepare
-usr/bin/intltool-update
-usr/bin/intltoolize
-#usr/share/aclocal/intltool.m4
-#usr/share/intltool
-#usr/share/intltool/Makefile.in.in
-#usr/share/man/man8/intltool-extract.8
-#usr/share/man/man8/intltool-merge.8
-#usr/share/man/man8/intltool-prepare.8
-#usr/share/man/man8/intltool-update.8
-#usr/share/man/man8/intltoolize.8
diff --git a/lfs/intltool b/lfs/intltool
deleted file mode 100644
index 234262f62..000000000
--- a/lfs/intltool
+++ /dev/null
@@ -1,85 +0,0 @@ 
-###############################################################################
-#                                                                             #
-# IPFire.org - A linux based firewall                                         #
-# Copyright (C) 2007-2018  IPFire Team  <info@ipfire.org>                     #
-#                                                                             #
-# This program is free software: you can redistribute it and/or modify        #
-# it under the terms of the GNU General Public License as published by        #
-# the Free Software Foundation, either version 3 of the License, or           #
-# (at your option) any later version.                                         #
-#                                                                             #
-# This program is distributed in the hope that it will be useful,             #
-# but WITHOUT ANY WARRANTY; without even the implied warranty of              #
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the               #
-# GNU General Public License for more details.                                #
-#                                                                             #
-# You should have received a copy of the GNU General Public License           #
-# along with this program.  If not, see <http://www.gnu.org/licenses/>.       #
-#                                                                             #
-###############################################################################
-
-###############################################################################
-# Definitions
-###############################################################################
-
-include Config
-
-VER        = 0.40.5
-
-THISAPP    = intltool-$(VER)
-DL_FILE    = $(THISAPP).tar.bz2
-DL_FROM    = $(URL_IPFIRE)
-DIR_APP    = $(DIR_SRC)/$(THISAPP)
-TARGET     = $(DIR_INFO)/$(THISAPP)
-PROG       = intltool
-PAK_VER    = 1
-
-DEPS       =
-
-###############################################################################
-# Top-level Rules
-###############################################################################
-
-objects = $(DL_FILE)
-
-$(DL_FILE) = $(DL_FROM)/$(DL_FILE)
-
-$(DL_FILE)_MD5 = 3e7ded6f50d88cf8b8bb505eaf591775
-
-install : $(TARGET)
-
-check : $(patsubst %,$(DIR_CHK)/%,$(objects))
-
-download :$(patsubst %,$(DIR_DL)/%,$(objects))
-
-md5 : $(subst %,%_MD5,$(objects))
-
-
-###############################################################################
-# Downloading, checking, md5sum
-###############################################################################
-
-$(patsubst %,$(DIR_CHK)/%,$(objects)) :
-	@$(CHECK)
-
-$(patsubst %,$(DIR_DL)/%,$(objects)) :
-	@$(LOAD)
-
-$(subst %,%_MD5,$(objects)) :
-	@$(MD5)
-
-dist: 
-	@$(PAK)
-
-###############################################################################
-# Installation Details
-###############################################################################
-
-$(TARGET) : $(patsubst %,$(DIR_DL)/%,$(objects))
-	@$(PREBUILD)
-	@rm -rf $(DIR_APP) && cd $(DIR_SRC) && tar jxf $(DIR_DL)/$(DL_FILE)
-	cd $(DIR_APP) && ./configure --prefix=/usr
-	cd $(DIR_APP) && make $(MAKETUNING) $(EXTRA_MAKE)
-	cd $(DIR_APP) && make install
-	@rm -rf $(DIR_APP)
-	@$(POSTBUILD)
diff --git a/make.sh b/make.sh
index 3bca46d8f..56c93fb79 100755
--- a/make.sh
+++ b/make.sh
@@ -1385,7 +1385,6 @@  buildipfire() {
   lfsmake2 openvpn
   lfsmake2 mpage
   lfsmake2 dbus
-  lfsmake2 intltool
   lfsmake2 libdaemon
   lfsmake2 avahi
   lfsmake2 cups