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

Update gpsbabel_man.xml #1170

Merged
merged 1 commit into from
Sep 3, 2023
Merged

Update gpsbabel_man.xml #1170

merged 1 commit into from
Sep 3, 2023

Conversation

jidanni
Copy link
Contributor

@jidanni jidanni commented Sep 3, 2023

Without a dash this will resemble the "tar" command!

Oops... it seems I made a side patch... but I don't know how to delete it.

Without a dash this will resemble the "tar" command!
Copy link
Collaborator

@robertlipe robertlipe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://tdg.docbook.org/tdg/4.5/option.html#:~:text=Processing%20expectations&text=DocBook%20does%20not%20specify%20whether,Appendix%20F%2C%20Interchanging%20DocBook%20Documents.

Current approach is valid and, IMO, preferable as '-' is not an option.

Do you have a more authoritative source than DocBook's own "eh, whatever" answer?

@jidanni
Copy link
Contributor Author

jidanni commented Sep 3, 2023

All I know is the only man pages that don't use a dash are:

SYNOPSIS
       gpsbabel [iofFxsh] [file...]

SYNOPSIS
   Traditional usage
       tar {A|c|d|r|t|u|x}[GnSkUWOmpsMBiajJzZhPlRvwo] [ARG...]

Also there is still a dash here:

Usage:
    gpsbabel [options] -i INTYPE -f INFILE [filter] -o OUTTYPE -F OUTFILE
    gpsbabel [options] -i INTYPE -o OUTTYPE INFILE [filter] OUTFILE

So I would still keep one on the man page.

@robertlipe robertlipe merged commit 29d1573 into GPSBabel:master Sep 3, 2023
20 checks passed
@robertlipe
Copy link
Collaborator

That's not very authoritative, but I'll take it.

Before you go on another nitpicking bender, I'll tell you my next step in the man page is to delete it. The other doc is SO much better that I'm just not spending time on the man page.

@jidanni jidanni deleted the patch-7 branch September 4, 2023 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants