Merge lp:~benste/kdeedu/bugfix-lp-698056 into lp:~kubuntu-packagers/kubuntu-packaging/kdeedu
Proposed by
benste
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 120 | ||||
Proposed branch: | lp:~benste/kdeedu/bugfix-lp-698056 | ||||
Merge into: | lp:~kubuntu-packagers/kubuntu-packaging/kdeedu | ||||
Diff against target: |
26 lines (+7/-1) 2 files modified
debian/changelog (+6/-0) debian/control (+1/-1) |
||||
To merge this branch: | bzr merge lp:~benste/kdeedu/bugfix-lp-698056 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Harald Sitter | Approve | ||
Rohan Garg | Pending | ||
Review via email: mp+45408@code.launchpad.net |
Description of the change
new attempt, fixing the simple dependency issue, and using the kubuntu changelog style :-)
To post a comment you must log in.
As supsected in the bug report the change was not all that right, in fact Parley's scripting thingy uses kross which can be either krosspython or krossruby, the package now recommends both. Additionally the default scripts use pykde rather than pyqt, fixed that too.
Anyhow, there is cool things to be learned from this:
Since Parley still works without the python magic, depends is not the appropriate relation, but recommends is. The reason for this is that someone might want to install without the python packages and could still use Parley quite normal (well, except for the error), however for the average user recommends will just like depends install the packages and thus provide a good user experience :)
Recommends is a weaker relationship than depends, and for optional scripting capabilities usually recommends is what ought to be used (depends on the particular case at hand of course).