public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Rebecca Cran <rebecca@bluestop.org>
To: edk2-devel@lists.01.org
Subject: [PATCH] BaseTools: Convert PosixLike scripts to sh, reduce duplication and fix some bugs in edksetup.sh and BuildEnv
Date: Thu, 21 Feb 2019 22:12:52 -0700	[thread overview]
Message-ID: <20190222051253.51931-1-rebecca@bluestop.org> (raw)

To make it easier to use EDK2 on systems other than Linux, where the default shell might
not be bash, I'd like to convert the shell scripts to use POSIX sh where possible.

edksetup.sh and BuildEnv remain bash scripts, but I hope to convert them to sh in a future
changeset.

I did see the argument against using $0 in http://mywiki.wooledge.org/BashFAQ/028, but I'm
not sure anyone using EDK2 will run into problems the article describes.

I checked the scripts with shellcheck.




             reply	other threads:[~2019-02-22  5:13 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-22  5:12 Rebecca Cran [this message]
2019-02-22  5:12 ` [PATCH] Convert PosixLike scripts to sh, reduce duplication and fix some bugs Rebecca Cran
2019-02-26  3:11   ` Feng, Bob C
2019-02-26  3:50     ` Rebecca Cran

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-list 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=20190222051253.51931-1-rebecca@bluestop.org \
    --to=devel@edk2.groups.io \
    /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