Bug 894392

Summary:	net-im/telegram-desktop-4.6.2: missing spell useflag
Product:	Gentoo Linux	Reporter:	kamal <gentoo.kamal>
Component:	Current packages	Assignee:	Esteve Varela Colominas <esteve.varela>
Status:	RESOLVED INVALID
Severity:	normal	CC:	proxy-maint
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---

Description kamal 2023-02-14 13:27:54 UTC

Versions <=4.6.2 allowed disabling spellchecking altogether with -spell.
In 4.6.2 spell checking is forced due to "^^ ( enchant hunspell )".
However, the DESKTOP_APP_DISABLE_SPELLCHECK cmake argument is still present upstream.
Is this intended for stability reasons or can the useflag added again?


Reproducible: Always

Steps to Reproduce:
Emerge net-im/telegram-desktop-4.6.2 with USE="-spell -hunspell -enchant"
Actual Results:  
Emerge fails, unmet useflag requirements.

Expected Results:  
A Telegram installation without spell checking support.

Comment 1 Esteve Varela Colominas 2023-02-14 21:29:18 UTC

I knew someone was gonna report this as a bug, lol. Thanks for doing so, I'm glad to know people care.

The argument is currently broken upstream, as they've been phasing out support for the toggle, as it was only used for old macOS versions. See the comment at:

https://github.com/telegramdesktop/tdesktop/commit/f5be551ff8bb16f92638d4ddb5097a4e64507851#r99549281

Apparently, the existence of a toggle doesn't necessarily mean it works with this package. Personally, I've been having trouble disabling spellchecking in the client itself (now the feature flag is mandatory), so if anyone knows how to make the red squiggles disappear forever, I'd love to know.