Skip to content
This repository has been archived by the owner on May 4, 2021. It is now read-only.

Ticket26926: man pages #240

Merged
merged 8 commits into from Aug 1, 2018
Merged

Ticket26926: man pages #240

merged 8 commits into from Aug 1, 2018

Conversation

juga0
Copy link
Contributor

@juga0 juga0 commented Jul 25, 2018

No description provided.

@juga0 juga0 added the enhancement New feature or request label Jul 25, 2018
which is actually the most important for the user
@pastly pastly changed the title Ticket26926 Ticket26926: man pages Aug 1, 2018
@pastly
Copy link
Member

pastly commented Aug 1, 2018

Is there any way we can auto-generate one (or both) of these? I don't want to have to remember to update them when we add a new config option or argument.

In anticipation of the answer being "no, not really", I'm merging this. Thanks @juga0.

@pastly pastly merged commit 137cc27 into torproject:master Aug 1, 2018
@juga0
Copy link
Contributor Author

juga0 commented Aug 3, 2018

Is there any way we can auto-generate one (or both) of these?

which "both" do you mean?

I don't want to have to remember to update them when we add a new config option or argument. In anticipation of the answer being "no, not really",

Do you mean to update the man for the config file?. Yeah, "not really".
The man for the binary could be created with the --help, but then it would not have the other typical sections.

@pastly
Copy link
Member

pastly commented Aug 3, 2018

Is there any way we can auto-generate one (or both) of these?

"both" means the both man_sbws.rst and man_sbws_ini.rst (or whatever their exact file names are)

The man for the config file is what I'm mainly thinking about. There's a lot of options. I imagine I could write a parsing script that could take the following content from config.default.ini and turn it into man page content, but that's potentially a lot of work and we have better things to be doing.

[general]
# TYPE: int
# DESCRIPTION: Days into the past that measurements are considered valid.
data_period = 5

Would become

general
  data_period = INT
    Days into the past that measurements are considered valid. (Default: 5)

@juga0
Copy link
Contributor Author

juga0 commented Aug 4, 2018

I imagine I could write a parsing script that could take the following content from config.default.ini and turn it into man page content, but that's potentially a lot of work and we have better things to be doing.

yeah, let's just change it manually for now. When sbws will be stable, i don't think we'll need to change the config much. We can still create a new ticket to do this in the future.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants