Merge pull request #15 from metalmatze/init-warning

Check for existing jsonnetfile.json before init
This commit is contained in:
Frederic Branczyk 2018-08-08 10:41:23 +02:00 committed by GitHub
commit 49e110fd83
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 16 additions and 7 deletions

View file

@ -23,7 +23,7 @@ build:
install: build
@$(eval OUTPUT=$(OUT_DIR)/$(GOOS)/$(GOARCH)/$(BIN))
@echo ">> copying $(BIN) into $(GOPATH)/$(BIN)"
@echo ">> copying $(BIN) into $(GOPATH)/bin/$(BIN)"
@cp $(OUTPUT) $(GOPATH)/bin/$(BIN)
test:

View file

@ -101,9 +101,19 @@ func Main() int {
}
func initCommand() int {
err := ioutil.WriteFile(pkg.JsonnetFile, []byte("{}"), 0644)
exists, err := pkg.FileExists(pkg.JsonnetFile)
if err != nil {
kingpin.Fatalf("Failed to write new jsonnetfile.json: %v", err)
kingpin.Errorf("Failed to check for jsonnetfile.json: %v", err)
return 1
}
if exists {
kingpin.Errorf("jsonnetfile.json already exists")
return 1
}
if err := ioutil.WriteFile(pkg.JsonnetFile, []byte("{}"), 0644); err != nil {
kingpin.Errorf("Failed to write new jsonnetfile.json: %v", err)
return 1
}

View file

@ -147,9 +147,8 @@ func insertDependency(deps []spec.Dependency, newDep spec.Dependency) ([]spec.De
return res, nil
}
func LockExists(dir string) (bool, error) {
lockfile := path.Join(dir, JsonnetLockFile)
_, err := os.Stat(lockfile)
func FileExists(path string) (bool, error) {
_, err := os.Stat(path)
if os.IsNotExist(err) {
return false, nil
}
@ -166,7 +165,7 @@ func ChooseJsonnetFile(dir string) (string, bool, error) {
filename := lockfile
isLock := true
lockExists, err := LockExists(dir)
lockExists, err := FileExists(filepath.Join(dir, JsonnetLockFile))
if err != nil {
return "", false, err
}