<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/D15277">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/D15277#inline-84613">View Inline</a><span style="color: #4b4d51; font-weight: bold;">elvisangelaccio</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kmtpd.json:10</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Currently the module is spawned by the ioslave on-demand. But imho it should be possible to browse the device using only dbus, without having to open dolphin first.</p>
<p style="padding: 0; margin: 8px;">Can we try to autostart the module?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I don't understand what you mean here. How is a user supposed to "use only dbus"? ;-) Is this really about increasing overhead for every Plasma user (even those who never use mtp) for the benefit of an hypothetical command-line user?<br />
I'm pretty sure qdbus can trigger on-demand loading too, even, since that's done by the dbus server, when using a dbus service file.</p>
<p style="padding: 0; margin: 8px;">In case this is what you meant: please don't load the module at plasma startup. On demand is perfect. And it can be on demand of other apps than kioslaves, in case any app wants to do some MTP stuff (e.g. Amarok).</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R320 KIO Extras</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15277">https://phabricator.kde.org/D15277</a></div></div><br /><div><strong>To: </strong>akrutzler, elvisangelaccio, ltoscano, hetzenecker, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, kfm-devel, feverfew, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp<br /></div>