| Summary: | Install headers and .pc files when plugins are enabled | ||
|---|---|---|---|
| Product: | Telepathy | Reporter: | Will Thompson <will> |
| Component: | gabble | Assignee: | Telepathy bugs list <telepathy-bugs> |
| Status: | RESOLVED FIXED | QA Contact: | Telepathy bugs list <telepathy-bugs> |
| Severity: | normal | ||
| Priority: | medium | CC: | smcv |
| Version: | unspecified | Keywords: | patch |
| Hardware: | Other | ||
| OS: | All | ||
| URL: | http://git.collabora.co.uk/?p=user/wjt/telepathy-gabble-wjt.git;a=shortlog;h=refs/heads/plugin-headers | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
|
Description
Will Thompson
2009-12-03 10:36:16 UTC
I'd prefer it if the headers were designed to be included like <gabble/sidecar.h>; you can either install them like $includedir/telepathy-1.0/gabble/gabble/sidecar.h or $includedir/telepathy-1.0/gabble/sidecar.h, but please put a parent of their directory on the -I path, rather than their directory itself. You basically have to use one of these styles, if you want to avoid colliding header names: * install to $FOO/foo/bar.h, use -I$FOO and consistently include <foo/bar.h> (telepathy-glib) * call your headers foo-*.h (breaking tab completion), install to $FOO/foo-bar.h and use -I$FOO (NetworkManager) * both of the above, redundantly (D-Bus, GLib) Updated the branch accordingly. Merged with review by Simon, thanks. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.