Opened 13 months ago
Closed 13 months ago
#19474 closed enhancement (fixed)
vte-0.76.0
| Reported by: | Bruce Dubbs | Owned by: | Tim Tassonis |
|---|---|---|---|
| Priority: | normal | Milestone: | 12.2 |
| Component: | BOOK | Version: | git |
| Severity: | normal | Keywords: | |
| Cc: |
Description ¶
New minor version.
Change History (9)
comment:1 by , 13 months ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:2 by , 13 months ago
The Homepage (https://github.com/lz4/lz4) describes it as: "Extremely Fast Compression algorithm". Well, who can argue with that...
comment:3 by , 13 months ago
To be fair vte has been depending on zlib for years, and zlib just happens to be in LFS. A compression algorithm is needed by terminal emulation to compress the scrolling buffer data unless you want to spend some money to increase the RAM amount.
The switching from zlib to lz4 is based on a profiling showing zlib compression has became a major performance bottleneck after other bottlenecks (mainly "no GPU rendering") were resolved.
I've raised a discussion about this via IRC months ago and bdubbs said it might be better to add lz4 into LFS instead of BLFS, because zstd already optionally depends on lz4.
comment:4 by , 13 months ago
I don't really care if it is in LFS or BLFS, but has anybody already got the build/install instructions?
I'm also fine with waiting a bit until those questions are resolved. I'm tending to BLFS, as I'd like to have only stuff in LFS that is really needed for LFS. But of course this is a blurred line, what is needed for LFS, and what isn't.
comment:5 by , 13 months ago
Shall I just put an external link to lz4 in the vte page until this is resolved, or better wait for the decision?
follow-up: 8 comment:7 by , 13 months ago
I saw that lz4 is now in lfs devel, so I'll finish the ticket. In that case, lz4 has not to be mentioned, afaik.
comment:8 by , 13 months ago
Replying to Tim Tassonis:
I saw that lz4 is now in lfs devel, so I'll finish the ticket. In that case, lz4 has not to be mentioned, afaik.
That is correct, lz4 doesn't need to be mentioned now that it's in LFS. I'm doing some work in BLFS for other packages that use it before I start a full system rebuild for the new GNOME and to help with Plasma.
comment:9 by , 13 months ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fixed in commit 48aa301c9a
The idiots at Gnome/IBM/Microsoft or whoever is now in charge at Red Hat really leave no stone unturned: vte now unconditionally requires liblz4, I'm sure that's totally needed for terminal emulation. Shall we add this to the book? I guess we have to.