Cleanup warnings reported by mandoc -Tlint -Wwarning for bin/.
Details
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
bin/cat/cat.1 | ||
---|---|---|
198 ↗ | (On Diff #36092) | I do not really like going backwards from markup to no-markup. |
bin/date/date.1 | ||
135 ↗ | (On Diff #36092) | "using", while you're here. |
bin/kenv/kenv.1 | ||
95 ↗ | (On Diff #36092) | This comma could be removed while you're here. |
96 ↗ | (On Diff #36092) | There is a .Sq macro for single quotes around things, but maybe that is too much churn to ask you to add into here (as there are several uses visbile in the context). |
bin/rm/rm.1 | ||
237 ↗ | (On Diff #36092) | I attempted to verify that the "posix 1003.1 edition 2013" referred to by bapt in the message for r290480 corresponds to this, but don't have full confidence in wikipedia for this role. Can you say what verification you did? |
bin/sh/sh.1 | ||
1036 ↗ | (On Diff #36092) | It seems to work just as well if the "Ns" is dropped entirely, per line 1322's behavior when rendered. |
bin/rm/rm.1 | ||
---|---|---|
237 ↗ | (On Diff #36092) | The problem is that "-p1003.1-2013" didn't render at all. If I understand it correctly, 2013 edition incorporates the Technical Corrigendum 1 for 2008 one, and looking at 2016 edition I see only the "POSIX.1-2008, Technical Corrigendum 2, XCU/TC2-2008/0163 [542], XCU/TC2-2008/0164 [819], and XCU/TC2-2008/0165 [542] are applied." listed in Change History, so it's my *guess* that saying "-p1003.1-2008" should be enough here. |
bin/rm/rm.1 | ||
---|---|---|
237 ↗ | (On Diff #36092) | Although -p1003.1-2008 should suffice in this particular case, I think it makes more sense to add the 2013 and 2016 versions to the list of versions. |
I hope so, yes :-) but let me remove the rm(1) change WRT the .St, which is fixed by bapt adding the standard to st.in already.
bin/kenv/kenv.1 | ||
---|---|---|
94 ↗ | (On Diff #36212) | Confusingly, "whitespace" is both the singular and the plural, so no extra 's' is needed. |
Oversight, though almost all of the changes do not affect the rendered version, and the ones that do are sufficiently minor that I don't plan to do a follow-up.
Thanks for the reminder!
Sure. Just wanted to make sure it was intentional, and if not some intentional decision made. I'm cool with this outcome, btw, since I agree these are minor.
Yes, it was intentional, as my understanding is that date bump is required for *content* changes, and not simple markup fixes.