Skip to content

docs: sync dlt-adaptor-stdin man page with actual CLI options#841

Open
aki1770-del wants to merge 1 commit into
COVESA:masterfrom
aki1770-del:docs/adaptor-stdin-manpage-813
Open

docs: sync dlt-adaptor-stdin man page with actual CLI options#841
aki1770-del wants to merge 1 commit into
COVESA:masterfrom
aki1770-del:docs/adaptor-stdin-manpage-813

Conversation

@aki1770-del
Copy link
Copy Markdown

Problem

The dlt-adaptor-stdin.1.md man page was out of sync with the implementation in src/adaptor/dlt-adaptor-stdin.c in several ways (as reported in #813):

Issue Details
-s option in SYNOPSIS Doesn't exist — getopt string is "a:c:bht:v:", no -s
-v level not documented Fully implemented: sets verbosity to FATAL/ERROR/WARN/INFO/DEBUG/VERBOSE
EXAMPLES uses -b -s The non-existent -s is used in the dmesg example
Typo in -c description "Set context ID tp ctid" should be "to"

Fix

  • Remove -s from SYNOPSIS
  • Add -v level to SYNOPSIS and OPTIONS
  • Remove -s from EXAMPLES (dmesg pipe example)
  • Fix "tp" → "to" typo in -c option description
  • Clarify -b description wording

Fixes #813

The man page was out of sync with the implementation in
src/adaptor/dlt-adaptor-stdin.c:

- SYNOPSIS listed -s which does not exist in getopt string
- -v level option (verbosity: FATAL/ERROR/WARN/INFO/DEBUG/VERBOSE)
  was implemented but not documented
- EXAMPLES used '-b -s' — the non-existent -s is removed
- Minor: 'context ID tp ctid' typo fixed to 'to'
- Minor: -b description clarified

Fixes COVESA#813
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.

dlt-adaptor-stdin man page out of sync

1 participant