[hafqa] [hafqa] [Bug 1027] maemo-launcher does not install headers

[From: bugzilla-daemon at maemo.org]

https://maemo.org/bugzilla/show_bug.cgi?id=1027





------- Additional Comments From guillem.jover at nokia.com  2007-02-28 23:09 -------
Hey guys,

Sorry for the delay, have been a bit busy lately.

(In reply to comment #6)
> Created an attachment (id=191)
 --> (https://maemo.org/bugzilla/attachment.cgi?id=191&action=view)
> Patch install headers and adds pkg-config file

Thanks for the patch, although it's a bit wrong. ;)
The .pc should not Require all those packages, those are specific to the
launcher no to the boosters. Anyway as I said I've .pc files around I'll
polish them tomorrow.

> With this patch the public headers (report.h and booster.h) are installed to
> {includedir}/maemo-launcher-1.0/maemo-launcher and a pkg-config file
> (maemo-launcher-1.0.pc) is installed in the pkg-config path.

Also I've been meaning to get rid of the report.h dependency from the
boosters, so that I can stop using -rdynamic on the launcher. And I'll have
to remove the booster_load_module from the header as I don't want to expose
that one.

> Please tell if there is anything that needs to be changed or if I should also
> add a -dev debian package.

It's a bit late now, so I'm heading home, but don't worry will take care of
the fixes and commit tomorrow.

-- 
Configure bugmail: https://maemo.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.