<table><tr><td style="">bshah updated this revision to Diff 26050.<br />bshah retitled this revision from "Disable keyboard geometry preview code" to "Don't unconditionally enable the keyboard geometry parser".<br />bshah edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-lpsdv2slaeklab6/" rel="noreferrer">(Show Details)</a><br />bshah removed subscribers: sitter, ngraham, hein, broulik, graesslin, anthonyfieroni, davidedmundson.<br />bshah added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10118" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>take another approch, fix code to not force-enable the keyboard geometry parser</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">it is not clear if this code is functional at this moment or not</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Don't turn on keyboard geometry parser if Boost is found</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">given<br />
I can't find anything related to preview on my Arch Linux default Arch</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">developer may</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Linux, but nevertheless, this code is</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">have</span> disabled <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">due to another reason</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">building this code.</span><br />
<br />
This code makes C++ compiler OOM even with 16GB of RAM, In my opinion<div style="padding: 8px 0;">...</div>If someone wants to <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">re-enabl</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">mak</span>e this <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">code</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">on by default</span>, it needs to be optimized so<br />
that it doesn't bring down full-fledged CI server nodes..<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
Note for release: Keyboard geometry parser is not enabled by default,<br />
distribution may pass -DNEW_GEOMETRY option to re-enable this.<br />
<br />
BUG: 362946</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D10118?vs=26004&id=26050" rel="noreferrer">https://phabricator.kde.org/D10118?vs=26004&id=26050</a></div></div><br /><div><strong>BRANCH</strong><div><div>bshah/disable-geometry</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10118" rel="noreferrer">https://phabricator.kde.org/D10118</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/keyboard/CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>bshah, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>