X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=cmd%2Fbinapi-generator%2Fmain.go;h=fcd85ae97b2da5614a82771af8084b6305f1b4b6;hb=d372b4efe26650dbc83908ca0bbee38d90aed3e9;hp=8bf765b3831c3c719336c249cc4930cccfb6fad0;hpb=2c419b100f5f90c1215617d0fac81fa6749c4d6b;p=govpp.git

diff --git a/cmd/binapi-generator/main.go b/cmd/binapi-generator/main.go
index 8bf765b..fcd85ae 100644
--- a/cmd/binapi-generator/main.go
+++ b/cmd/binapi-generator/main.go
@@ -19,9 +19,9 @@ import (
 	"encoding/json"
 	"flag"
 	"fmt"
+	"go/format"
 	"io/ioutil"
 	"os"
-	"os/exec"
 	"path/filepath"
 	"strings"
 
@@ -204,10 +204,15 @@ func generateFromFile(inputFile, outputDir string, typesPkgs []*context) error {
 		}
 	}
 
-	// generate Go package code
+	// generate Go package
 	var buf bytes.Buffer
 	if err := generatePackage(ctx, &buf); err != nil {
-		return fmt.Errorf("generating code for package %s failed: %v", ctx.packageName, err)
+		return fmt.Errorf("generating Go package for %s failed: %v", ctx.packageName, err)
+	}
+	// format generated source code
+	gosrc, err := format.Source(buf.Bytes())
+	if err != nil {
+		return fmt.Errorf("formatting source code for package %s failed: %v", ctx.packageName, err)
 	}
 
 	// create output directory
@@ -216,16 +221,10 @@ func generateFromFile(inputFile, outputDir string, typesPkgs []*context) error {
 		return fmt.Errorf("creating output dir %s failed: %v", packageDir, err)
 	}
 	// write generated code to output file
-	if err := ioutil.WriteFile(ctx.outputFile, buf.Bytes(), 0666); err != nil {
+	if err := ioutil.WriteFile(ctx.outputFile, gosrc, 0666); err != nil {
 		return fmt.Errorf("writing to output file %s failed: %v", ctx.outputFile, err)
 	}
 
-	// go format the output file (fail probably means the output is not compilable)
-	cmd := exec.Command("gofmt", "-w", ctx.outputFile)
-	if output, err := cmd.CombinedOutput(); err != nil {
-		return fmt.Errorf("gofmt failed: %v\n%s", err, string(output))
-	}
-
 	return nil
 }