D10755: Add a MsmApp::skillsPath method so the skills folder is not hardcoded

2018-02-23 Thread Phabricator
This revision was automatically updated to reflect the committed changes.
Closed by commit R846:ab71b7d7b05b: Add a MsmApp::skillsPath method so the 
skills folder is not hardcoded (authored by Antonio Larrosa 
antonio.larr...@gmail.com).

REPOSITORY
  R846 Mycroft Plasma integration

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D10755?vs=27818=27833

REVISION DETAIL
  https://phabricator.kde.org/D10755

AFFECTED FILES
  plasmoid/contents/ui/MsmView.qml
  plugin/msmapp.cpp
  plugin/msmapp.h

To: antlarr, Aiix
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart


D10755: Add a MsmApp::skillsPath method so the skills folder is not hardcoded

2018-02-22 Thread Aditya Mehra
Aiix accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R846 Mycroft Plasma integration

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D10755

To: antlarr, Aiix
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart


D10755: Add a MsmApp::skillsPath method so the skills folder is not hardcoded

2018-02-22 Thread Antonio Larrosa Jimenez
antlarr created this revision.
antlarr added a reviewer: Aiix.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
antlarr requested review of this revision.

REVISION SUMMARY
  The skills folder was hardcoded to /opt/mycroft/skills in MsmView.qml.
  When using the openSUSE packages I'm preparing, that's changed
  to ~/.mycroft/skills, so now we use that one if it exists and the
  system folder in other case.

REPOSITORY
  R846 Mycroft Plasma integration

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D10755

AFFECTED FILES
  plasmoid/contents/ui/MsmView.qml
  plugin/msmapp.cpp
  plugin/msmapp.h

To: antlarr, Aiix
Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart