Merge lp://staging/~evgeni/libappindicator/use-autoreconf.mk into lp://staging/~ubuntu-desktop/libappindicator/ubuntu
Proposed by
Evgeni Golov
| Status: | Merged |
|---|---|
| Merged at revision: | 296 |
| Proposed branch: | lp://staging/~evgeni/libappindicator/use-autoreconf.mk |
| Merge into: | lp://staging/~ubuntu-desktop/libappindicator/ubuntu |
| Diff against target: |
24 lines (+2/-2) 1 file modified
debian/rules (+2/-2) |
| To merge this branch: | bzr merge lp://staging/~evgeni/libappindicator/use-autoreconf.mk |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Robert Ancell | Approve | ||
| Ken VanDine | Pending | ||
|
Review via email:
|
|||
Description of the change
use autoreconf.mk instead of calling dh_autoreconf by hand
- fixes FTBFS when there are multiple Python versions
(dh_autoreconf would be called multiple times in this case, which isn't
allowed)
- properly cleans autoreconf'ed files in the clean target
- patchsys-quilt.mk needs to be included *before* autoreconf.mk or
debhelper will leave uncleaned files in debian/, leading to
E: libappindicator source: temporary-
from lintian
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks good, uploaded to quantal.