r4473 - packages/trunk/ketm/debian

[Eddy Petrișor]

(are you aware that you have the Reply-To set, even if you sent to the list?)

Gonéri Le Bouder wrote:
> On Tue, Oct 30, 2007 at 01:04:26PM +0200, Eddy Petrișor wrote:
>>> +	# Install desktop and icon files
>>> +	install -D -m 644 debian/ketm.desktop debian/ketm/usr/share/applications/ketm.desktop
>>> +	install -D -m 644 debian/ketm.xpm debian/ketm/usr/share/pixmaps/ketm.xpm
>>> +
>> Could you stop using install directly and use dh_install* instead? Pretty please!
> Can you explain the reason. The manpage is not very clear about
> that.

- First note that I said 'dh_install*' not 'dh_install'.
- Second, consistency with the rest of the rules file and ease of following up
of the flow (e.g. "oh, they're using debhelper, ok, all thos missing files in
the .install file")
- debian/rules is cleaner this way
- usually dh_* scripts are a lot smarter that any individual can come up with in
a shot
- why would you use install for some of files and leave the rest to dh_install*?
- dh_* offers really nice ways to deal with exceptions


Question of the day: Did you know that debhelper compat 5 does not install 0
sized files?

-- 
Regards,
EddyP
=============================================
"Imagination is more important than knowledge" A.Einstein

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: OpenPGP digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-games-devel/attachments/20071031/c701beb2/attachment.pgp