.github/workflows/ci.yaml

@@ -3,12 +3,9 @@ name: CI
 on:
   push:
     branches:
-    - main
+    - master
   pull_request:
 
-permissions:
-  contents: read
-
 jobs:
   test:
     name: Test
@@ -18,9 +15,9 @@ jobs:
         os: [ubuntu-latest, macos-latest, windows-latest]
     steps:
     - name: Checkout code
-      uses: actions/checkout@v3
+      uses: actions/checkout@v2
     - name: Setup Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v2
       with:
         go-version: 1.x
     - name: Test

.github/workflows/release.yaml

@@ -5,24 +5,28 @@ on:
     tags:
     - 'v*'
 
-permissions:
-  contents: write
-
 jobs:
   release:
     name: Release
     runs-on: ubuntu-latest
     steps:
     - name: Checkout code
-      uses: actions/checkout@v3
+      uses: actions/checkout@v2
     - name: Setup Go
-      uses: actions/setup-go@v4
+      uses: actions/setup-go@v2
       with:
         go-version: 1.x
     - name: Cross build
       run: make cross
     - name: Create Release
-      uses: ncipollo/release-action@v1
+      id: create_release
+      uses: actions/create-release@v1
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
       with:
-        name: Release ${{ github.ref_name }}
+        tag_name: ${{ github.ref }}
-        artifacts: 'goxz/*'
+        release_name: Release ${{ github.ref }}
+    - name: Upload
+      run: make upload
+      env:
+        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

CHANGELOG.md

@@ -1,13 +1,4 @@
 # Changelog
-## [v0.1.6](https://github.com/itchyny/mmv/compare/v0.1.5..v0.1.6) (2023-04-26)
-* Remove dependency on shell for splitting EDITOR with spaces.
-
-## [v0.1.5](https://github.com/itchyny/mmv/compare/v0.1.4..v0.1.5) (2023-04-09)
-* Support EDITOR with spaces in the editor path.
-
-## [v0.1.4](https://github.com/itchyny/mmv/compare/v0.1.3..v0.1.4) (2021-09-18)
-* Release `arm64` artifacts.
-
 ## [v0.1.3](https://github.com/itchyny/mmv/compare/v0.1.2..v0.1.3) (2021-01-10)
 * Support renaming when one of the paths is a parent directory of another.
 

LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2020-2023 itchyny
+Copyright (c) 2020-2021 itchyny
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

Makefile

@@ -1,9 +1,10 @@
 BIN := mmv
 VERSION := $$(make -s show-version)
 VERSION_PATH := cmd/$(BIN)
-CURRENT_REVISION = $(shell git rev-parse --short HEAD)
+CURRENT_REVISION := $(shell git rev-parse --short HEAD)
-BUILD_LDFLAGS = "-s -w -X main.revision=$(CURRENT_REVISION)"
+BUILD_LDFLAGS := "-s -w -X main.revision=$(CURRENT_REVISION)"
 GOBIN ?= $(shell go env GOPATH)/bin
+export GO111MODULE=on
 
 .PHONY: all
 all: build
@@ -14,40 +15,40 @@ build:
 
 .PHONY: install
 install:
-	go install -ldflags=$(BUILD_LDFLAGS) ./cmd/$(BIN)
+	go install -ldflags=$(BUILD_LDFLAGS) ./...
 
 .PHONY: show-version
 show-version: $(GOBIN)/gobump
-	@gobump show -r "$(VERSION_PATH)"
+	@gobump show -r $(VERSION_PATH)
 
 $(GOBIN)/gobump:
-	@go install github.com/x-motemen/gobump/cmd/gobump@latest
+	@cd && go get github.com/x-motemen/gobump/cmd/gobump
 
 .PHONY: cross
 cross: $(GOBIN)/goxz CREDITS
 	goxz -n $(BIN) -pv=v$(VERSION) -build-ldflags=$(BUILD_LDFLAGS) ./cmd/$(BIN)
 
 $(GOBIN)/goxz:
-	go install github.com/Songmu/goxz/cmd/goxz@latest
+	cd && go get github.com/Songmu/goxz/cmd/goxz
 
 CREDITS: $(GOBIN)/gocredits go.sum
 	go mod tidy
 	gocredits -w .
 
 $(GOBIN)/gocredits:
