Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation,cmd/dlv: clean up command line usage help #3395

Merged
merged 1 commit into from
Aug 9, 2023

Commits on Aug 1, 2023

  1. Documentation,cmd/dlv: clean up command line usage help

    Due to some very old mistakes too many of Delve's flags are declared as
    persistent on cobra's root command. For example the headless flag is a
    global flag but does not apply to connect, dap or trace; the backend
    flag does not apply to replay, core and dap; etc.
    
    Almost all global flags should have been declared as local flags on
    individual subcommands. Unfortunately we can not change this without
    breaking backwards compatibility, for example:
    
       dlv --headless debug
    
    would not parse if headless was a flag of debug instead of a global
    flag.
    
    Instead we alter usage function and the markdown generation script to
    strategically hide the flags that don't apply.
    
    Fixes go-delve#2361
    aarzilli committed Aug 1, 2023
    Configuration menu
    Copy the full SHA
    932bc80 View commit details
    Browse the repository at this point in the history