Comment 2 for bug 1050026

Revision history for this message
Charles Kerr (charlesk) wrote :

Background information from IRC:

[15:43:51] <charles> mterry: as an aside, could you clarify what you're looking for in the sync-menu project side of https://bugs.launchpad.net/indicator-sync/+bug/1050026
[15:44:13] <charles> the automake rules in there are the same as in the other indicators, so I'm not sure what you're looking for
[15:44:24] <-- Ursinha` (~ursinha@177.97.248.219) has quit (Ping timeout: 246 seconds)
[15:44:44] <mterry> charles, the other indicators are wrong too
[15:44:45] <charles> (and if it's broken in i-sync, is it broken everywhere else and just worked around in the packaging rules)
[15:45:07] <charles> mterry: could you point to a Makefile.am that does it correctly, so that I can use it as reference?
[15:45:31] <mterry> charles, it's just a matter of changing libexec to pkglibexec
[15:45:45] <mterry> charles, so in the service.in file and in the two Makefile.am's that reference it
[15:46:01] <mterry> charles, the reason this may seem like a new issue is because dh9 changed its default behavior
[15:46:17] <mterry> charles, there is a long-standing bug that debhelper will include the source package name in libexecdir, when it shouldn't
[15:46:40] <mterry> charles, which meant pkglibexecdir always looked like /usr/lib/package/package, which most people thought looked stupid and so didn't use it
[15:46:56] <mterry> charles, but dh compat level 9 fixes that, so upstreams should really use the correct variable
[15:47:29] <mterry> charles, let me dig up a correct Makefile.am just for clarity
[15:47:46] <charles> okay. So I would just replace "libexec_PROGRAMS = indicator-foo-service" with "pkglibexec_PROGRAMS = indicator-foo-service"
[15:47:51] <charles> iiuc
[15:48:13] <mterry> charles, http://bazaar.launchpad.net/~deja-dup-hackers/deja-dup/24/view/head:/data/Makefile.am for example
[15:48:14] <charles> + the service.in
[15:48:17] <mterry> charles, yeah
[15:48:24] <mterry> charles, and the data/Makefile.am
[15:48:28] <mterry> which does the sed
[15:48:44] <mterry> charles, and spread the word to the rest of the indicator team :)
[15:48:47] <charles> gotcha, thanks mterry