<table><tr><td style="">dfaure created this revision.
</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/D7243" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This allows to remove the dependency on the installed file, fixing<br />
"make test" in kpackage before installing it.</p>

<p>I extended the unittest to have a list of deps, but it's really in a<br />
.desktop file that it would matter for checking that the servicetype<br />
is correctly found; didn't find a unittest for that though<br />
(both a desktop file and dependencies).</p>

<p>[We could also just get rid of the servicetype desktop file being passed<br />
to KPluginMetadata, AFAICS, given that its properties are just strings<br />
and dependencies are looked up explicitly as a stringlist - but I guess<br />
we want to keep it in case there are more properties in the future]</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>make uninstall ; ctest .</p></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7243" rel="noreferrer">https://phabricator.kde.org/D7243</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/data/testpackagesdep/metadata.json<br />
autotests/mockdepresolver/main.cpp<br />
src/kpackage/CMakeLists.txt<br />
src/kpackage/kpackage.qrc<br />
src/kpackage/package.cpp<br />
src/kpackage/private/packagejobthread.cpp</div></div></div><br /><div><strong>To: </strong>dfaure, mart<br /><strong>Cc: </strong>Frameworks<br /></div>