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

Updating fcpgenkey, fcpinvertkey man pages to 0.2.5 #10

Merged
merged 8 commits into from
Feb 22, 2015

Conversation

Soliloque
Copy link
Contributor

Then a few cosmetic changes and modifying other man pages to keep those uniform.

Removed html version of man pages as those are

  1. redondant
  2. obsolete
  3. automaticaly generated from the other man pages version.

Valided original author's email adress.

@Soliloque Soliloque changed the title Updating fcpgenkey man pages from 0.1.4 to 0.2.5 Updating fcpgenkey, fcpinvertkey man pages to 0.2.5 Feb 20, 2015
@Thynix
Copy link
Contributor

Thynix commented Feb 21, 2015

Ooh, getting rid of those HTML versions will be nice. Some of these commit messages do not meet the commit message standards. Could you please update them?

@Soliloque
Copy link
Contributor Author

Done.

@@ -2,5 +2,5 @@ The PyFCP applications and library were written
by David McNab (aum) in May-June, 2006.

Contact the author at:
david@freenet.org.nz
david@rebirthing.co.nz
Copy link
Contributor

Choose a reason for hiding this comment

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

Where did this email come from?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

From the man pages. Then I checked http://rebirthing.co.nz in search of this email adress. Did not find it but find a picture of David McNab on the first page. I write him to be certain. No reply yet but I did not receive any error. So this email is most probably working.

Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if David still wants to be listed as maintainer for this codebase. I suspect not.

@Thynix
Copy link
Contributor

Thynix commented Feb 21, 2015

The way the author sections are written makes it sound like this was written in one go in 2006, and hasn't been changed since, which isn't the case.

Any ideas on how to avoid that?

@Soliloque
Copy link
Contributor Author

It was spelt wrong. ;)

@Thynix
Copy link
Contributor

Thynix commented Feb 21, 2015

I'm sorry, these still don't meet the commit message standards. Are they unclear? I'd like to clarify them - any suggestions?

For example, in pyFreenet man pages: Change SYNTAX->SYNOPSIS the context doesn't need to include "pyFreenet" because it is already in the pyFreenet repo. The standards suggest a context of the "area / module" if helpful, which could mean a class but in this case means the start of the path. Words are also preferable to symbols, so I'd recommend changing this one to manpages: rename SYNTAX section to SYNOPSIS.

Commit summaries (meaning the first line) should not end with a full stop.

The rest are not imperative - meaning commands to the codebase:

  • updating -> update
  • new -> add
  • Removed -> Remove
  • AUTHOR sections expanded -> expand AUTHOR sections

Thanks!

@Soliloque
Copy link
Contributor Author

English isn't my native language. The PyFCP applications and library were originaly written by David McNab (aum) in May-June, 2006 and are maintained by the Freenet Project team.

Replace the name of SYNTAX sections by the universally standard SYNOPSIS.
Update fcpgenkey's man page with current supported options
and apply some cosmetic changes.
@Thynix
Copy link
Contributor

Thynix commented Feb 21, 2015

Alright - sorry about that. Would you like me to merge once the content looks good and fix the commit messages where necessary?

That sounds good for the authors section. originaly -> originally. (I assume you plan to do the pyFCP -> pyFreenet later?)

Add a new REPORTING BUGS section to all man pages
with information from the BUGS file in lib-pyFreenet's
root directory.
@Soliloque
Copy link
Contributor Author

Answering to a lot of things at once.

  • I'm fixing everything right now. I don't want to give you too much job.
  • As I wrote him, I suppose McNab will tell me if he doesn't want to be a maintainer anymore. I'll remove him if needed.
  • I don't want to replace pyFCP by pyFreenet only on a few files and further add to the imprecision.

Expand AUTHORS sections in all man pages with information
from the AUTHORS file in lib-pyFreenet root directory.
Correct the original author's email with the one he provided.
Update fcpinvertkey's man page with the options supported by the
current version of the software. Make a few cosmetic change.
Update the email of the original author with the one he instructed
me to use. Cite the current maintainers.
@Soliloque
Copy link
Contributor Author

I think it begins to look good. Please notice the last commit and the ones about AUTHORS and email.

.LP
USK@ are really just a user-friendly wrapper around SSK@, which hide
the process of searching for more recent versions of a site. Any
address written using an SSK@ key can also be written using an USK@
Copy link
Contributor

Choose a reason for hiding this comment

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

This way isn't strictly correct, but the other way is. Any USK can be written as an SSK, but not every SSK can be written as a USK.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree. I'll reverse it.

.TP
.B
Syntax of SSK@ key:
SSK@<key>/<name_of_site>-<version#>/
Copy link
Contributor

Choose a reason for hiding this comment

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

How about the USK goes first and then the SSK is "Syntax of equivalent SSK@ key:"? SSKs don't have to be formatted that way.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Add a discussion about the sameness of USK and SSK keys into
the manpage for fcpgenkey as this is one of the things I found
harder to understand as a newbie.
@Thynix Thynix merged commit 9e4127c into hyphanet:master Feb 22, 2015
@Thynix
Copy link
Contributor

Thynix commented Feb 22, 2015

Looks good, thanks!

@Soliloque
Copy link
Contributor Author

Thank you for your patience. Should I notify you next time I do a pull request?

@Thynix
Copy link
Contributor

Thynix commented Feb 22, 2015

No worries! :) Up to you - I should get an email, but if it looks like I've forgotten feel free to poke me about it.

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