On Wed, Jan 27, 2010 at 04:54:08PM -0800, Stephen Hahn wrote: > * Gary Pennington <[email protected]> [2010-01-27 12:35]: > > The guiding principles are: > > > > - produce output that standard unix text processing tools can process > > - assume that repetition of redundant data to support line-oriented > > reporting is fine > > - don't print headers when using parseable mode > > - assume that "something better" is coming along in the future and this > > approach is good enough for now > > > gary...@osol:~$ pkg publisher -p > > opensolaris.org true true true http://pkg.opensolaris.org/dev/ > > origin online > > opensolaris.org true true true > > http://pkg-na-2.opensolaris.org/dev/ mirror online > > osol false false false http://192.168.2.100:10000/ origin > > online > > This needs to be "-F tsv" (if those are tabs), not "-p". Other > subcommands will be producing formatted output. >
It is tabs... Do you mean that there should be an additional format specifier? e.g: pkg publisher -p -F tsv or why not ? pkg publisher -p tsv pkg publisher -p <other format> Also, even if other commands are using -p, that won't make it inapplicable to the publisher command will it? At this stage, perhaps it's best to putback what I have and file a bug to harmonise these requirements at a later date when other subcommands introduce formatted output? I'll file this today. I won't update documentation or manpage until this issue is resolved. > > gary...@osol:~$ pkg publisher -pn > > opensolaris.org true true true http://pkg.opensolaris.org/dev/ > > origin online > > opensolaris.org true true true > > http://pkg-na-2.opensolaris.org/dev/ mirror online > > Replace "-n" with "--enabled". We're using "-n" for no action taken. This option exists. I'm not introducing it. It's not documented in the manpage, but in the code: if opt == "-n": inc_disabled = False I just included that example to show that it interacted correctly with the existing options. Thanks, Gary > > - Stephen > > -- > [email protected] http://blogs.sun.com/sch/ _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
