浏览代码

vFormat: use gci instead of customized goimports (#1020)

Loyalsoldier 4 年之前
父节点
当前提交
b57ebd5c1c
共有 3 个文件被更改,包括 5 次插入7 次删除
  1. 1 1
      format.go
  2. 3 5
      infra/vformat/main.go
  3. 1 1
      transport/internet/headers/http/linkedreadRequest.go

+ 1 - 1
format.go

@@ -1,4 +1,4 @@
 package core
 
-//go:generate go install -v github.com/v2fly/tools/cmd/goimports@latest
+//go:generate go install -v github.com/daixiang0/gci@latest
 //go:generate go run ./infra/vformat/

+ 3 - 5
infra/vformat/main.go

@@ -3,7 +3,6 @@ package main
 import (
 	"fmt"
 	"go/build"
-	"io/ioutil"
 	"os"
 	"os/exec"
 	"path/filepath"
@@ -42,7 +41,7 @@ func GetRuntimeEnv(key string) (string, error) {
 	}
 	var data []byte
 	var runtimeEnv string
-	data, readErr := ioutil.ReadFile(file)
+	data, readErr := os.ReadFile(file)
 	if readErr != nil {
 		return "", readErr
 	}
@@ -50,7 +49,7 @@ func GetRuntimeEnv(key string) (string, error) {
 	for _, envItem := range envStrings {
 		envItem = strings.TrimSuffix(envItem, "\r")
 		envKeyValue := strings.Split(envItem, "=")
-		if strings.EqualFold(strings.TrimSpace(envKeyValue[0]), key) {
+		if len(envKeyValue) == 2 && strings.TrimSpace(envKeyValue[0]) == key {
 			runtimeEnv = strings.TrimSpace(envKeyValue[1])
 		}
 	}
@@ -123,7 +122,7 @@ func main() {
 		suffix = ".exe"
 	}
 	gofmt := "gofmt" + suffix
-	goimports := "goimports" + suffix
+	goimports := "gci" + suffix
 
 	if gofmtPath, err := exec.LookPath(gofmt); err != nil {
 		fmt.Println("Can not find", gofmt, "in system path or current working directory.")
@@ -172,7 +171,6 @@ func main() {
 
 	goimportsArgs := []string{
 		"-w",
-		"-r",
 		"-local", "github.com/v2fly/v2ray-core",
 	}
 

+ 1 - 1
transport/internet/headers/http/linkedreadRequest.go

@@ -3,7 +3,7 @@ package http
 import (
 	"bufio"
 	"net/http"
-	_ "unsafe" // required to use //go:linkname
+	_ "unsafe"
 )
 
 //go:linkname readRequest net/http.readRequest