Skip to content
This repository has been archived by the owner. 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

Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@juga0
Copy link
Contributor

@juga0 juga0 commented Jul 25, 2018

No description provided.

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 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
1 check passed
@juga0
Copy link
Contributor Author

@juga0 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 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 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.