Differential D3860

appstreamtest: handle non-installed programs
ClosedPublic
Actions

Authored by ervin on Dec 29 2016, 3:29 PM.

Details

Reviewers
apol
Group Reviewers
Frameworks
Commits
R240:5ed7ba07e4f3: appstreamtest: handle non-installed programs
Summary

If you just built the software without installing it and then run ctest,
this will systematically fail while trying to read install_manifest.txt.
With this patch this case is now handled gracefully, not forcing to
install to have a test suite which fully passes.

Diff Detail

Repository
R240 Extra CMake Modules
Lint
Lint Skipped
Unit
Unit Tests Skipped
ervin updated this revision to Diff 9458.Dec 29 2016, 3:29 PM
ervin retitled this revision from to appstreamtest: handle non-installed programs.
ervin updated this object.
ervin edited the test plan for this revision. (Show Details)
ervin added reviewers: Frameworks, apol.
ervin updated this revision to Diff 9459.Dec 29 2016, 3:40 PM

Of course, I notice just now I got the order of the parameters wrong.
Now properly working both with and without install_manifest.txt in the
build directory.

apol accepted this revision.Dec 29 2016, 10:10 PM
apol edited edge metadata.
This revision is now accepted and ready to land.Dec 29 2016, 10:10 PM
Closed by commit R240:5ed7ba07e4f3: appstreamtest: handle non-installed programs (authored by ervin). · Explain WhyDec 30 2016, 7:42 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkde-modules/appstreamtest.cmake (10 lines)

Diff 9459

View Options

kde-modules/appstreamtest.cmake

Loading...