Jonas Smedegaard <d...@jones.dk> wrote: > On Mon, Apr 06, 2009 at 07:50:40PM -0400, Jay Berkenbilt wrote: >> >>Looking into this further, I believe no patch to cdbs is required. >>Simply including /usr/share/cdbs/1/class/gnome.mk sometime after >>including /usr/share/cdbs/1/rules/debhelper.mk appears to be >>sufficient to cause dh_desktop to be run at the correct time. At >>least for my package, this is sufficient to remove the lintian >>warning. > > Hm. If you won't do it then I will: It is wrong to require a specific > order of including the CDBS snippets, if at all possible to avoid it. > > > Thanks for raising attention to the issue, even if you do not see it as > relevant to fix it (properly). :-)
Actually, I didn't really check carefully about the order. The order may not matter. I didn't have gnome.mk included at all, so I figured the problem was just my omission. However, I see code in gnome.mk that is conditional inside of ifdef _cdbs_rules_debhelper ... endif which is most likely order-dependent since the ifdef will be evaluated at the time that the file is loaded rather than at the time that the rules are evaluated as with ordinary macros. Anyway, if you'd still like me to patch, I can still look into it. I wasn't trying to punt it...just thinking that the error was actually in my rules rather than in cdbs. :-) However, I won't have another chance to look at it until at least the weekend. --Jay -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org