Code review comment for lp:~statik/tarmac/makefile

Revision history for this message
Paul Hummer (rockstar) wrote :

Thanks for the branch!

 vote approve
 status approve

> === added file 'Makefile'
> --- Makefile 1970-01-01 00:00:00 +0000
> +++ Makefile 2009-03-05 02:22:12 +0000
> @@ -0,0 +1,8 @@
> +TESTFLAGS=
> +test:
> + trial $(TESTFLAGS) tarmac
> +
> +test-fail:
> + trial this-will-fail
> +
> +.PHONY: test test-fail
>

This is a clever way of dealing with the difficulty with testing. I like it.

--
Paul Hummer
http://theironlion.net
1024/862FF08F C921 E962 58F8 5547 6723 0E8C 1C4D 8AC5 862F F08F

review: Approve

« Back to merge proposal