Opened 6 weeks ago

Closed 5 weeks 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 Tim Tassonis, 6 weeks ago

Owner: changed from blfs-book to Tim Tassonis
Status: newassigned

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.

comment:2 by Tim Tassonis, 6 weeks ago

The Homepage (https://github.com/lz4/lz4) describes it as: "Extremely Fast Compression algorithm". Well, who can argue with that...

comment:3 by Xi Ruoyao, 6 weeks 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 Tim Tassonis, 6 weeks 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 Tim Tassonis, 6 weeks ago

Shall I just put an external link to lz4 in the vte page until this is resolved, or better wait for the decision?

comment:6 by Xi Ruoyao, 6 weeks ago

The build/test/installation instructions are in lfs:#5463.

comment:7 by Tim Tassonis, 5 weeks 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.

in reply to:  7 comment:8 by Douglas R. Reno, 5 weeks 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 Tim Tassonis, 5 weeks ago

Resolution: fixed
Status: assignedclosed

Fixed in commit 48aa301c9a

Note: See TracTickets for help on using tickets.