Browse Source

Reorganize the makefile.

Try to make it a bit easier to follow the makefile.
mark_story 11 years ago
parent
commit
dfee8a07d9
1 changed files with 20 additions and 11 deletions
  1. 20 11
      Makefile

+ 20 - 11
Makefile

@@ -35,6 +35,18 @@ help:
 	@echo "All other tasks are not intended to be run directly."
 
 
+test: install
+	vendor/bin/phpunit
+
+
+# Utility target for checking required parameters
+guard-%:
+	@if [ "$($*)" = '' ]; then \
+		echo "Missing required $* variable."; \
+		exit 1; \
+	fi;
+
+
 # Download composer
 composer.phar:
 	curl -sS https://getcomposer.org/installer | php
@@ -43,15 +55,9 @@ composer.phar:
 install: composer.phar
 	php composer.phar install
 
-test: install
-	vendor/bin/phpunit
 
-guard-%:
-	@if [ "$($*)" = '' ]; then \
-		echo "Missing required $* variable."; \
-		exit 1; \
-	fi;
 
+# Version bumping & tagging for CakePHP itself
 # Update VERSION.txt to new version.
 bump-version: guard-VERSION
 	@echo "Update VERSION.txt to $(VERSION)"
@@ -69,7 +75,9 @@ tag-release: guard-VERSION bump-version
 	git push $(REMOTE)
 	git push $(REMOTE) --tags
 
-# Tasks for tagging the app skeletong and
+
+
+# Tasks for tagging the app skeleton and
 # creating a zipball of a fully built app skeleton.
 .PHONY: clean tag-app build-app package
 
@@ -90,9 +98,6 @@ tag-app: guard-VERSION build/app
 	cd build/app && git push $(REMOTE)
 	cd build/app && git push $(REMOTE) --tags
 
-# Easier to type alias for zip balls
-package: tag-app dist/cakephp-$(VERSION).zip
-
 dist/cakephp-$(VERSION).zip: composer.phar
 	@if [ ! -d dist ]; \
 	then \
@@ -106,6 +111,10 @@ dist/cakephp-$(VERSION).zip: composer.phar
 	@echo "Building zipball for $(VERSION)"
 	find build/app -not -path '*.git*' | zip dist/cakephp-$(VERSION).zip -@
 
+# Easier to type alias for zip balls
+package: tag-app dist/cakephp-$(VERSION).zip
+
+
 
 # Tasks to publish zipballs to github.
 .PHONY: publish release