Bart Smaalders wrote: > Frank Ludolph wrote: >> Shawn Walker wrote: >>> I'm adding a --debug option to the depot server with the idea that >>> you name the "debug feature" you want to enable. >>> >>> In this case, --debug=headers, which will cause the headers of every >>> request to be logged. >>> >>> In the future, there will likely be other "debug features"; how >>> would you expect the user to specify those? >>> >>> --debug=feature1 --debug=feature2 --debug=feature3 >>> >>> or: >>> >>> --debug=feature1,feature2,feature3 >>> >>> ? >>> >> According to the CLIP guidelines (actually the CLIP Companion) both >> are acceptable. I would suggest the comma separated list - less >> typing - though blanks are also acceptable separators and reasonable >> since it appears that quoting would never be required. >> > > > In general, I use stuff of the form: > > --variant arch=sparc --variant debug=true > > to simplify parsing/escaping/quoting issues. Options with arguments as arguments to other options? I don't think this is valid syntax according to the CLIP guidelines. It also adds a lot of extra typing. They should more properly be options of their own: --arch=sparc --debug=true
Frank _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
