Openembedded Core Discussions
 help / color / mirror / Atom feed
* [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