All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Cc: Andreas Vogel <Andreas.Vogel@anvo-it.de>
Subject: Re: Device instability, gettext and default
Date: Sat, 03 Mar 2012 16:06:36 +0100	[thread overview]
Message-ID: <4F52337C.4050909@gmail.com> (raw)
In-Reply-To: <4F522FFC.20308@anvo-it.de>

On 03.03.2012 15:51, Andreas Vogel wrote:
> Vladimir,
>
> this enhancement is part of my patches which i sent some days ago.....
>
> I introduced an option "--label STRING" which will be used for display 
> if it's set and which can also have environment variables (another 
> part of my patches).
>
> It's just a matter of taste though if to use a new option --id for an 
> invariant menu id or to use the actual menuentry title as the 
> invariant id and to have a new option --label for the display string. 
> Both ways provide backward compatibility.
>
> BTW, I just wonder a little that you refused my enhancements due to 
> code freeze but are now enhancing anyway ;-)
>
The reason is simple: you submitted a huge patch for several things, 
most of them new features. If you had submitted small patches each 
making one consistent logical change (but it may depend on other 
changes) then I would have reviewed the ones fixing bugs (like the one 
you describe). That's the general truth that 20 small patches go in much 
faster than one patch big patch doing 20 things put together.
> Andreas
>
>
> Am 03.03.2012 14:37, schrieb Vladimir 'φ-coder/phcoder' Serbinenko:
>> Hello, all. Currently we use and recommend using the title for 
>> default variable. It has however following problems:
>> 1) When device names change the title changes (because of the "(on 
>> $device)" part)
>> 2) If user changes locale the part ", with" gets translated and again 
>> the title changes
>> Attached patch changes it to the use of IDs specified by --id but 
>> keeps title possiblity for backward compatibility. Any comments?
>>
>>
>>
>> _______________________________________________
>> Grub-devel mailing list
>> Grub-devel@gnu.org
>> https://lists.gnu.org/mailman/listinfo/grub-devel
>
> -- 
> Andreas Vogel
> Dipl.-Inform.
> Hellerweg 60
> 73728 Esslingen
> Germany
> ____________________________________________
> E-Mail:Andreas.Vogel@anvo-it.de
> Office:  +49 (711) 937 1742
> FAX:     +49 (711) 937 1745
> Mobile:  +49 (172) 730 7440
>
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko



  reply	other threads:[~2012-03-03 15:06 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-03 13:37 Device instability, gettext and default Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-03 14:27 ` Jordi Mallach
2012-03-03 14:51 ` Andreas Vogel
2012-03-03 15:06   ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
2012-03-03 19:48 ` Andreas Vogel
2012-03-03 20:04   ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-03 20:08 ` Andreas Vogel
2012-03-03 20:10   ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-04 23:51 ` Andreas Vogel
2012-03-05  0:03   ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-05  0:20     ` Andreas Vogel
2012-03-05  0:34       ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-03-05 11:51         ` Andreas Vogel
2012-03-05 12:37           ` 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=4F52337C.4050909@gmail.com \
    --to=phcoder@gmail.com \
    --cc=Andreas.Vogel@anvo-it.de \
    --cc=grub-devel@gnu.org \
    /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.