Am Samstag, den 26.01.2013, 16:54 +0100 schrieb Mathieu Malaterre:
> On Fri, Jan 4, 2013 at 1:45 AM, Benjamin Drung <bdr...@debian.org> wrote:
> > Am Montag, den 31.12.2012, 16:01 +0100 schrieb Mathieu Malaterre:
> >> Package: distro-info
> >> Version: 0.9~bpo60+1
> >> Severity: normal
> >>
> >>
> >> As per man page one should be able to use:
> >>
> >>        -c, --codename
> >>               print the codename (default)
> >>
> >>        -r, --release
> >>               print the release version
> >>
> >>        -f, --fullname
> >>               print the full name
> >>
> >>
> >> However those options always leads to:
> >>
> >> $ distro-info --fullname
> >> debian-distro-info: You have to select exactly one of --alias, --all, 
> >> --devel, --oldstable, --stable, --supported, --testing, --unsupported.
> >
> > -c, -r, and -f modify the output format. You still have to specify one
> > of the above options. You specified none of the above. Do you want to
> > list all distributions? Do you want to know testing?
> >
> > Can you suggest how to improve the error message?
> 
> Indeed this is not quite trivial. I am starring at `man printf`, and
> they make a distinction in between [FORMAT] and [ARGUMENT].

We could split the options into [FORMAT] and [OPTIONS]. The synopsis
would be "distro-info [FORMAT] [OPTIONS]" then. The order of the options
do not matter, so "distro-info [OPTIONS] [FORMAT]" will work as well. So
I am not fully convinced.

> Another solution would be to add an EXAMPLES section with shows a call
> with --codename and the other with --release

Yes, adding an example section would be a good idea.

PS: Patches are always welcome.

-- 
Benjamin Drung
Debian & Ubuntu Developer


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to