All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Chargin <jimccrown@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] HUSH logical AND/OR expressions
Date: Tue, 28 Apr 2015 12:13:18 -0700	[thread overview]
Message-ID: <553FDBCE.1000207@gmail.com> (raw)
In-Reply-To: <CANr=Z=adAw3XPer4VLdecYhv1Ptg93x_cHecLxqsGEZTCbM=zQ@mail.gmail.com>

Dear Joe,

On 04/28/2015 11:35 AM, Joe Hershberger wrote:
> Hi James,
>
> On Tue, Apr 28, 2015 at 1:19 PM, James Chargin <jimccrown@gmail.com> wrote:
>> Dear Joe Hershberger,
>>
>> On 04/28/2015 11:00 AM, Joe Hershberger wrote:
>>>
>>> Hi Joakim,
>>>
>>> On Mon, Apr 27, 2015 at 8:39 AM, Joakim Tjernlund
>>> <joakim.tjernlund@transmode.se> wrote:
>>>>
>>>> Trying to get a better handle of HUSH shell expressions, this does not
>>>> work as I expect:
>>>> => false && true || echo ECHO
>>>> => false && false || echo ECHO

...

>
> I always just assumed this was a limitation of hush, but it's now
> looking like a bug instead.
>
> I haven't debugged into it, but I tend to stay away from hush these
> days. When I've submitted a bug fix to this area I've been told that
> we should upgrade wholesale instead, and I've haven't been up for that
> task so far.  http://lists.denx.de/pipermail/u-boot/2012-November/139841.html

That's a great conversation with Wolfgang, thanks for the reference.

Like you, I have tended to use hush carefully. It is similar to sh/bash, 
but different enough that I never count on an easy port of bash script 
fragments.

I noticed you asked for a reference to the original hush code that is 
the origin for U-Boot's adaptation. I couldn't find a reply to that 
question. I also haven't been able to find anything with a quick we 
search. Do you know where it did come from?

>
> Perhaps there is an easy, minor bugfix for this that would be palette-able.

I've been "afraid" of the hush code for a while. I don't think I want to 
mess with it now. I'll just continue to work with what we've got, carefully.

Thanks again for your attention,

Jim

  reply	other threads:[~2015-04-28 19:13 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-27 13:39 [U-Boot] HUSH logical AND/OR expressions Joakim Tjernlund
2015-04-28 18:00 ` Joe Hershberger
2015-04-28 18:19   ` James Chargin
2015-04-28 18:35     ` Joe Hershberger
2015-04-28 19:13       ` James Chargin [this message]
2015-04-28 19:47         ` Joe Hershberger
2015-04-28 22:51           ` Wolfgang Denk
2015-04-29 15:19             ` James Chargin

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=553FDBCE.1000207@gmail.com \
    --to=jimccrown@gmail.com \
    --cc=u-boot@lists.denx.de \
    /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.