* [PATCH 0/1] oe-init-build-env-memres: swap parameter order
@ 2017-06-21 19:49 Paul Eggleton
2017-06-21 19:49 ` [PATCH 1/1] " Paul Eggleton
0 siblings, 1 reply; 2+ messages in thread
From: Paul Eggleton @ 2017-06-21 19:49 UTC (permalink / raw)
To: openembedded-core
The following changes since commit 1d9a88f635549e68562de681e297b9270ad02d4e:
ovmf: fix secureboot PACKAGECONFIG + OpenSSL update (2017-06-16 10:58:19 +0100)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib paule/memres-init-param-order
http://cgit.openembedded.org/openembedded-core-contrib/log/?h=paule/memres-init-param-order
Paul Eggleton (1):
oe-init-build-env-memres: swap parameter order
oe-init-build-env-memres | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
--
2.9.4
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH 1/1] oe-init-build-env-memres: swap parameter order
2017-06-21 19:49 [PATCH 0/1] oe-init-build-env-memres: swap parameter order Paul Eggleton
@ 2017-06-21 19:49 ` Paul Eggleton
0 siblings, 0 replies; 2+ messages in thread
From: Paul Eggleton @ 2017-06-21 19:49 UTC (permalink / raw)
To: openembedded-core
This script expected the port number to be specified first, which is
somewhat counter-intuitive especially if you're used to
oe-init-build-env; besides, in local usage you are unlikely to need to
specify a custom port. Given that few people are using this yet (based
on the issues I have recently fixed), switch the arguments around so
that the two scripts behave consistently.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
oe-init-build-env-memres | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/oe-init-build-env-memres b/oe-init-build-env-memres
index 9e1425e..3f0d239 100755
--- a/oe-init-build-env-memres
+++ b/oe-init-build-env-memres
@@ -19,18 +19,18 @@
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
#
-# Normally this is called as '. ./oe-init-build-env-memres <portnumber> <builddir>'
+# Normally this is called as '. ./oe-init-build-env-memres <builddir> [portnumber]'
#
# This works in most shells (not dash), but not all of them pass the arguments
-# when being sourced. To workaround the shell limitation use "set <portnumber>
-# <builddir>" prior to sourcing this script.
+# when being sourced. To workaround the shell limitation use "set <builddir>
+# [portnumber]" prior to sourcing this script.
#
-if [ -z "$1" ]; then
+if [ -z "$2" ]; then
echo "No port specified, using dynamically selected port"
port=-1
else
- port=$1
- shift
+ port=$2
+ set -- "$1" "$3" "$4" "$5" "$6"
fi
if [ -n "$BASH_SOURCE" ]; then
--
2.9.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-06-21 19:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-21 19:49 [PATCH 0/1] oe-init-build-env-memres: swap parameter order Paul Eggleton
2017-06-21 19:49 ` [PATCH 1/1] " Paul Eggleton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox