On Fri, Oct 02, 2009 at 04:21:00PM -0400, Felipe Sateler wrote:
On Fri, 2009-10-02 at 13:51 +0200, Jonas Smedegaard wrote:On Thu, Oct 01, 2009 at 01:56:07PM -0400, Felipe Sateler wrote:>I'm not sure if this should be a new bug, but this one is close >enough.>>cdbs (debhelper.mk) provides install/binarypackage targets for each >binary package. However, it does not provide the install target >which sometimes is very useful. Adding it should be a matter of >adding:> >install:: $(patsubst %,install/%,$(DEB_ALL_PACKAGES)) > > >Somewhere in debhelper.mkProviding a new make target has a risk of clashing with existing target of same name but of different type - i.e. above will break packages currently crestihng an install: target (using single colon).Hmm. A different name is an option. The important thing is to have a convenient way to perform the install stage, but not create the binary debs. cdbs-install
Ahh, so it is not because you want some rules attached to that target, just that you want a single target that all install targets depend on?
I think that is a good idea idea for cdbs v2 but I do not imagine an elegant way to add it to current cdbs structure without risk of causing surprises.
Also, I dislike the debhelper.mk rule to implement generic convenience targets not used by itself. If generally useful they might make sense to implement in some core place - but only if not risking clash with local rules files as I fear is the case here.Well, that is another problem. Although the truth is that debhelper.mk is as core as it gets. cdbs is not very useful without it, as it is the only file that can actually generate the debs.
I begin to follow your logic here. And even if in theory other install-related mechanisms is implemented later on, using an :: target allows them to use same target name, so even in theory be used concurrently.
Regards, - Jonas -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
signature.asc
Description: Digital signature
