Modify ls to use our internal structure

Makefile

@@ -11,7 +11,7 @@ COVER_DIR=cover
 
 # Package lists
 TOPLEVEL_PKG := jsh
-CMD_LIST := jsh/ps jsh/free jsh/ls
+CMD_LIST := jsh/ps jsh/free jsh/ls jsh/cat jsh/df
 
 # List building
 ALL_LIST = $(TOPLEVEL_PKG) $(CMD_LIST)

files.go

@@ -0,0 +1,14 @@
+package jsh
+
+type File struct {
+	Name  string
+	Size  int
+	Mode  int
+	Inode uint64
+}
+
+type Permission struct {
+	Read    bool
+	Write   bool
+	Execute bool
+}

ls/main.go

@@ -4,72 +4,58 @@ import (
 	"flag"
 	"fmt"
 	"io/ioutil"
+	"jsh"
 	"log"
-	"strconv"
 	"syscall"
 )
 
-func get_fileinfo(f string, size bool, mode bool, inode bool) string {
+// func get_fileinfo(f string) string {
-	var stat syscall.Stat_t
-	var ret string
-	ret = "{\"files\": [{\"name\":\"" + f + "\""
-	if err := syscall.Stat(f, &stat); err != nil {
-		log.Fatal(err)
-	}
-	if size {
-		ret = ret + ", \"size\":" + strconv.FormatInt(stat.Size, 10)
-	}
-	if mode {
-		ret = ret + ", \"mode\":" + strconv.Itoa(int(stat.Mode))
-	}
-	if inode {
-		ret = ret + ", \"inode\":" + strconv.FormatUint(stat.Ino, 10)
-	}
-	ret = ret + "}]}"
-	return ret
-}
 
-func main() {
+// 	ret = ret + ", \"size\":" + strconv.FormatInt(stat.Size, 10)
-	// here be the ls flags
+// 	ret = ret + ", \"mode\":" + strconv.Itoa(int(stat.Mode))
-
+// 	ret = ret + ", \"inode\":" + strconv.FormatUint(stat.Ino, 10)
-	var a_flag bool     // all files, even ones starting with .
+// 	ret = ret + "}]}"
-	var mode_flag bool  // flags
+// 	return ret
-	var inode_flag bool // inode
+// }
-	var size_flag bool  // size
-	var first = true
-
-	flag.BoolVar(&a_flag, "a", false, "lists all files in directory, even hidden ones")
-	flag.BoolVar(&mode_flag, "f", false, "include flags for file")
-	flag.BoolVar(&inode_flag, "i", false, "include flags for file")
-	flag.BoolVar(&size_flag, "s", false, "include flags for file")
-	// end ls flag
 
+func listFiles(queue chan *jsh.JshFrame) {
 	flag.Parse()
 	root := "." //flag.Arg(0)
 	dir, _ := ioutil.ReadDir(root)
-
+	for _, entry := range dir {
-	fmt.Printf("[{\"StdOut\": [\n")
+		var stat syscall.Stat_t
-
+		if err := syscall.Stat(entry.Name(), &stat); err != nil {
-	if !a_flag {
+			log.Fatal(err)
-		for _, entry := range dir {
-			if entry.Name()[0] != '.' {
-				if !first {
-					fmt.Printf(",")
-				} else {
-					first = false
-				}
-				fmt.Printf("%s\n", get_fileinfo(entry.Name(), size_flag, mode_flag, inode_flag))
-			}
 		}
-	} else {
+		if entry.Name()[0] != '.' {
-		for _, entry := range dir {
+			output := jsh.File{}
-			if !first {
+			output.Name = entry.Name()
-				fmt.Printf(",")
+			output.Size = int(stat.Size)
-			} else {
+			output.Mode = int(stat.Mode)
-				first = false
+			output.Inode = stat.Ino
-			}
+			frame := jsh.JshFrame{output, ""}
-			fmt.Printf("%s\n", get_fileinfo(entry.Name(), size_flag, mode_flag, inode_flag))
+			queue <- &frame
 		}
 	}
-	fmt.Printf("], \"StdErr\": []}]\n")
+}
+
+func runJsonMode() {
+	queue := make(chan *jsh.JshFrame)
+	done := make(chan bool)
+	go jsh.OutputFrames(queue, done)
+	listFiles(queue)
+	close(queue)
+	<-done
+}
+
+func main() {
+	// TODO: Support more flags
+	jsonModePtr := flag.Bool("json", false, "whether to use json mode for input and output")
+	flag.Parse()
+
+	if !*jsonModePtr {
+		fmt.Printf("%s", jsh.FallbackWithArgs("ls", flag.Args()))
+	} else {
+		runJsonMode()
+	}
 }