From 6d74699580cd1d25e97e129a6e796bf324298663 Mon Sep 17 00:00:00 2001 From: Jeff Carr Date: Sun, 26 Oct 2025 10:17:37 -0500 Subject: argv is better now --- argv.custom.go | 23 ---------------- argv.go | 36 +++++++++++++++++++++++++ argv.struct.go | 36 ------------------------- argv.template.go | 81 -------------------------------------------------------- complete.go | 37 ++++++++++++++++++++++++++ http.go | 3 ++- main.go | 2 +- structs.go | 2 -- 8 files changed, 76 insertions(+), 144 deletions(-) delete mode 100644 argv.custom.go create mode 100644 argv.go delete mode 100644 argv.struct.go delete mode 100644 argv.template.go create mode 100644 complete.go diff --git a/argv.custom.go b/argv.custom.go deleted file mode 100644 index ed6b731..0000000 --- a/argv.custom.go +++ /dev/null @@ -1,23 +0,0 @@ -package main - -import ( - "os" - - "go.wit.com/lib/protobuf/argvpb" -) - -// are sent via -ldflags at buildtime -var VERSION string -var BUILDTIME string -var APPNAME string = "forged" - -// sends the strings to bash or zsh that will be your options -func (a args) SendCompletionStrings(pb *argvpb.Argv) { - if pb.GetCmd() == "" { - base := []string{"--bash", "repos", "gui", "patches", "--daemon", "missing"} - pb.SendStrings(base) - } else { - pb.SubCommand(pb.Goargs...) - } - os.Exit(0) -} diff --git a/argv.go b/argv.go new file mode 100644 index 0000000..ac67ed8 --- /dev/null +++ b/argv.go @@ -0,0 +1,36 @@ +package main + +var argv args + +type args struct { + Repos *ReposCmd `arg:"subcommand:repos" help:"the forged repos"` + Patch *PatchCmd `arg:"subcommand:patches" help:"the forged patches"` + Gui *EmptyCmd `arg:"subcommand:gui" help:"show gui"` + Missing *EmptyCmd `arg:"subcommand:missing" help:"list missing repos"` + Port int `arg:"--port" default:"2520" help:"port to run on"` + Hostname string `arg:"--hostname" help:"hostname to use"` + Daemon bool `arg:"--daemon" help:"run as a daemon"` + Force bool `arg:"--force" help:"try to strong arm things"` + Verbose bool `arg:"--verbose" help:"show more output"` +} + +type EmptyCmd struct { +} + +type PatchCmd struct { + List *EmptyCmd `arg:"subcommand:list" help:"list the patches"` + Clean *EmptyCmd `arg:"subcommand:clean" help:"clean the patches"` + Init *EmptyCmd `arg:"subcommand:init" help:"init"` +} + +type ReposCmd struct { + List *EmptyCmd `arg:"subcommand:list" help:"list the repos"` + Pull *EmptyCmd `arg:"subcommand:pull" help:"'git pull' on the repos"` + Scan *EmptyCmd `arg:"subcommand:scan" help:"scan the forged repo dir"` + Fix *EmptyCmd `arg:"subcommand:fix" help:"try to fix the repo PB"` + Devel *EmptyCmd `arg:"subcommand:devel" help:"list repos with devel branches"` + Reload *EmptyCmd `arg:"subcommand:reload" help:"gitpb.Reload() each repo"` + // Reload *EmptyCmd `arg:"subcommand:reload" help:"do Reload() on each git repo"` + // Clean *EmptyCmd `arg:"subcommand:clean" help:"clean the repos"` + // Merge *EmptyCmd `arg:"subcommand:merge" help:"merge in new patchsets"` +} diff --git a/argv.struct.go b/argv.struct.go deleted file mode 100644 index ac67ed8..0000000 --- a/argv.struct.go +++ /dev/null @@ -1,36 +0,0 @@ -package main - -var argv args - -type args struct { - Repos *ReposCmd `arg:"subcommand:repos" help:"the forged repos"` - Patch *PatchCmd `arg:"subcommand:patches" help:"the forged patches"` - Gui *EmptyCmd `arg:"subcommand:gui" help:"show gui"` - Missing *EmptyCmd `arg:"subcommand:missing" help:"list missing repos"` - Port int `arg:"--port" default:"2520" help:"port to run on"` - Hostname string `arg:"--hostname" help:"hostname to use"` - Daemon bool `arg:"--daemon" help:"run as a daemon"` - Force bool `arg:"--force" help:"try to strong arm things"` - Verbose bool `arg:"--verbose" help:"show more output"` -} - -type EmptyCmd struct { -} - -type PatchCmd struct { - List *EmptyCmd `arg:"subcommand:list" help:"list the patches"` - Clean *EmptyCmd `arg:"subcommand:clean" help:"clean the patches"` - Init *EmptyCmd `arg:"subcommand:init" help:"init"` -} - -type ReposCmd struct { - List *EmptyCmd `arg:"subcommand:list" help:"list the repos"` - Pull *EmptyCmd `arg:"subcommand:pull" help:"'git pull' on the repos"` - Scan *EmptyCmd `arg:"subcommand:scan" help:"scan the forged repo dir"` - Fix *EmptyCmd `arg:"subcommand:fix" help:"try to fix the repo PB"` - Devel *EmptyCmd `arg:"subcommand:devel" help:"list repos with devel branches"` - Reload *EmptyCmd `arg:"subcommand:reload" help:"gitpb.Reload() each repo"` - // Reload *EmptyCmd `arg:"subcommand:reload" help:"do Reload() on each git repo"` - // Clean *EmptyCmd `arg:"subcommand:clean" help:"clean the repos"` - // Merge *EmptyCmd `arg:"subcommand:merge" help:"merge in new patchsets"` -} diff --git a/argv.template.go b/argv.template.go deleted file mode 100644 index 6948afe..0000000 --- a/argv.template.go +++ /dev/null @@ -1,81 +0,0 @@ -package main - -// these are stubbed in functions needed -// just copy this file from another working app for now -// you shouldn't need to change anything here -// TODO: clean this up in argv - -import ( - "os" - - "go.wit.com/dev/alexflint/arg" - "go.wit.com/gui" - "go.wit.com/lib/env" - "go.wit.com/lib/fhelp" - "go.wit.com/log" -) - -func (args) InitArgv() (string, string, string) { - return APPNAME, BUILDTIME, VERSION -} - -// this function will send the current argv PB to go-args for parsing -func (args) ParseFlags(flags []string) error { - var err error - if me.pp == nil { - // log.Info("Parse Flags GOT flags:", flags) - me.pp, err = arg.ParseFlags(flags, &argv) - // panic("got to the app's ParseFlags()") - } else { - panic("me.pp was not nil") - } - return err -} - -// add this funcgion: this will print the help -func (args) WriteHelpForSubcommand(cmd string) error { - me.pp.WriteHelpForSubcommand(os.Stderr, cmd) - return nil -} - -// this will print the help for the subcmd -func (args) WriteHelpForAutocomplete(part string, subcmd ...string) error { - return me.pp.WriteHelpForAutocomplete(os.Stderr, os.Stdout, part, subcmd...) -} - -func (args) WriteHelpForAutocompleteDebug(part string, subcmd ...string) error { - f, _ := os.OpenFile("/tmp/argv.junk", os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0644) - return me.pp.WriteHelpForAutocomplete(f, os.Stdout, part, subcmd...) -} - -// add this funcgion: this will print the help -func (args) WriteHelp() error { - me.pp.WriteHelp(os.Stderr) - return nil -} - -func (args) InitGui() error { - // panic("got here") - arg.Register(&gui.ArgvGui) - // me.gui = gui.PreInit() - me.myGui = fhelp.Gui() - return nil -} - -func (args) Exit() { - gui.UnloadToolkits() - if env.Verbose() { - log.Info("argv.Exit() called", APPNAME+".Exit()") - } - // remove this from the template for your app (or make one for youself if you need it) - // forgeExit() // custom forge shutdown function -} - -func (args) Help() string { - return "got app help" -} - -func (args) MustParse() error { - me.pp = arg.MustParse(&argv) - return nil -} diff --git a/complete.go b/complete.go new file mode 100644 index 0000000..178f57d --- /dev/null +++ b/complete.go @@ -0,0 +1,37 @@ +package main + +import ( + "fmt" + "strings" + + "go.wit.com/dev/alexflint/arg" + "go.wit.com/lib/protobuf/argvpb" +) + +// are sent via -ldflags at buildtime +var VERSION string +var BUILDTIME string +var APPNAME string = "forged" + +func (args) MustParse() error { + me.pp = arg.MustParse(&argv) + return nil +} + +// sends the strings to bash or zsh that will be your options +func (a args) DoAutoComplete() error { + if argvpb.PB.GetCmd() == "" { + matches := []string{"--bash", "repos", "gui", "patches", "--daemon", "missing"} + fmt.Fprintf(argvpb.Stdout, " %s", strings.Join(matches, " ")) + return nil + } + var err error + if me.pp == nil { + me.pp, err = arg.ParseFlagsArgv(&argv) + if err != nil { + return err + } + } + err = me.pp.WriteHelpForAutocomplete(argvpb.PB.Partial, argvpb.PB.Real...) + return err +} diff --git a/http.go b/http.go index e965061..436d118 100644 --- a/http.go +++ b/http.go @@ -5,6 +5,7 @@ import ( "net/http" "strings" + "go.wit.com/lib/protobuf/argvpb" "go.wit.com/lib/protobuf/forgepb" "go.wit.com/lib/protobuf/gitpb" "go.wit.com/lib/protobuf/httppb" @@ -30,7 +31,7 @@ func okHandler(w http.ResponseWriter, r *http.Request) { if route == "/" { w.Header().Set("Content-Type", "text") - fmt.Fprintf(w, "go.wit.com/apps/utils/forged Version: %s\n", me.argv.Version()) + fmt.Fprintf(w, "go.wit.com/apps/utils/forged Version: %s\n", argvpb.Version()) fmt.Fprintf(w, "\n") return } diff --git a/main.go b/main.go index 1cd490a..3cf1d1b 100644 --- a/main.go +++ b/main.go @@ -31,7 +31,7 @@ func saveMissing() error { func main() { me = new(mainType) - me.argv = argvpb.Autocomplete(&argv) // adds shell auto complete to go-args + argvpb.Init(&argv, APPNAME, BUILDTIME, VERSION) // adds shell auto-complete var err error me.forge, err = forgepb.InitByFullpath("/etc/forged/forge.text") diff --git a/structs.go b/structs.go index 8de8f18..b4f3bca 100644 --- a/structs.go +++ b/structs.go @@ -6,7 +6,6 @@ package main import ( "go.wit.com/dev/alexflint/arg" "go.wit.com/lib/fhelp" - "go.wit.com/lib/protobuf/argvpb" "go.wit.com/lib/protobuf/forgepb" "go.wit.com/lib/protobuf/gitpb" ) @@ -15,7 +14,6 @@ var me *mainType // this app's variables type mainType struct { - argv *argvpb.Argv // more experiments for bash handling pp *arg.Parser // for parsing the command line args. Yay to alexf lint! myGui *fhelp.GuiPrep // for initializing the GUI toolkits forge *forgepb.Forge // your customized repo preferences and settings -- cgit v1.2.3