🧹 make 'make build' a thing (mondoohq#69)

* make 'make build' a thing The README mentions running 'make build', but there is no such Makefile target. Add a 'make build' target to build the cnquery binary. Signed-off-by: Joel Diaz <[email protected]> * move 'make build' to more appropriate section of Makefile Rename it to 'make cnquery/build', and update README to mention the proper names of the makefile targets. Signed-off-by: Joel Diaz <[email protected]>
benr · Sep 20, 2022 · 5a779f2 · 5a779f2
1 parent d3f91e1
commit 5a779f2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,6 +2,7 @@
 dist
 alpine-container.tar
 centos-container.tar
+cnquery
 file1
 id_rsa
 id_rsa.pub

diff --git a/Makefile b/Makefile
@@ -37,7 +37,6 @@ clean/proto:
 version:
  @echo $(VERSION)
 
-
 # 🔨 TOOLS #
 
 prep: prep/tools
@@ -201,6 +200,10 @@ mqlc/test:
 
 # 🏗 Binary / Build #
 
+.PHONY: cnquery/build
+cnquery/build:
+ go build -o cnquery ${LDFLAGSDIST} apps/cnquery/cnquery.go
+
 .PHONY: cnquery/install
 cnquery/install:
  GOBIN=${GOPATH}/bin go install ${LDFLAGSDIST} apps/cnquery/cnquery.go

diff --git a/README.md b/README.md
@@ -28,8 +28,8 @@ Building:
 make prep 
 
 # build and install cnquery
-make build
-make install
+make cnquery/build
+make cnquery/install
 ```
 
 Some files in this repo are auto-generated. Whenever a proto or resource pack is changed, these will need to be rebuilt. Please re-run: