<table><tr><td style="">thomasp 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/D7922" rel="noreferrer">View Revision</a></tr></table><br /><div><div>

<blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D7922#147893" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7922#147893</a>, <a href="https://phabricator.kde.org/p/nicolasfella/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@nicolasfella</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I would be ok with keeping the KMix codepath, but I think its quite a corner-case since most people use pulseaudio anyway. If we keep it we should make sure people are aware that they need pactl OR kmix for that feature. I am not aware that Kmix is a dependency on any distro (I wasn't even aware that we have this feature until yesterday :D)</p></div>
</blockquote>

<p>I am totally fine with kmix support being removed. Please disable the option in the UI when it will not work if at all possible. It will prevent grieve for the dying breed of pulseaudio deniers.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Any comments on the cmake part? I don't know much about cmake.</p></blockquote>

<p>See <tt style="background: #ebebeb; font-size: 13px;">man cmake-commands</tt> for what <tt style="background: #ebebeb; font-size: 13px;">find_program</tt> does. Also take a look at <tt style="background: #ebebeb; font-size: 13px;">option</tt> and <tt style="background: #ebebeb; font-size: 13px;">configure_file</tt>. The <tt style="background: #ebebeb; font-size: 13px;">configure_file</tt> example shows how to transport values from cmake to c++.<br />
This of course only detects if pactl was found at compile time. A runtime check is also an option.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7922" rel="noreferrer">https://phabricator.kde.org/D7922</a></div></div><br /><div><strong>To: </strong>nicolasfella, KDE Connect<br /><strong>Cc: </strong>thomasp, albertvaka, KDE Connect, daniel.z.tg, jeanv, tfella, seebauer, bugzy, progwolff, MayeulC, menasshock, ach, apol<br /></div>