-
Notifications
You must be signed in to change notification settings - Fork 242
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
Manpage updates #1173
Comments
Also here are translation updates from various translators: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1080487 [ro] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1076260 [fr] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1043536 [fr] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=968304 [pt] |
Hi @zeha , To be able to review the changes to the manual pages, I'd like to see them in patches with commit messages that justifies the changes. A huge diff without justification isn't easy to review. |
Could you ask that user to provide a patch set (not a single patch) where each patch does one thing and is justified? |
I fear this will be unlikely to bear much fruit. Please just take whatever you think is right, and I can close the rest pointing here, if need be. |
Hmmmm, okay. How about if you send the diffs as a huge PR with no details at all? We can keep that PR as a draft, and then I'll slowly pick stuff from there. Is that okay for you? |
Here's what I got out of the first three bugs: Notably from the first bug a lot was already applied in master, and patch failed to apply most of the remaining hunks. The chage submission doesn't include a patch against the XML, so I didn't include that. |
Thanks! |
Hi,
Markus Hiereth sent some man page updates to the Debian BTS, and @hallyn interacted there with them in the past. Could upstream please either take or reject these updates?
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1005253
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1004688
Thanks :)
The text was updated successfully, but these errors were encountered: