fixed deprecation notices

Author: odino

Dockerfile

@@ -8,4 +8,4 @@ WORKDIR /abs
 RUN go install github.com/jteeuwen/go-bindata/...
 RUN go mod vendor
 
-CMD bash
+CMD ["bash"]

Makefile

@@ -6,14 +6,14 @@ fmt:
 build:
 	docker build -t abs .
 bench:
-	CONTEXT=abs go test `go list ./... | grep -v "/js"` -bench=.
+	CONTEXT=abs go test `go list -buildvcs=false ./... | grep -v "/js"` -bench=.
 test_all: bench test
 test:
 	# we don't want to test the JS package
-	CONTEXT=abs go test `go list ./... | grep -v "/js"`
+	CONTEXT=abs go test `go list -buildvcs=false ./... | grep -v "/js"`
 test_verbose:
 	# this will show successful error [line:col] tests per #38
-	CONTEXT=abs go test `go list ./... | grep -v "/js"` -v
+	CONTEXT=abs go test `go list -buildvcs=false ./... | grep -v "/js"` -v
 repl:
 	go run main.go
 build_simple:

evaluator/functions.go

@@ -6,7 +6,6 @@ import (
 	"encoding/csv"
 	"encoding/json"
 	"fmt"
-	"io/ioutil"
 	"math"
 	"math/big"
 	mrand "math/rand"
@@ -2153,7 +2152,7 @@ var packageAliasesLoaded bool
 
 func requireFn(tok token.Token, env *object.Environment, args ...object.Object) object.Object {
 	if !packageAliasesLoaded {
-		a, err := ioutil.ReadFile("./packages.abs.json")
+		a, err := os.ReadFile("./packages.abs.json")
 
 		// We couldn't open the packages, file, possibly doesn't exists
 		// and the code shouldn't fail
@@ -2225,7 +2224,7 @@ func doSource(tok token.Token, env *object.Environment, fileName string, args ..
 		code, error = Asset("stdlib/" + fileName[1:])
 	} else {
 		// load the source file
-		code, error = ioutil.ReadFile(fileName)
+		code, error = os.ReadFile(fileName)
 	}
 
 	if error != nil {

install/install.go

@@ -5,7 +5,6 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
-	"io/ioutil"
 	"net/http"
 	"os"
 	"path/filepath"
@@ -200,7 +199,12 @@ func createAlias(module string) (string, error) {
 	}
 	defer f.Close()
 
-	b, err := ioutil.ReadAll(f)
+	b, err := io.ReadAll(f)
+
+	if err != nil {
+		fmt.Printf("Could not read alias file %s\n", err)
+		return "", err
+	}
 
 	data := make(map[string]string)
 	moduleName := filepath.Base(module)

repl/repl.go

@@ -3,7 +3,6 @@ package repl
 import (
 	"bytes"
 	"fmt"
-	"io/ioutil"
 	"log"
 	"os"
 	"path/filepath"
@@ -34,7 +33,7 @@ func getAbsInitFile(env *object.Environment) {
 	}
 	initFile = filePath
 	// read and eval the abs init file
-	code, err := ioutil.ReadFile(initFile)
+	code, err := os.ReadFile(initFile)
 	if err != nil {
 		// abs init file is optional -- nothing to do here
 		return
@@ -137,7 +136,7 @@ func BeginRepl(args []string, version string) {
 
 	// this is a script
 	// let's parse our argument as a file and run it
-	code, err := ioutil.ReadFile(args[1])
+	code, err := os.ReadFile(args[1])
 	if err != nil {
 		fmt.Fprintln(env.Stdio.Stdout, err.Error())
 		os.Exit(99)

terminal/history.go

@@ -72,7 +72,7 @@ func getHistory(historyFile string, maxLines int) []string {
 	}
 	fd.Close()
 	// read the file and split the lines into history[...]
-	bytes, err := ioutil.ReadFile(historyFile)
+	bytes, err := os.ReadFile(historyFile)
 	if err != nil {
 		return history
 	}

terminal/terminal.go

@@ -22,7 +22,6 @@ import (
 // autocompleter
 // navigate commands up
 // maybe only save incrementally in history https://stackoverflow.com/questions/7151261/append-to-a-file-in-go
-// remove deprecated ioutil methods
 // remove dependencies
 // worth renaming repl to runner? and maybe terminal back to repl
 // add prompt formatting tests

util/check_update.go

@@ -1,7 +1,7 @@
 package util
 
 import (
-	"io/ioutil"
+	"io"
 	"net/http"
 	"strings"
 )
@@ -16,7 +16,7 @@ func UpdateAvailable(version string) (string, bool) {
 	}
 	defer resp.Body.Close()
 
-	body, err := ioutil.ReadAll(resp.Body)
+	body, err := io.ReadAll(resp.Body)
 	if err != nil {
 		return version, false
 	}