Don't overwrite Makefile on generate (#169)

* Don't overwrite Makefile on generate

* Fix whitespace

Author: omkhegde

python/rpdk/go/codegen.py

@@ -77,6 +77,13 @@ def init(self, project):
         inter = project.root / "internal"
         inter.mkdir(parents=True, exist_ok=True)
 
+        # Makefile
+        path = project.root / "Makefile"
+        LOG.debug("Writing Makefile: %s", path)
+        template = self.env.get_template("Makefile")
+        contents = template.render()
+        project.overwrite(path, contents)
+
         # go.mod
         path = project.root / "go.mod"
         LOG.debug("Writing go.mod: %s", path)
@@ -167,10 +174,10 @@ def generate(self, project):
         project.overwrite(path, contents)
         format_paths.append(path)
 
-        # Makefile
-        path = project.root / "Makefile"
-        LOG.debug("Writing Makefile: %s", path)
-        template = self.env.get_template("Makefile")
+        # makebuild
+        path = project.root / "makebuild"
+        LOG.debug("Writing makebuild: %s", path)
+        template = self.env.get_template("makebuild")
         contents = template.render()
         project.overwrite(path, contents)
 

python/rpdk/go/templates/Makefile

@@ -1,8 +1,7 @@
 .PHONY: build test clean
 
 build:
-	cfn generate
-	env GOOS=linux go build -ldflags="-s -w" -tags="logging" -o bin/handler cmd/main.go
+	make -f makebuild  # this runs build steps required by the cfn cli
 
 test:
 	cfn generate

python/rpdk/go/templates/makebuild

@@ -0,0 +1,7 @@
+# This file is autogenerated, do not edit;
+# changes will be undone by the next 'generate' command.
+
+.PHONY: build
+build:
+	cfn generate
+	env GOOS=linux go build -ldflags="-s -w" -tags="$(TAGS)" -o bin/handler cmd/main.go

python/rpdk/go/version.py

@@ -25,7 +25,7 @@
 * Replace `model.{field}.Value()` with `aws.{Type}Value(model.{field})`
 
 Where {Type} is either String, Bool, Int, or Float64 and {field} is any field within your generated model.
-""",
+"""
 }