Changing handling on all-modules.go and the 'go generate' feature (#355)

# Description

When forking the repository, and running "go generate" modules/all-modules.go is created or updated, but since it was previously tracking in the repository it's then added to unstaged files for commit. This is not optimal, as it could potentially mean someone would push their custom all-modules.go file to the main repository.

It's already in .gitignore, but since it was already tracked this is not enough to have ti not appear as a changed file when running "go generate".

## Changes

- Added a default modules/all-modules.go to the repo
- Removed all-modules.go from .gitignore
- Removed references to 'go generate' in the Makefile
- Removed cmd/generate/_all-modules.go
- Added proper sorting of modules in modules-import.go

Author: MorquinDevlar

.gitignore

@@ -5,7 +5,6 @@ _datafiles/**/plugin-data/*
 **/config-overrides.yaml
 **/.roundcount
 vendor/
-modules/all-modules.go
 server.crt
 server.key
 tmp-gomud.exe

Makefile

@@ -36,29 +36,21 @@ docker-%:
 .PHONY: build_rpi_zero2w
 build_rpi_zero2w: generate ### Build a binary for a raspberry pi zero 2w
 	env GOOS=linux GOARCH=arm64 go build -o $(BIN)-rpi
-	make ungenerate
 
 .PHONY: build_win64
 build_win64: generate ### Build a binary for 64bit windows
 	env GOOS=windows GOARCH=amd64 go build -o $(BIN)-win64.exe
-	make ungenerate
 
 .PHONY: build_linux64
 build_linux64: generate ### Build a binary for linux
 	env GOOS=linux GOARCH=amd64 go build -o $(BIN)-linux64
-	make ungenerate
 
 .PHONY: build
 build: validate build_local  ### Validate the code and build the binary.
 
 .PHONY: build_local
 build_local: generate
 	CGO_ENABLED=0 go build -trimpath -a -o $(BIN)
-	make ungenerate
-
-.PHONY: ungenerate
-ungenerate: ### Reverts to a clean version of modules/all-modules.go
-	@cp cmd/generate/_all-modules.go modules/all-modules.go
 
 .PHONY: generate
 generate: ### Generates include directives for modules

cmd/generate/_all-modules.go

@@ -15,6 +15,7 @@ import (
 	"log"
 	"os"
 	"path/filepath"
+	"sort"
 	"strings"
 )
 
@@ -44,6 +45,7 @@ func main() {
 			}
 		}
 	}
+	sort.Strings(pkgs)
 	var buf bytes.Buffer
 	buf.WriteString("// Code generated by go:generate; DO NOT EDIT.\n\n")
 	buf.WriteString("// Generated with command: go generate\n")