Merge lp://qastaging/~laney/libappindicator/cli-pcfile-arch-all-trunk into lp://qastaging/libappindicator/13.10

Proposed by Iain Lane
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 258
Merged at revision: 258
Proposed branch: lp://qastaging/~laney/libappindicator/cli-pcfile-arch-all-trunk
Merge into: lp://qastaging/libappindicator/13.10
Diff against target: 36 lines (+11/-2)
3 files modified
bindings/mono/appindicator-sharp-0.1.pc.in (+1/-1)
debian/changelog (+9/-0)
debian/libappindicator0.1-cil-dev.install (+1/-1)
To merge this branch: bzr merge lp://qastaging/~laney/libappindicator/cli-pcfile-arch-all-trunk
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
PS Jenkins bot (community) continuous-integration Approve
Mathieu Trudel-Lapierre Approve
Review via email: mp+159166@code.qastaging.launchpad.net

Commit message

Install appindicator-sharp's pcfile into /usr/share/pkgconfig as this is an arch:all package and library. Update the assemblydir variable in this pcfile to refer to the correct location for the library instead of an incorrect multiarch directory.

Description of the change

The pcfile for appindicator-sharp is broken in a couple of related ways

  * It's installed into a Multi-Arch directory in an arch:all package.
  * It references a multiarch libdir when the library isn't installed in one.

Fix both of these by properly using the unversioned locations.

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

This is a resubmission of https://code.launchpad.net/~laney/libappindicator/cil-pcfile-arch-all/+merge/158086 - please could you set that to rejected or whatever is appropriate?

Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Approve.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

looking fine now! Thanks :)

review: Approve

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