*IMPORTANT* packaged builds not working with `patch.crates-io`

[solomoncyj]

https://copr.fedorainfracloud.org/coprs/build/8262222

as we are required to use system packages, we cannot use the crate provided by [patch.crates-io]

(for some reson open suse goes through, thodue to their vendoring policy https://build.opensuse.org/package/show/utilities/linutil)

[koibtw]

cargo publish fails too

[koibtw]

It's impossible to include a patched dep in a package.

rust-lang/cargo#13222

[koibtw]

The one way to fix this is to somehow convince the owner of tui-term to use vt100-ctt instead of vt100 or just for tui-term and maintain it with vt100-ctt.

https://crates.io/crates/vt100-ctt
https://github.com/a-kenji/tui-term

[solomoncyj]

why do you need to use vt100-ctt?

[solomoncyj]

(for some reson open suse goes through, thodue to their vendoring policy https://build.opensuse.org/package/show/utilities/linutil)

[jeevithakannan2]

why do you need to use vt100-ctt?

The original vt100 had a issue where it would panic if scroll is used when running a command. vt100-ctt patches that bug.

[a-kenji]

Hello! Maintainer of https://github.com/a-kenji/tui-term here.

I have often thought about maintaining a fork of vt100, and I guess the need for many people is there.
Seeing there are multiple forks on crates.io currently.

The one way to fix this is to somehow convince the owner of tui-term to use vt100-ctt instead of vt100 or just for tui-term and maintain it with vt100-ctt

While I am not opposed in general to having in-tree support of a vt100 fork, there is overhead associated with that.
I am not comfortable at the current time to add the extra maintenance cost - I want to check out further development of the fork first. And I just now became aware of the vt100-ctt fork.

Since I am well aware of the current situation though - the vt100 backend is already pluggable and a fork of tui-term is as far as I can tell not needed.

You would likely want to disable the default vt100 feature.
You can implement the following out of tree: https://github.com/a-kenji/tui-term/blob/db1fc6762fc99706e522338ec073c3fef222a241/src/vt100_imp.rs, so in essence you need to impl Screen and Cell from tui-term.
Here you can see how vercel is doing that with their fork: https://github.com/vercel/turborepo/blob/ac1eab9105534ac9c43fb6c403b651097375a3c3/crates/turborepo-vt100/src/tui_term.rs.

[jeevithakannan2]

@a-kenji I created a PR in tui-term regarding the vt100-ctt. The fork just applies the patch to the scroll panic bug. Nothing more.