Re: [Tuxpaint-devel] Tux Paint 0.9.15 release plans

[Ben A. <sy...@sa...>]

On Mon, 2005-10-10 at 08:43 -0300, Ben Armstrong wrote:
> On Sun, 2005-10-09 at 17:10 -0700, Bill Kendrick wrote:
> > I've added a "make release" target.  Thx! :)
> 
>         @cd .. ; \
>          find tuxpaint-$(VER_VERSION) -follow \
>              \( -wholename '*/CVS' -o -name .cvsignore \) \
>              -prune -o -type f -print0 | \
>            xargs -0 tar -czvf tuxpaint-$(VER_VERSION).tar.gz 
> 
> What creates the tuxpaint-$(VER_VERSION) tree?  I could find no target
> that does this.  Perhaps you created this tree manually?  The "release"
> target needs to either do this itself or depend on a target that does
> this.

By the way, I don't really like this business of making things outside
of the current tree.  The way I have most often see other packages
handle this is to create a directory called "build" which is populated
with the build target directories, something like:

Index: Makefile
===================================================================
RCS file: /cvsroot/tuxpaint/tuxpaint/Makefile,v
retrieving revision 1.79
diff -r1.79 Makefile
125c125,141
< release:
---
> releaseclean:
> 	@echo
> 	@echo "Cleaning release directory"
> 	@echo
> 	@rm -rf build/tuxpaint-$(VER_VERSION)
> 
> releasedir: releaseclean
> 	@echo
> 	@echo "Creating release directory"
> 	@echo
> 	@mkdir -p build/tuxpaint-$(VER_VERSION)
> 	@find . -follow \
> 	     \( -wholename '*/CVS' -o -name .cvsignore -o -name 'build' \) \
> 	     -prune -o -type f -exec cp --parents -vdp \{\} build/tuxpaint-$(VER_VERSION)/ \;
> 
> 	
> release: build/tuxpaint-$(VER_VERSION)
128,135c144,147
< 	@
< 	@make clean
< 	-@rm -f ../tuxpaint-$(VER_VERSION).tar
< 	@cd .. ; \
< 	 find tuxpaint-$(VER_VERSION) -follow \
< 	     \( -wholename '*/CVS' -o -name .cvsignore \) \
< 	     -prune -o -type f -print0 | \
< 	   xargs -0 tar -czvf tuxpaint-$(VER_VERSION).tar.gz 
---
> 	@echo
> 	-@rm -f build/tuxpaint-$(VER_VERSION).tar.gz
> 	@cd build ; \
> 	    tar -czvf tuxpaint-$(VER_VERSION).tar.gz tuxpaint-$(VER_VERSION)

Nothing cleans "build" here because there might be a build from a prior
version still kicking around.  Maybe we need a reallyclean target that
nukes absolutely everything, including all build dir contents.

Ben