amateurfunk/hamnet70
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

Visualizer #3

Merged
thomas merged 12 commits from rudi_s into main 2024-05-28 11:21:00 +02:00
Conversation 3 Commits 12 Files changed 12 +22 -12
1 changed files with 22 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit d5aeb6c0bc - Show all commits

34
impl/utils/visualizer/main.go
View file

@ -24,9 +24,9 @@ import (
var static embed.FS var static embed.FS
func main() { func main() {
if len(os.Args) != 3 { if len(os.Args) != 2 && len(os.Args) != 3 {
log.SetFlags(0) log.SetFlags(0)
log.Fatalf("usage: %s <fifo> <json-dup-prefix>", os.Args[0]) log.Fatalf("usage: %s <fifo> [<json-dup-prefix>]", os.Args[0])
} }
var mutex sync.Mutex var mutex sync.Mutex
@ -35,15 +35,21 @@ func main() {
go func() { go func() {
for { for {
path := os.Args[1] path := os.Args[1]
dupPath := os.Args[2] + "." + time.Now().Format(time.RFC3339) var dupPath string
thomas marked this conversation as resolved Outdated
thomas commented 2024-05-25 17:39:14 +02:00
Outdated
Review
Copy link

Eine Idee dazu noch: wenn man wirklich ein Replay von einem früheren JSON-Dump startet, indem man diesen statt dem FIFO als Args[1] angibt, wäre es cool, wenn man Args[2] weglassen könnte oder zumindest /dev/null angeben könnte. Letzteres geht leider nicht wegen dem Anhängen der Uhrzeit (was an sich aber schon sinnvoll ist).

Eine Idee dazu noch: wenn man wirklich ein Replay von einem früheren JSON-Dump startet, indem man diesen statt dem FIFO als Args[1] angibt, wäre es cool, wenn man Args[2] weglassen könnte oder zumindest /dev/null angeben könnte. Letzteres geht leider nicht wegen dem Anhängen der Uhrzeit (was an sich aber schon sinnvoll ist).
rudi_s commented 2024-05-28 09:02:36 +02:00
Outdated
Review
Copy link

Ist implementiert. Habs auch gleich rebased.

Ist implementiert. Habs auch gleich rebased.
thomas commented 2024-05-28 11:19:14 +02:00
Outdated
Review
Copy link

Schaut gut aus. Dann merge ich das mal :)

Schaut gut aus. Dann merge ich das mal :)
if len(os.Args) == 3 {
dupPath = os.Args[2] + "." + time.Now().Format(time.RFC3339)
}
fh, err := os.Open(path) fh, err := os.Open(path)
if err != nil { if err != nil {
log.Fatal(err) log.Fatal(err)
} }
dupFh, err := os.Create(dupPath) var dupFh *os.File
if err != nil { if dupPath != "" {
log.Fatal(err) dupFh, err = os.Create(dupPath)
if err != nil {
log.Fatal(err)
}
} }
r := bufio.NewReader(fh) r := bufio.NewReader(fh)
@ -56,9 +62,11 @@ func main() {
// Duplicate JSON input to a file so it can be replayed later // Duplicate JSON input to a file so it can be replayed later
// if necessary // if necessary
_, err = dupFh.WriteString(line) if dupFh != nil {
if err != nil { _, err = dupFh.WriteString(line)
log.Fatalf("write %q: %v", dupPath, err) if err != nil {
log.Fatalf("write %q: %v", dupPath, err)
}
} }
// Send to all listeners // Send to all listeners
@ -73,9 +81,11 @@ func main() {
if err != nil { if err != nil {
log.Fatalf("close %q: %v", path, err) log.Fatalf("close %q: %v", path, err)
} }
err = dupFh.Close() if dupFh != nil {
if err != nil { err = dupFh.Close()
log.Fatalf("close %q: %v", dupPath, err) if err != nil {
log.Fatalf("close %q: %v", dupPath, err)
}
} }
} }
}() }()