From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [PATCH v2 18/23] travis/gitlab/azure: Use bash to avoid a_test_which_does_not_exist
Date: Mon, 16 Mar 2020 17:24:27 -0400 [thread overview]
Message-ID: <20200316212427.GP12423@bill-the-cat> (raw)
In-Reply-To: <20200315234303.18598-15-sjg@chromium.org>
On Sun, Mar 15, 2020 at 05:42:58PM -0600, Simon Glass wrote:
> Bash allows for variables to expand only if non-empty:
>
> $ var=test
> $ echo ${var:+"$var"}
> test
> $ echo ${var:+"-k $var"}
> -k test
> $ var=
> $ echo ${var:+"-k $var"}
>
> Use this feature to avoid the workaround.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Tom Rini <trini@konsulko.com>
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200316/6be2a35a/attachment.sig>
next prev parent reply other threads:[~2020-03-16 21:24 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-15 23:42 [PATCH v2 00/23] gitlab: Simplify the test script Simon Glass
2020-03-15 23:42 ` [PATCH v2 01/23] sandbox: Add documentation about required/useful packages Simon Glass
2020-03-15 23:42 ` [PATCH v2 02/23] main: Drop show_boot_progress() prototype Simon Glass
2020-03-15 23:42 ` [PATCH v2 03/23] buildman: Document the members of BuilderJob Simon Glass
2020-03-15 23:42 ` [PATCH v2 04/23] bulidman: Add support for a simple build Simon Glass
2020-03-15 23:42 ` [PATCH v2 05/23] buildman: Update help for -d Simon Glass
2020-03-15 23:42 ` [PATCH v2 06/23] buildman: Allow ignoring warnings in the return code Simon Glass
2020-03-16 20:50 ` Tom Rini
2020-03-17 16:11 ` Simon Glass
2020-03-15 23:42 ` [PATCH v2 07/23] buildman: Be more selective about which directories to remove Simon Glass
2020-03-15 23:42 ` [PATCH v2 08/23] buildman: Allow building within a subdir of the current dir Simon Glass
2020-03-15 23:42 ` [PATCH v2 09/23] buildman: Drop the -a option Simon Glass
2020-03-15 23:42 ` [PATCH v2 10/23] travis: Don't copy files into .bm-work/ Simon Glass
2020-03-16 21:23 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 11/23] travis: Split the building into two parts Simon Glass
2020-03-16 21:23 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 12/23] gitlab/azure: Use the -w option for sandbox_spl Simon Glass
2020-03-16 21:23 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 13/23] travis/gitlab/azure: Use --board buildman flag with test.py Simon Glass
2020-03-16 21:24 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 14/23] travis/gitlab/azure: Drop BUILDMAN variable " Simon Glass
2020-03-16 21:16 ` Tom Rini
2020-03-17 16:29 ` Simon Glass
2020-03-17 16:42 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 15/23] travis/gitlab/azure: Drop the buildman -d flag Simon Glass
2020-03-16 21:24 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 16/23] gitlab/azure: Drop unnecessary if..fi when using test.py Simon Glass
2020-03-16 21:24 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 17/23] gitlab/azure: Use -w flag for all test.py builds Simon Glass
2020-03-16 21:24 ` Tom Rini
2020-03-15 23:42 ` [PATCH v2 18/23] travis/gitlab/azure: Use bash to avoid a_test_which_does_not_exist Simon Glass
2020-03-16 21:24 ` Tom Rini [this message]
2020-03-15 23:42 ` [PATCH v2 19/23] travis/gitlab/azure: Drop the -E flag Simon Glass
2020-03-16 21:21 ` Tom Rini
2020-03-17 16:11 ` Simon Glass
2020-03-17 16:22 ` Tom Rini
2020-03-17 16:32 ` Simon Glass
2020-03-17 16:42 ` Tom Rini
2020-03-15 23:43 ` [PATCH v2 20/23] travis/gitlab/azure: Enable test_handoff Simon Glass
2020-03-16 21:24 ` Tom Rini
2020-03-15 23:43 ` [PATCH v2 21/23] travis/gitlab/azure: Simplify the exit code for test.py Simon Glass
2020-03-15 23:43 ` [PATCH v2 22/23] travis/gitlab/azure: Drop repeated buildman call with test.py Simon Glass
2020-03-16 21:24 ` Tom Rini
2020-03-15 23:43 ` [PATCH v2 23/23] test/py: Allow using buildman to build U-Boot Simon Glass
2020-03-16 21:24 ` Tom Rini
2020-03-16 22:08 ` Stephen Warren
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=20200316212427.GP12423@bill-the-cat \
--to=trini@konsulko.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox