All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andreas Vogel <Andreas.Vogel@anvo-it.de>
To: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
Cc: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: [BUG] GRUBs option parsing needs fixing
Date: Sun, 11 Mar 2012 02:59:25 +0100	[thread overview]
Message-ID: <4F5C06FD.7030209@anvo-it.de> (raw)
In-Reply-To: <4F5BF963.3000307@gmail.com>

Am 11.03.2012 02:01, schrieb Vladimir 'φ-coder/phcoder' Serbinenko:
>> It hurts that you think that I don't follow basic rules of communication
>> and cooperation.
> That's the expression I've got. My position shifted somewhat but yours
> remains unchanged. You look stubborn from this angle.
Then I'm sorry that I was not able to make it clear what I really
wanted. First of all I wanted to get feedback about my ideas and about
the possibilities for changes.

> Yes. And allowing "-s root" to continue is more important than to allow
> -s with no argument.
Don't get your point here. "-s" with no argument is already allowed.
This is actually causing the "unexpected behavior" I'm talking about.

> Actually another possibility is to keep this or this + 2 other
> occurencies to this behaviour while making all future uses to the GNU
> standards.
I think we are on the same road.

I understand now that it's impossible to change the argument parsing for
existing options taking optional arguments. I was just missing this
clear statement from your side (btw, right now I wonder myself why i
didn't ask you this directly).

I just didn't expect that you would even think about to agree having 3
options to behave the "old" way while allowing a "new" GRUB conformant
way which will be used for new options taking optional arguments.
This is a compromise which I think we really should go for.

> We can rename ARG_OPTIONAL to ARG_OLD_OPTIONAL, make ARG_OLD_OPTIONAL to
> behave as to ignore X in  --set X as possible argument.
I don't understand that.

The "old" behavior, e.g. for "search -s|--set", is that if there is no
argument "-s" or "--set" needs to be followed by another option or by
"--". In other words: any argument which is not an option that follows
-s or --set will be taken as the argument for -s or --set. That's the
current situation.





  reply	other threads:[~2012-03-11  1:59 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-05  1:13 Ideas for the future Andreas Vogel
2012-03-05  1:30 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-05  1:35   ` Samuel Thibault
2012-03-05  1:40     ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-05  9:23       ` Samuel Thibault
2012-03-05 11:16   ` Andreas Vogel
2012-03-05 12:54     ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-05 15:43       ` Andreas Vogel
2012-03-08 13:12         ` [BUG] GRUBs option parsing needs fixing Andreas Vogel
2012-03-08 14:15           ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-08 14:32             ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-08 15:03               ` Andreas Vogel
2012-03-08 15:18                 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-08 15:28                   ` Lennart Sorensen
2012-03-08 15:37                     ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-08 16:11                       ` Andreas Vogel
2012-03-08 15:45                   ` Andreas Vogel
2012-03-10 19:50                     ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-11  0:08                       ` Andreas Vogel
2012-03-11  1:01                         ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-11  1:59                           ` Andreas Vogel [this message]
2012-03-11  2:15                             ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-08 15:25                 ` Lennart Sorensen
2012-03-08 15:58                   ` Andreas Vogel
2012-03-08 15:24           ` Lennart Sorensen
2012-03-08 14:11         ` Ideas for the future Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-08 15:14           ` Andreas Vogel
2012-03-08 15:18             ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-05 16:33       ` Andreas Vogel
2012-03-08 14:23     ` Vladimir 'φ-coder/phcoder' Serbinenko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F5C06FD.7030209@anvo-it.de \
    --to=andreas.vogel@anvo-it.de \
    --cc=grub-devel@gnu.org \
    --cc=phcoder@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.