Dear Joshua, thanks for your reply and for your comments. My reply follows the quoted text. On Fri, 2009-11-27 at 14:07 -0500, Joshua Brindle wrote: > Joshua Brindle wrote: > > Guido Trentalancia wrote: > >> Dear Eamon, > >> > >> here are the two "maintenance" patches that I did post earlier this > >> month (along with the new manual pages). They are intended to close > >> Ticket #1 [1672486] that I found open on Tresys pages > >> (http://userspace.selinuxproject.org/trac/ticket/1). > >> > >> I do apologize for not putting the keyword "[PATCH]" in the original > >> message. > >> > >> Here is a summary of what has been changed for policycoreutils: > >> > >> - introduced proper handling of -h, -V options and their respective long > >> formats --help and --version to all binaries that are produced from C > >> code. The same issue is not tackled for Python-based tools (e.g. > >> semanage); > >> - relative manual pages have also been updated accordingly. > >> > >> And here is what has been changed for checkpolicy/checkmodule: > >> > >> - introduced proper handling of -h, -V and the long formats --help and > >> --version to all binaries (checkpolicy/checkmodule); > >> - introduced the handling of long options for some of the other > >> available options; > >> - manual pages have also been updated accordingly (and a few > >> undocumented options have been documented). > >> > >> One of the original two patches (the one against policycoreutils) did > >> not compile cleanly anymore against the current HEAD of SELinux, so I > >> have created an updated one, which is the one attached here. > >> > >> It's just a matter of some really minor issues, but I hope it helps, at > >> least the ticket can now be closed... > >> > >> Kind regards, > >> > >> Guido > >> > > > > > > Thank you for doing this. Unfortunately it looks like your mail client > > decided to base64 encode the attachment. > > > > It is always best to use git to generate patch emails, please see: > > http://andrewprice.me.uk/weblog/entry/generating-patch-emails-with-git > > > > I also have a couple comments about this patch. First I'm not so sure > about the: > "[program name] version [version]." > > format. For people to be able to test versions in shell scripts I think > the output should simply be the version. > > I'm not sure what the "libtool" hack thing is all about, why is that > necessary? In fact I'm not sure what the whole opt_program_name thing is > about, what are you trying to address here? That change makes the patch > very messy since you are going through and replacing every occurrence of > argv[0] with program_name. Typically changes should be separated in > patches to ease review, I'm not sure what this has to do with adding > --version to the apps. > > Also, why -V rather than -v ? The "[program name] version [version] is the standard (see other GNU packages like gcc or binutils). Scripts that only need the "version" argument can still filter out the rest using awk or something like that. The opt_program_name thing has been taken from secon.c. It is intended to provide the name of the program without the path to the binary or other garbage. It has to do with adding version simply because of the standard format mentioned above which requires the name of the program. If you have not received the patch because of the encoding performed by my mail client, I can send it again. I look forward to hearing from you. Thank you. Greetings, Guido Trentalancia -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