-	go install github.com/Songmu/gocredits/cmd/gocredits@latest
+	cd && go get github.com/Songmu/gocredits/cmd/gocredits
 
 .PHONY: test
 test: build
-	go test -v -race ./...
+	go test -v ./...
 
 .PHONY: lint
-lint: $(GOBIN)/staticcheck
+lint: $(GOBIN)/golint
 	go vet ./...
-	staticcheck -checks all ./...
+	golint -set_exit_status ./...
 
-$(GOBIN)/staticcheck:
+$(GOBIN)/golint:
-	go install honnef.co/go/tools/cmd/staticcheck@latest
+	cd && go get golang.org/x/lint/golint
 
 .PHONY: clean
 clean:
@@ -56,9 +57,21 @@ clean:
 
 .PHONY: bump
 bump: $(GOBIN)/gobump
-	test -z "$$(git status --porcelain || echo .)"
+ifneq ($(shell git status --porcelain),)
-	test "$$(git branch --show-current)" = "main"
+	$(error git workspace is dirty)
+endif
+ifneq ($(shell git rev-parse --abbrev-ref HEAD),master)
+	$(error current branch is not master)
+endif
 	@gobump up -w "$(VERSION_PATH)"
 	git commit -am "bump up version to $(VERSION)"
 	git tag "v$(VERSION)"
-	git push --atomic origin main tag "v$(VERSION)"
+	git push origin master
+	git push origin "refs/tags/v$(VERSION)"
+
+.PHONY: upload
+upload: $(GOBIN)/ghr
+	ghr "v$(VERSION)" goxz
+
+$(GOBIN)/ghr:
+	cd && go get github.com/tcnksm/ghr

README.md

