Bug 213013
Summary: | getopt.3: further clarify behaviour | ||
---|---|---|---|
Product: | Documentation | Reporter: | James Hunt (jamesodhunt) |
Component: | man-pages | Assignee: | documentation_man-pages (documentation_man-pages) |
Status: | RESOLVED DOCUMENTED | ||
Severity: | normal | CC: | alx |
Priority: | P1 | ||
Hardware: | All | ||
OS: | Linux | ||
Kernel Version: | Subsystem: | ||
Regression: | No | Bisected commit-id: | |
Attachments: |
getopt.3: further clarify behaviour
getopt.3 test program Updated patch (2021-05-30) Updated patch (2021-07-14) |
Description
James Hunt
2021-05-10 09:13:25 UTC
Created attachment 296705 [details]
getopt.3 test program
Build as:
```bash
gcc -o test-getopt test-getopt.c
```
Could you please send the patches to the mailing list? There they will get much more review (I think I'm the only one reading this bugzilla). You can CC <bugzilla-daemon@bugzilla.kernel.org> with '[BUG 213013]' as a subject prefix so that a copy ends up here. +Note that the plus symbol may also be used as an option character if it +does not appear as the first character in +.IR optstring . I'd change the wording there to be more correct (it doesn't matter if it _also_ appears as the first character), and also to use syntax similar to the rest of the page (using the characters in single quotes instead of their names): Note that \(aqx\(aq may be used as an option character if it is in a position other than the first character in .IR optstring. +If +.B POSIXLY_CORRECT +behaviour is required in this case +.I optstring +will contain two plus symbols. I don't think this is necessary. Not sure. Created attachment 297043 [details]
Updated patch (2021-05-30)
Hi Alex, Thanks again for reviewing. I've updated the patch but I still think it needs to be "spelt out" that two plus symbols may be required since that avoids any confusion. Also, I'm afraid I'm using a webmailer so submitting patches to a mailing list is going to be painful for me, whereas this bugzilla is quick and easy (and doesn't require me to register, format mails in particular ways, plus of course to manage a ton more emails ;) But if the real action is happening on the ML, I wonder if in the future it might be possible to forward all bugzilla activity automatically to the ML rather than _vice versa_? Thanks again for your help. Hi James, Sorry for the delay! (In reply to James Hunt from comment #4) > Thanks again for reviewing. I've updated the patch but I still think it > needs to be "spelt out" that two plus symbols may be required since that > avoids any confusion. Okay. > Also, I'm afraid I'm using a webmailer so submitting patches to a mailing > list is going to be painful for me, whereas this bugzilla is quick and easy > (and doesn't require me to register, BTW, you don't need to subscribe to <linux-man@vger.kernel.org> to post there :-) But, I understand the rest. > format mails in particular ways, plus > of course to manage a ton more emails ;) But if the real action is happening > on the ML, I wonder if in the future it might be possible to forward all > bugzilla activity automatically to the ML rather than _vice versa_? Yes. After you said these, I remembered that I asked about that some time ago. The ML was subscribed again to this bugzilla activity :) Now the review: Could you please use '.IR ...' instead of '\fI...\fP...'? See this extract from man-manpages(7): [[ The preferred way to write this in the source file is: .BR fcntl () (Using this format, rather than the use of "\fB...\fP()" makes it easier to write tools that parse man page source files.) ]] > Thanks again for your help. Thank you! Alex Created attachment 297861 [details]
Updated patch (2021-07-14)
Thanks Alex,
Updated patch attached.
Kind regards,
James
Hi James, Patch applied. Thanks, Alex |