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 | ||||||||
Related bugs: |
|
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.