Differential D3953

Improve error message
ClosedPublic
Actions

Authored by apol on Jan 4 2017, 11:58 AM.

Details

Reviewers
mart
Group Reviewers
Plasma
Commits
R290:f9857670704f: Improve error message
Summary

DesktopToJson will convert lists such as "Something=a, b, c" into
JSON "Something": ["a", "b ", "c "]. Making sure there's quotes will lessen
this problem to some extent.

Test Plan

Now the error message is:

Error: Installation of /home/apol/Downloads/kdeokktester/tester failed: Could not install dependency: ' kns://plasma-themes.knsrc/api.kde-look.org/1136471'

Diff Detail

Repository
R290 KPackage
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
apol updated this revision to Diff 9696.Jan 4 2017, 11:58 AM
apol retitled this revision from to Improve error message.
apol updated this object.
apol edited the test plan for this revision. (Show Details)
apol added reviewers: Plasma, mart.
Restricted Application added projects: Plasma, Frameworks. · View Herald TranscriptJan 4 2017, 11:58 AM
Restricted Application added subscribers: Frameworks, plasma-devel. · View Herald Transcript
mart accepted this revision.Jan 4 2017, 12:32 PM
mart edited edge metadata.
This revision is now accepted and ready to land.Jan 4 2017, 12:32 PM

Revision Contents
Changeset List

Diff 9696

View Options

src/kpackage/private/packagejobthread.cpp

Loading...