Merge pull request grafana#4807 from grafana/makefile

torkelo · torkelo · commit 60855643c4f7 · 2016-04-25T19:01:44.000+02:00
add basic Makefile, remove need for global grunt-cli
diff --git a/Makefile b/Makefile
@@ -0,0 +1,17 @@
+all: deps build
+
+deps:
+	go run build.go setup
+	godep restore
+	npm install
+
+build:
+	go run build.go build
+	npm run build
+
+test:
+	godep go test -v ./pkg/...
+	npm test
+
+run:
+	./bin/grafana-server
diff --git a/README.md b/README.md
@@ -103,8 +103,7 @@ npm (v2.5.0) and grunt (v0.4.5). Run the following:
 
 ```bash
 npm install
-npm install -g grunt-cli
-grunt
+npm run build
 ```
 
 ### Recompile backend on source change
@@ -145,4 +144,3 @@ please [sign the CLA](http://docs.grafana.org/project/cla/)
 
 Grafana is distributed under Apache 2.0 License.
 Work in progress Grafana 2.0 (with included Grafana backend)
-
diff --git a/build.go b/build.go
@@ -306,7 +306,7 @@ func ChangeWorkingDir(dir string) {
 }
 
 func grunt(params ...string) {
-	runPrint("./node_modules/grunt-cli/bin/grunt", params...)
+	runPrint("./node_modules/.bin/grunt", params...)
 }
 
 func setup() {
diff --git a/circle.yml b/circle.yml
@@ -25,12 +25,12 @@ test:
      # Go test
      - godep go test -v ./pkg/...
      # js tests
-     - ./node_modules/grunt-cli/bin/grunt test
+     - npm test
      - npm run coveralls
 
 deployment:
   master:
     branch: master
     owner: grafana
-    commands: 
+    commands:
       - ./trigger_grafana_packer.sh ${TRIGGER_GRAFANA_PACKER_CIRCLECI_TOKEN}
diff --git a/package.json b/package.json
@@ -62,6 +62,7 @@
     "npm": "2.14.x"
   },
   "scripts": {
+    "build": "grunt",
     "test": "grunt test",
     "coveralls": "grunt karma:coveralls && rm -rf ./coverage"
   },

Original file line number	Diff line number	Diff line change
`@@ -306,7 +306,7 @@ func ChangeWorkingDir(dir string) {`
`306`	`306`	`}`
`307`	`307`
`308`	`308`	`func grunt(params ...string) {`
`309`		`- runPrint("./node_modules/grunt-cli/bin/grunt", params...)`
	`309`	`+ runPrint("./node_modules/.bin/grunt", params...)`
`310`	`310`	`}`
`311`	`311`
`312`	`312`	`func setup() {`