[v3] Modern shell scripting (use $() instead of ``)

[Qemu-devel] [PATCH v3 0/2] Modern shell scripting (use $() instead of ``)

Mao Zhongyi posted 2 patches 7 years ago

Diff against v1 v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20181022084824.555-1-maozhongyi@cmss.chinamobile.com

Test docker-clang@ubuntu passed

Test checkpatch passed

Test asan passed

Test docker-mingw@fedora failed

Test docker-quick@centos7 passed

po/Makefile                                   | 2 +-
tests/docker/dockerfiles/debian-bootstrap.pre | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v3 0/2] Modern shell scripting (use $() instead of ``)

Posted by Mao Zhongyi 7 years ago

Various shell files contain a mix between obsolete `` and
modern $(); It would be nice to convert to using $()
everywhere.

On https://lists.gnu.org/archive/html/qemu-devel/2018-10/msg02920.html

I just replaced `` in scripts dir,  so this series is a 
thorough cleanup of all obsolete `` in the source tree. 

v3:
-removed the patch1 from this series.  [Eric Blake]

v2:
-doubling "$" avoid it "escape" in a Makefile.  [Thomas Huth]

Cc: philmd@redhat.com
Cc: peter.maydell@linaro.org
Cc: thuth@redhat.com
Cc: sw@weilnetz.de
Cc: famz@redhat.com
Cc: eblake@redhat.com

Mao Zhongyi (2):
  debian-bootstrap.pre: Modern shell scripting (use $() instead of ``)
  po/Makefile: Modern shell scripting (use $() instead of ``)

 po/Makefile                                   | 2 +-
 tests/docker/dockerfiles/debian-bootstrap.pre | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.17.1