<table><tr><td style="">dfaure added inline comments.
</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/D28169">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D28169#inline-159953">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ahmadsamir</span> wrote in <span style="color: #4b4d51; font-weight: bold;">desktopexecparser.cpp:350</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Indeed.</p>

<p style="padding: 0; margin: 8px;">However I still think the condition is flawed:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">vlc.desktop says it supports smb:// so "supported" is true, !supported is false, we go the branch of ||, userName is not empty but password is empty, go to KIOFuse</li>
<li class="remarkup-list-item">but what if both the userName and password _aren't_ empty, vlc is already out of the picture</li>
</ul>

<p style="padding: 0; margin: 8px;">IIUC, vlc has native smb support which will beat any FUSE equivalent, right?</p>

<p style="padding: 0; margin: 8px;">I'll think about it some more.... :/</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The point is that KIO has no way to pass the password to VLC directly (passing it on the command line would be very insecure, other users could see it).<br />
So it's not like "VLC is out of the picture", it's rather "let's use FUSE (which will get the password) and pass the path to VLC" instead of "let's call VLC directly, but it will fail for lack of password".</p>

<p style="padding: 0; margin: 8px;">At least that's the intent of this code, which seems somewhat logical to me, and isn't the heart of the matter in this commit which is supposed to fix the CI regression :-)</p>

<p style="padding: 0; margin: 8px;">Shouldn't we just skip tests if ktelnetservice wasn't found?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28169">https://phabricator.kde.org/D28169</a></div></div><br /><div><strong>To: </strong>ahmadsamir, Frameworks, dfaure, sitter, feverfew, ngraham<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>