<table><tr><td style="">broulik updated this revision to Diff 37098.<br />broulik retitled this revision from "Kill solidautoeject" to "Build solidautoeject only on FreeBSD".<br />broulik edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-o25fal3i3wl4cay/">(Show Details)</a><br />broulik edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-p6yklp5tuevqeol/">(Show Details)</a><br />broulik 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/D13752">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Keep it around for FreeBSD</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">Solid's UDisks backend never emits the `Solid::OpticalDrive::ejectPressed` signal.<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
</span>It is only present in HAL <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">which is obsolete</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">whereas Linux uses UDev these days</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">FreeBSD supposedly still uses HAL,</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Thanks to @adridg for FreeBSD testing.</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">though?<br />
I don't have a CD drive.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Turned out this is still needed there.<br />
<br />
The "proper" approach would have been to check for `HAVE_UDEV` or something like that but HAL on Linux has been obsolete for ages</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D13752?vs=36747&id=37098">https://phabricator.kde.org/D13752?vs=36747&id=37098</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13752">https://phabricator.kde.org/D13752</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, Frameworks, adridg, davidedmundson, dfaure, fvogt, ervin<br /><strong>Cc: </strong>anthonyfieroni, mart, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>