Merge lp:~vericalcroft/ubuntu/quantal/classmate-initramfs/added-misc-depends into lp:ubuntu/quantal/classmate-initramfs
Proposed by
Verica
on 2012-09-20
| Status: | Rejected |
|---|---|
| Rejected by: | Stéphane Graber on 2012-11-11 |
| Proposed branch: | lp:~vericalcroft/ubuntu/quantal/classmate-initramfs/added-misc-depends |
| Merge into: | lp:ubuntu/quantal/classmate-initramfs |
| 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:~vericalcroft/ubuntu/quantal/classmate-initramfs/added-misc-depends |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Micah Gersten | Disapprove on 2012-11-09 | ||
| Michael Terry | Approve on 2012-10-04 | ||
| Ubuntu branches | 2012-09-20 | Pending | |
|
Review via email:
|
|||
Description of the Change
Added ${misc:Depends} in the Depends field inside the debian/control file
To post a comment you must log in.
| Micah Gersten (micahg) wrote : | # |
I'm going to upload this with a suite change of s/quantal/raring/
| Micah Gersten (micahg) wrote : | # |
Actually, I'm going to file for removal since this isn't useful anymore
review:
Disapprove
Unmerged revisions
- 7. By Verica Lazova <email address hidden> on 2012-09-20
-
debian/control: added ${misc:Depends} in the Depends field


Thanks for the patch! You're on a misc:Depends roll. :) While correct, this patch also doesn't seem to be necessary. That is, it doesn't seem to add new dependencies.
I'll mark approved, but this can wait until after 12.10 or for another change to be made alongside it.