@@ -1,7 +1,7 @@
 # mmv
 [![CI Status](https://github.com/itchyny/mmv/workflows/CI/badge.svg)](https://github.com/itchyny/mmv/actions)
 [![Go Report Card](https://goreportcard.com/badge/github.com/itchyny/mmv)](https://goreportcard.com/report/github.com/itchyny/mmv)
-[![MIT License](https://img.shields.io/badge/license-MIT-blue.svg)](https://github.com/itchyny/mmv/blob/main/LICENSE)
+[![MIT License](http://img.shields.io/badge/license-MIT-blue.svg)](https://github.com/itchyny/mmv/blob/master/LICENSE)
 [![release](https://img.shields.io/github/release/itchyny/mmv/all.svg)](https://github.com/itchyny/mmv/releases)
 [![pkg.go.dev](https://pkg.go.dev/badge/github.com/itchyny/mmv)](https://pkg.go.dev/github.com/itchyny/mmv)
 
@@ -24,7 +24,7 @@ brew install itchyny/tap/mmv
 
 ### Build from source
 ```bash
-go install github.com/itchyny/mmv/cmd/mmv@latest
+go get github.com/itchyny/mmv/cmd/mmv
 ```
 
 ## Features

cmd/mmv/main.go

@@ -4,12 +4,14 @@ import (
 	"errors"
 	"flag"
 	"fmt"
+	"io/ioutil"
+	"math/rand"
 	"os"
 	"os/exec"
 	"runtime"
 	"strings"
+	"time"
 
-	"github.com/kballard/go-shellquote"
 	_ "github.com/mattn/getwild"
 	"github.com/mattn/go-tty"
 
@@ -18,10 +20,14 @@ import (
 
 const name = "mmv"
 
-const version = "0.1.6"
+const version = "0.1.3"
 
 var revision = "HEAD"
 
+func init() {
+	rand.Seed(time.Now().UnixNano())
+}
+
 func main() {
 	os.Exit(run(os.Args[1:]))
 }
@@ -80,7 +86,7 @@ func rename(args []string) error {
 		xs[src] = true
 	}
 
-	f, err := os.CreateTemp("", name+"-")
+	f, err := ioutil.TempFile("", name+"-")
 	if err != nil {
 		return err
 	}
@@ -103,10 +109,7 @@ func rename(args []string) error {
 	if editor == "" {
 		editor = "vi"
 	}
-	editorWithArgs, err := shellquote.Split(editor)
+	editorWithArgs := strings.Fields(editor)
-	if err != nil {
-		return fmt.Errorf("%s: %s", err, editor)
-	}
 	editorWithArgs = append(editorWithArgs, f.Name())
 
 	cmd := exec.Command(editorWithArgs[0], editorWithArgs[1:]...)
@@ -117,7 +120,7 @@ func rename(args []string) error {
 		return fmt.Errorf("abort renames: %s", err)
 	}
 
-	cnt, err := os.ReadFile(f.Name())
+	cnt, err := ioutil.ReadFile(f.Name())
 	if err != nil {
 		return err
 	}

go.mod

@@ -1,14 +1,10 @@
 module github.com/itchyny/mmv
 
-go 1.20
+go 1.15
 
 require (
-	github.com/kballard/go-shellquote v0.0.0-20180428030007-95032a82bc51
 	github.com/mattn/getwild v0.0.2-0.20200919000855-c2e221927ad6
-	github.com/mattn/go-tty v0.0.4
+	github.com/mattn/go-isatty v0.0.12 // indirect
-)
+	github.com/mattn/go-tty v0.0.3
-
+	golang.org/x/sys v0.0.0-20210108172913-0df2131ae363 // indirect
-require (
-	github.com/mattn/go-isatty v0.0.18 // indirect
-	golang.org/x/sys v0.7.0 // indirect
 )

go.sum

@@ -1,18 +1,17 @@
-github.com/kballard/go-shellquote v0.0.0-20180428030007-95032a82bc51 h1:Z9n2FFNUXsshfwJMBgNA0RU6/i7WVaAegv3PtuIHPMs=
-github.com/kballard/go-shellquote v0.0.0-20180428030007-95032a82bc51/go.mod h1:CzGEWj7cYgsdH8dAjBGEr58BoE7ScuLd+fwFZ44+/x8=
 github.com/mattn/getwild v0.0.2-0.20200919000855-c2e221927ad6 h1:uWR+2CTTaHQzDS/DApbJ2H8UEPQl90atrKtczXj2xcs=
 github.com/mattn/getwild v0.0.2-0.20200919000855-c2e221927ad6/go.mod h1:AG+GKQydHp7iLJn+VV+D7y8LeYs5bQ0Xz4fmKd5o1Sg=
 github.com/mattn/go-colorable v0.1.4/go.mod h1:U0ppj6V5qS13XJ6of8GYAs25YV2eR4EVcfRqFIhoBtE=
 github.com/mattn/go-isatty v0.0.8/go.mod h1:Iq45c/XA43vh69/j3iqttzPXn0bhXyGjM0Hdxcsrc5s=
 github.com/mattn/go-isatty v0.0.10/go.mod h1:qgIWMr58cqv1PHHyhnkY9lrL7etaEgOFcMEpPG5Rm84=
-github.com/mattn/go-isatty v0.0.18 h1:DOKFKCQ7FNG2L1rbrmstDN4QVRdS89Nkh85u68Uwp98=
+github.com/mattn/go-isatty v0.0.12 h1:wuysRhFDzyxgEmMf5xjvJ2M9dZoWAXNNr5LSBS7uHXY=
-github.com/mattn/go-isatty v0.0.18/go.mod h1:W+V8PltTTMOvKvAeJH7IuucS94S2C6jfK/D7dTCTo3Y=
+github.com/mattn/go-isatty v0.0.12/go.mod h1:cbi8OIDigv2wuxKPP5vlRcQ1OAZbq2CE4Kysco4FUpU=
-github.com/mattn/go-runewidth v0.0.7/go.mod h1:H031xJmbD/WCDINGzjvQ9THkh0rPKHF+m2gUSrubnMI=
+github.com/mattn/go-runewidth v0.0.6/go.mod h1:H031xJmbD/WCDINGzjvQ9THkh0rPKHF+m2gUSrubnMI=
-github.com/mattn/go-tty v0.0.4 h1:NVikla9X8MN0SQAqCYzpGyXv0jY7MNl3HOWD2dkle7E=
+github.com/mattn/go-tty v0.0.3 h1:5OfyWorkyO7xP52Mq7tB36ajHDG5OHrmBGIS/DtakQI=
-github.com/mattn/go-tty v0.0.4/go.mod h1:u5GGXBtZU6RQoKV8gY5W6UhMudbR5vXnUe7j3pxse28=
+github.com/mattn/go-tty v0.0.3/go.mod h1:ihxohKRERHTVzN+aSVRwACLCeqIoZAWpoICkkvrWyR0=
+golang.org/x/sync v0.0.0-20190911185100-cd5d95a43a6e/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
 golang.org/x/sys v0.0.0-20190222072716-a9d3bda3a223/go.mod h1:STP8DvDyc/dI5b8T5hshtkjS+E42TnysNCUPdjciGhY=
 golang.org/x/sys v0.0.0-20191008105621-543471e840be/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20191120155948-bd437916bb0e/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
-golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
+golang.org/x/sys v0.0.0-20200116001909-b77594299b42/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
-golang.org/x/sys v0.7.0 h1:3jlCCIQZPdOYu1h8BkNvLz8Kgwtae2cagcG/VamtZRU=
+golang.org/x/sys v0.0.0-20210108172913-0df2131ae363 h1:wHn06sgWHMO1VsQ8F+KzDJx/JzqfsNLnc+oEi07qD7s=
-golang.org/x/sys v0.7.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
+golang.org/x/sys v0.0.0-20210108172913-0df2131ae363/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=

mmv.go

@@ -1,11 +1,10 @@
-// Package mmv provides a method to rename multiple files.
 package mmv
 
 import (
-	"crypto/rand"
+	"math/rand"
-	"encoding/base64"
 	"os"
 	"path/filepath"
+	"strconv"
 	"strings"
 )
 
@@ -251,12 +250,8 @@ func buildRenames(files map[string]string) ([]rename, error) {
 
 // create a temporary path where there is no file currently
 func temporaryPath(dir string) (string, error) {
-	bs := make([]byte, 16)
 	for i := 0; i < 256; i++ {
-		if _, err := rand.Read(bs); err != nil {
+		path := filepath.Join(dir, strconv.FormatUint(rand.Uint64()|1<<60, 16))
-			return "", err
-		}
-		path := filepath.Join(dir, base64.RawURLEncoding.EncodeToString(bs))
 		if _, err := os.Stat(path); err != nil && os.IsNotExist(err) {
 			return path, nil
 		}

mmv_test.go

@@ -1,6 +1,7 @@
 package mmv
 
 import (
+	"io/ioutil"
 	"os"
 	"path/filepath"
 	"reflect"
@@ -384,9 +385,9 @@ func TestRename(t *testing.T) {
 	}
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			dir, err := os.MkdirTemp("", "mmv-")
+			dir, err := ioutil.TempDir("", "mmv-")
 			if err != nil {
-				t.Fatalf("os.MkdirTemp returned an error: %s", err)
+				t.Fatalf("ioutil.TempDir returned an error: %s", err)
 			}
 			t.Cleanup(func() { os.RemoveAll(dir) })
 			if err := os.Chdir(dir); err != nil {
@@ -422,7 +423,7 @@ func setupFiles(contents map[string]string) error {
 				return err
 			}
 		}
-		if err := os.WriteFile(f, []byte(cnt), 0o600); err != nil {
+		if err := ioutil.WriteFile(f, []byte(cnt), 0o600); err != nil {
 			return err
 		}
 	}
@@ -431,7 +432,7 @@ func setupFiles(contents map[string]string) error {
 
 func fileContents(dir string) map[string]string {
 	m := make(map[string]string)
-	fis, _ := os.ReadDir(dir)
+	fis, _ := ioutil.ReadDir(dir)
 	for _, fi := range fis {
 		if fi.IsDir() {
 			for k, v := range fileContents(filepath.Join(dir, fi.Name())) {
@@ -439,7 +440,7 @@ func fileContents(dir string) map[string]string {
 			}
 		} else {
 			path := filepath.Join(dir, fi.Name())
-			cnt, _ := os.ReadFile(path)
+			cnt, _ := ioutil.ReadFile(path)
 			m[filepath.ToSlash(path)] = string(cnt)
 		}
 	}