What's the appropriate Makefile-fu to change someport.conf to someport.conf.sample?

[Tillman Hodgson]

On Mon, Dec 22, 2003 at 09:36:10AM -0600, Pete Fritchman wrote:
> * Mon, 22 Dec 2003 09:04:23 CST - Tillman Hodgson:
> | pre-install:
> |         cp ${WRKSRC}/latd.conf ${WRKSRC}/latd.conf.sample
> 
> That will make a file named .sample, but not actually install it.  If
> you're using the upstream Makefile for installing (as opposed to a
> do-install target), you'll probably need to patch that to know about
> installing latd.conf.sample (and not installing latd.conf over top of
> an existing latd.conf!).

Is the following acceptable practice?

pre-configure:
	@${CP} ${WRKSRC}/latd.conf ${WRKSRC}/latd.conf.sample
	@${SED} -i .dist -e 's/sysconf_DATA = latd.conf/sysconf_DATA = latd.conf.sample/' ${WRKSRC}/Makefile.in

`portlint` thinks that it looks fine, for what that's worth :-)

-T

-- 
Infinity attracts us like a floodlight in the night, blinding us to the 
excesses it can inflict upon the finite.
	- Meditations from Bifrost Eyrie, Buddislamic Text