Merge lp:~hloeung/ubuntu/trusty/pollen/depend-on-ent into lp:ubuntu/trusty/pollen
Proposed by
Haw Loeung
Status: | Rejected |
---|---|
Rejected by: | Jamie Strandboge |
Proposed branch: | lp:~hloeung/ubuntu/trusty/pollen/depend-on-ent |
Merge into: | lp:ubuntu/trusty/pollen |
Diff against target: |
14 lines (+2/-2) 1 file modified
debian/control (+2/-2) |
To merge this branch: | bzr merge lp:~hloeung/ubuntu/trusty/pollen/depend-on-ent |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jamie Strandboge | Disapprove | ||
Review via email: mp+198192@code.launchpad.net |
Description of the change
check_pollen nagios plugin uses ent so make it a dependency.
To post a comment you must log in.
Unmerged revisions
- 8. By Haw Loeung
-
[hloeung] Depend on ent as required by check_pollen nagios plugin.
Thanks for the patch!
I would agree that this should be moved to Depends if the nagios plugin was in its own package. However, it is not and the rest of the package will function correctly if it is not installed. The Recommends will pull ent in on installs though. From Ubuntu Policy:
"Recommends
This declares a strong, but not absolute, dependency.
The Recommends field should list packages that would be found together with this one in all but unusual installations.
"
This seems to be the case here.