Bug#962464: Remove ICU dependency

2022-10-22 Thread Bastian Germann

Hi,

On Thu, 21 Oct 2021 11:04:04 +0200 Bastian Germann  wrote:

Control: tags -1 patch

On Wed, 17 Mar 2021 17:30:59 +0100 Bastian Germann wrote:>
> I second this request. Alternatively, it would be great to have a build 
> profile that excludes ICU.


A patch with noi18n profile is enclosed.


I would like to see this resolved with bookworm.
As you are probably uploading CVE fixes soon I want to remind you of this 
request.
It would very much help with creating tiny root file systems that have libxml2 
in them.

My request to join the team has not been answered, so I have to rely on you to 
make the change.

Thanks,
Bastian



Bug#962464: Remove ICU dependency

2021-10-21 Thread Bastian Germann

Control: tags -1 patch

On Wed, 17 Mar 2021 17:30:59 +0100 Bastian Germann wrote:>
I second this request. Alternatively, it would be great to have a build 
profile that excludes ICU.


A patch with noi18n profile is enclosed.
From: Bastian Germann 
Date: Thu, 21 Oct 2021 10:54:53 +0200
Subject: No icu dependency with noi18n build profile

---
 debian/control | 2 +-
 debian/rules   | 4 
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/debian/control b/debian/control
index 48b34e6..cdab6cb 100644
--- a/debian/control
+++ b/debian/control
@@ -11,7 +11,7 @@ Build-Depends-Indep:
  pkg-config,
 Build-Depends-Arch:
  dh-sequence-python3 ,
- libicu-dev,
+ libicu-dev ,
  liblzma-dev,
  libpython3-all-dbg ,
  libpython3-all-dev ,
diff --git a/debian/rules b/debian/rules
index 8436da5..aec8313 100755
--- a/debian/rules
+++ b/debian/rules
@@ -28,7 +28,11 @@ doconfigure-%:
 	dh_auto_configure --builddirectory=builddir/$(*) -- $(CONFIGURE_FLAGS)
 	touch $@
 
+ifeq (,$(filter noi18n,$(DEB_BUILD_PROFILES)))
 doconfigure-main: CONFIGURE_FLAGS += --without-python --with-icu
+else
+doconfigure-main: CONFIGURE_FLAGS += --without-python
+endif
 doconfigure-python%: CONFIGURE_FLAGS += --with-python=/usr/bin/$(*) --with-python-install-dir=/usr/lib/$(*)/dist-packages
 
 dobuild-%: BUILD_DIR=builddir/$(*)


Bug#962464: Remove ICU dependency

2021-03-17 Thread Bastian Germann
On Mon, 8 Jun 2020 14:54:42 +0200 Nick Wellnhofer  
wrote:

Package: libxml2

Bug #776254 requested that libxml2 uses ICU for character set conversion 
because of some unspecified issues with Chromium. This adds an otherwise 
useless, multi-megabyte dependency (larger than libxml2 itself). I propose to 
revert that change. This means that libxml2 will use iconv for character 
conversion as before.


I'm sure that the Chromium issues can be worked around in another way. I'm 
also disappointed that the Chromium developers never reached out to the 
libxml2 maintainers.


I second this request. Alternatively, it would be great to have a build 
profile that excludes ICU.




Bug#962464: Remove ICU dependency

2020-06-08 Thread Nick Wellnhofer

Package: libxml2

Bug #776254 requested that libxml2 uses ICU for character set conversion 
because of some unspecified issues with Chromium. This adds an otherwise 
useless, multi-megabyte dependency (larger than libxml2 itself). I propose to 
revert that change. This means that libxml2 will use iconv for character 
conversion as before.


I'm sure that the Chromium issues can be worked around in another way. I'm 
also disappointed that the Chromium developers never reached out to the 
libxml2 maintainers.