community/texlive: add icu-data-full dependency to xetex v1 APPLIED

Cormac Stephenson: 1
 community/texlive: add icu-data-full dependency to xetex

 1 files changed, 1 insertions(+), 1 deletions(-)

Cormac Stephenson <c7s@kasku.net> replied via email:

```
Sorry about that, sent the email by accident.

Anyway, I wanted to try and remove icu-data-full to see if luatex had
the same thing going on, but apk didn't let me, including via xetex,
so I assume this is fine as is and I just ended up with my system in a
weird configuration based on when I did updates or something.

```

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/34659#note_237480

Cormac Stephenson <c7s@kasku.net> replied via email:

```
The gist is that I think it might be unnecessary, and I was only
having problems because I upgraded during a rebuild, or something.

I tried to uninstall icu-data-full to test with luatex, but I
couldn't. Including "not removed due to: [...] xetex", which is what
should happen. This is without this change applied.

At worst it's redundant, but I probably made a mistake and jumped to
conclusions, because I fixed the problem by installing icu-data-full
originally.

```

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/34659#note_237511

[PATCH] community/texlive: add icu-data-full dependency to xetex Export this patch

---
 community/texlive/APKBUILD | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/community/texlive/APKBUILD b/community/texlive/APKBUILD
index 0d74ee53e0..5a332f8567 100644
--- a/community/texlive/APKBUILD
+++ b/community/texlive/APKBUILD
@@ -231,7 +231,7 @@ libsynctex() {
 xetex() {
 	# Split off XeTeX because it depends on graphite/harfbuzz/icu
 	pkgdesc="TeX typesetting engine supporting modern typography and bidirectional text"
-	depends="$pkgname"
+	depends="$pkgname icu-data-full"
 
 	install -d -m 0755 "$subpkgdir"/usr/bin
 	mv "$pkgdir"/usr/bin/xetex "$subpkgdir"/usr/bin/
-- 
2.36.1

Kevin Daudt <no-reply@dispatch.alpinelinux.org>

1 year, 11 months ago

The commit lacks context, but c7s mentioned on IRC that they got the following error:

```
/usr/share/texmf-dist/tex/latex/base/size10.clointernal error; cannot read font names`
```

Which would be solved by installing `icu-data-full`.

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/34659#note_237472

Marian Buschsieweke <no-reply@dispatch.alpinelinux.org>

1 year, 11 months ago

I don't quite get the gist of the last mal. I have to admit that I personally don't use xetex and I am likely to miss any bugs related to it.

Since thr split of icu was done after the last texlive updates, I would have missed any issued anyway. Maybe I should do a careful round of testing later today.

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/34659#note_237487