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.

Sign up for GitHub

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

Jump to bottom

gdaldem: fix help message for subcommands #11224

Merged
merged 1 commit into from
Nov 9, 2024
Merged

gdaldem: fix help message for subcommands #11224

merged 1 commit into from
Nov 9, 2024

Conversation

rouault
Copy link
Member

@rouault rouault commented Nov 9, 2024

Now:

$ gdaldem TPI --help
Usage: gdaldem TPI [--help] [--long-usage] [--help-general]
                   [-of <output_format>] [-compute_edges] [-b <value>] [-co <NAME>=<VALUE>]... [--quiet]
                   input_dem output_TPI_map

Note: gdaldem TPI --long-usage for full help.

vs before:

$ gdaldem TPI --help
Usage: TPI [--help] [--long-usage] [--help-general]
           [-of <output_format>] [-compute_edges] [-b <value>] [-co <NAME>=<VALUE>]... [--quiet]
           input_dem output_TPI_map

Note: TPI --long-usage for full help.

argparse.hpp stream fix submitted to upstream in p-ranav/argparse#382

@rouault
gdaldem: fix help message for subcommands
6966463 
Now:
```
$ gdaldem TPI --help
Usage: gdaldem TPI [--help] [--long-usage] [--help-general]
                   [-of <output_format>] [-compute_edges] [-b <value>] [-co <NAME>=<VALUE>]... [--quiet]
                   input_dem output_TPI_map

Note: gdaldem TPI --long-usage for full help.
```

vs before:
```
$ gdaldem TPI --help
Usage: TPI [--help] [--long-usage] [--help-general]
           [-of <output_format>] [-compute_edges] [-b <value>] [-co <NAME>=<VALUE>]... [--quiet]
           input_dem output_TPI_map

Note: TPI --long-usage for full help.
```

argparse.hpp stream fix submitted to upstream in p-ranav/argparse#382
@rouault rouault added backport release/3.10 Backport to release/3.10 branch funded through GSP Work funded through the GDAL Sponsorship Program labels Nov 9, 2024
@coveralls
Copy link
Collaborator

Coverage Status

coverage: 73.628%. remained the same
when pulling 6966463 on rouault:gdaldem_usage_fix
into 0958e1b on OSGeo:master.

@rouault rouault merged commit 88295af into OSGeo:master Nov 9, 2024
37 of 38 checks passed
@rouault rouault mentioned this pull request Nov 9, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backport release/3.10 Backport to release/3.10 branch funded through GSP Work funded through the GDAL Sponsorship Program
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rouault @coveralls