Merge lp://staging/~dpm/qreator/packaging-fixes into lp://staging/qreator

Proposed by David Planella
Status: Merged
Merged at revision: 191
Proposed branch: lp://staging/~dpm/qreator/packaging-fixes
Merge into: lp://staging/qreator
Diff against target: 37 lines (+12/-5)
1 file modified
debian/control (+12/-5)
To merge this branch: bzr merge lp://staging/~dpm/qreator/packaging-fixes
Reviewer Review Type Date Requested Status
Qreator Hackers Pending
Review via email: mp+190790@code.staging.launchpad.net

Commit message

Fixed some packaging issues and added extra fields to the control file

Description of the change

Fix for bug 1210408

Packaging fixes
- Changed priority from extra to optional
- Added python-qrencode and dh-translations to the build dependencies
- Added Homepage and VCS-related fields
- Removed yelp from the dependencies
- Made software-center a Recommends

One thing I'm not sure about is whether it is necessary at all to add python-qrencode to the build dependencies. If we're adding it, we might as well add all the runtime dependencies, as without it qreator won't run during the build step. But we're not actually running qreator during the build.

To post a comment you must log in.
180. By David Planella

Fixed homepage

181. By David Planella

Added dependencies to geolocation providers and to the GNOME symbolic icons package so that they get installed in those distributions in which they're not by default (e.g. Xubuntu)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches