zsh-workers
 help / color / mirror / code / Atom feed
From: Mikael Magnusson <mikachu@gmail.com>
To: sergio <mailbox@sergio.spb.ru>
Cc: zsh-workers@zsh.org
Subject: Re: PATCH: zcalc prompt fix
Date: Mon, 14 Jun 2010 01:54:09 +0200	[thread overview]
Message-ID: <AANLkTikkB-747sL6TVo5BW0UwmlkqhBGp_tWSa3fh9hx@mail.gmail.com> (raw)
In-Reply-To: <4C1567F9.6050406@sergio.spb.ru>

That's not very surprising considering 4.3.10 was released before the fix.
If you want debian to backport fixes, maybe you should talk to them?

On 14 June 2010 01:21, sergio <mailbox@sergio.spb.ru> wrote:
> Hello.
>
> This bug still presents in 4.3.10-13 in debian.
>
>> There are problems when using color in ZCALCPROMPT.
>> This is simple fix.
>>
>> --- /usr/share/zsh/functions/Misc/zcalc 2009-12-28 06:57:20.000000000 >
>> +0300
>> +++ zcalc       2010-01-06 14:55:15.000000000 +0300
>> @@ -152,7 +152,7 @@
>>  done
>>
>>  psvar[1]=$num
>> -while vared -cehp "${(%)ZCALCPROMPT}" line; do
>> +while vared -cehp "${ZCALCPROMPT}" line; do
>>    [[ -z $line ]] && break
>>    # special cases
>>    # Set default base if `[#16]' or `[##16]' etc. on its own.
>
> On 01/08/2010 12:28 PM, Peter Stephenson wrote:
>>
>> On Fri, 08 Jan 2010 02:11:11 +0300
>> sergio<mailbox@sergio.spb.ru>  wrote:
>>>
>>> -p for vared indicates, that the string will be taken as the prompt, and
>>> it doesn't need the prompt expansion flag (%).
>>
>> Ah, right, vared -p obeys the PROMPTPERCENT option so we don't need the
>> (%) in any case.
>
> --
> sergio.
>



-- 
Mikael Magnusson


  reply	other threads:[~2010-06-13 23:54 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-06 18:57 sergio
2010-01-07 21:33 ` Peter Stephenson
2010-01-07 23:11   ` sergio
2010-01-08  0:15     ` Richard Hartmann
2010-01-08  1:00       ` sergio
2010-01-08  9:21         ` Richard Hartmann
2010-01-08  9:28     ` Peter Stephenson
2010-06-13 23:21       ` sergio
2010-06-13 23:54         ` Mikael Magnusson [this message]
2010-06-14 11:11           ` sergio
2010-06-13 23:22       ` sergio

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=AANLkTikkB-747sL6TVo5BW0UwmlkqhBGp_tWSa3fh9hx@mail.gmail.com \
    --to=mikachu@gmail.com \
    --cc=mailbox@sergio.spb.ru \
    --cc=zsh-workers@zsh.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.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).