All of lore.kernel.org
 help / color / mirror / Atom feed
* [base][PATCH 1/2] setup-environment: Add support for set distro
@ 2016-11-07 11:54 Fabio Berton
  2016-11-07 11:54 ` [base][PATCH 2/2] setup-environment: Move error log from top to bottom Fabio Berton
  2016-11-07 13:17 ` [base][PATCH 1/2] setup-environment: Add support for set distro Mario Domenech Goulart
  0 siblings, 2 replies; 3+ messages in thread
From: Fabio Berton @ 2016-11-07 11:54 UTC (permalink / raw)
  To: meta-freescale

To run setup-environment you need to set one of distros listed on
supported distros using command like:

MACHINE=imx6qsabresd DISTRO=fslc-framebuffer source setup-environment build

Note that DISTRO is a required variable and won't be set if not defined.

Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
---
 setup-environment | 26 ++++++++++++++++++++++----
 1 file changed, 22 insertions(+), 4 deletions(-)

diff --git a/setup-environment b/setup-environment
index 099b21b..a955182 100755
--- a/setup-environment
+++ b/setup-environment
@@ -27,19 +27,25 @@ usage()
 {
     echo -e "\nUsage: source $PROGNAME <build-dir>
     <build-dir>: specifies the build directory location (required)
+    <distro>:    specifies the disto name
 
 If undefined, this script will set \$MACHINE to 'imx6qsabresd'.
+You need to set DISTRO to use this environment.
 "
 
     ls sources/*/conf/machine/*.conf > /dev/null 2>&1
+    ls sources/meta-freescale-distro/conf/distro/fslc-*.conf > /dev/null 2>&1
     if [ $? -eq 0 ]; then
         echo -e "
 Supported machines: `echo; ls sources/*/conf/machine/*.conf \
 | sed s/\.conf//g | sed -r 's/^.+\///' | xargs -I% echo -e "\t%"`
 
-To build for a machine listed above, run this script as:
-MACHINE=<machine> source $PROGNAME <build-dir>
-"
+Supported distros: `echo; ls sources/meta-freescale-distro/conf/distro/fslc-*.conf \
+| sed s/\.conf//g | sed -r 's/^.+\///' | xargs -I% echo -e "\t%"`
+
+To build for a distro and machine listed above, run this script as:
+MACHINE=<machine> DISTRO=<distro> source $PROGNAME <build-dir>
+Ex: MACHINE=imx6qsabresd DISTRO=fslc-framebuffer source $PROGNAME build"
     fi
 }
 
@@ -105,7 +111,19 @@ if [ -z "$SDKMACHINE" ]; then
 fi
 
 if [ -z "$DISTRO" ]; then
-    DISTRO='poky'
+    echo "You must set DISTRO before setting up the environment."
+fi
+# Check the machine type specified
+LIST_DISTROS=`ls -1 $CWD/sources/meta-freescale-distro/conf/distro`
+VALID_DISTRO=`echo -e "$LIST_DISTROS" | grep ${DISTRO}.conf$ | wc -l`
+if [ "x$DISTRO" = "x" ] || [ "$VALID_DISTRO" = "0" ]; then
+    echo -e "The \$DISTRO you have specified ($DISTRO) is not supported by this build setup\n"
+    usage && clean_up
+    return 1
+else
+    if [ ! -e $1/conf/local.conf.sample ]; then
+        echo "Configuring for ${DISTRO}"
+    fi
 fi
 
 OEROOT=$PWD/sources/poky
-- 
2.1.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [base][PATCH 2/2] setup-environment: Move error log from top to bottom
  2016-11-07 11:54 [base][PATCH 1/2] setup-environment: Add support for set distro Fabio Berton
@ 2016-11-07 11:54 ` Fabio Berton
  2016-11-07 13:17 ` [base][PATCH 1/2] setup-environment: Add support for set distro Mario Domenech Goulart
  1 sibling, 0 replies; 3+ messages in thread
From: Fabio Berton @ 2016-11-07 11:54 UTC (permalink / raw)
  To: meta-freescale

If setup-environment won't be running like expect show error message
at bottom of log to be more visible.

Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
---
 setup-environment | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/setup-environment b/setup-environment
index a955182..53c324f 100755
--- a/setup-environment
+++ b/setup-environment
@@ -97,8 +97,9 @@ fi
 LIST_MACHINES=`ls -1 $CWD/sources/*/conf/machine`
 VALID_MACHINE=`echo -e "$LIST_MACHINES" | grep ${MACHINE}.conf$ | wc -l`
 if [ "x$MACHINE" = "x" ] || [ "$VALID_MACHINE" = "0" ]; then
+    usage
     echo -e "\nThe \$MACHINE you have specified ($MACHINE) is not supported by this build setup"
-    usage && clean_up
+    clean_up
     return 1
 else
     if [ ! -e $1/conf/local.conf.sample ]; then
@@ -117,8 +118,9 @@ fi
 LIST_DISTROS=`ls -1 $CWD/sources/meta-freescale-distro/conf/distro`
 VALID_DISTRO=`echo -e "$LIST_DISTROS" | grep ${DISTRO}.conf$ | wc -l`
 if [ "x$DISTRO" = "x" ] || [ "$VALID_DISTRO" = "0" ]; then
+    usage
     echo -e "The \$DISTRO you have specified ($DISTRO) is not supported by this build setup\n"
-    usage && clean_up
+    clean_up
     return 1
 else
     if [ ! -e $1/conf/local.conf.sample ]; then
-- 
2.1.4



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [base][PATCH 1/2] setup-environment: Add support for set distro
  2016-11-07 11:54 [base][PATCH 1/2] setup-environment: Add support for set distro Fabio Berton
  2016-11-07 11:54 ` [base][PATCH 2/2] setup-environment: Move error log from top to bottom Fabio Berton
@ 2016-11-07 13:17 ` Mario Domenech Goulart
  1 sibling, 0 replies; 3+ messages in thread
From: Mario Domenech Goulart @ 2016-11-07 13:17 UTC (permalink / raw)
  To: Fabio Berton; +Cc: meta-freescale

Hi Fabio,

On Mon,  7 Nov 2016 09:54:31 -0200 Fabio Berton <fabio.berton@ossystems.com.br> wrote:

> To run setup-environment you need to set one of distros listed on
> supported distros using command like:
>
> MACHINE=imx6qsabresd DISTRO=fslc-framebuffer source setup-environment build
>
> Note that DISTRO is a required variable and won't be set if not defined.
>
> Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
> ---
>  setup-environment | 26 ++++++++++++++++++++++----
>  1 file changed, 22 insertions(+), 4 deletions(-)
>
> diff --git a/setup-environment b/setup-environment
> index 099b21b..a955182 100755
> --- a/setup-environment
> +++ b/setup-environment
> @@ -27,19 +27,25 @@ usage()
>  {
>      echo -e "\nUsage: source $PROGNAME <build-dir>
>      <build-dir>: specifies the build directory location (required)
> +    <distro>:    specifies the disto name

Small typo: s/disto/distro/


>  If undefined, this script will set \$MACHINE to 'imx6qsabresd'.
> +You need to set DISTRO to use this environment.
>  "
>  
>      ls sources/*/conf/machine/*.conf > /dev/null 2>&1
> +    ls sources/meta-freescale-distro/conf/distro/fslc-*.conf > /dev/null 2>&1
>      if [ $? -eq 0 ]; then
>          echo -e "
>  Supported machines: `echo; ls sources/*/conf/machine/*.conf \
>  | sed s/\.conf//g | sed -r 's/^.+\///' | xargs -I% echo -e "\t%"`
>  
> -To build for a machine listed above, run this script as:
> -MACHINE=<machine> source $PROGNAME <build-dir>
> -"
> +Supported distros: `echo; ls sources/meta-freescale-distro/conf/distro/fslc-*.conf \
> +| sed s/\.conf//g | sed -r 's/^.+\///' | xargs -I% echo -e "\t%"`
> +
> +To build for a distro and machine listed above, run this script as:
> +MACHINE=<machine> DISTRO=<distro> source $PROGNAME <build-dir>
> +Ex: MACHINE=imx6qsabresd DISTRO=fslc-framebuffer source $PROGNAME build"
>      fi
>  }
>  
> @@ -105,7 +111,19 @@ if [ -z "$SDKMACHINE" ]; then
>  fi
>  
>  if [ -z "$DISTRO" ]; then
> -    DISTRO='poky'
> +    echo "You must set DISTRO before setting up the environment."
> +fi
> +# Check the machine type specified
> +LIST_DISTROS=`ls -1 $CWD/sources/meta-freescale-distro/conf/distro`
> +VALID_DISTRO=`echo -e "$LIST_DISTROS" | grep ${DISTRO}.conf$ | wc -l`
> +if [ "x$DISTRO" = "x" ] || [ "$VALID_DISTRO" = "0" ]; then
> +    echo -e "The \$DISTRO you have specified ($DISTRO) is not supported by this build setup\n"
> +    usage && clean_up
> +    return 1
> +else
> +    if [ ! -e $1/conf/local.conf.sample ]; then
> +        echo "Configuring for ${DISTRO}"
> +    fi
>  fi
>  
>  OEROOT=$PWD/sources/poky
> -- 
> 2.1.4

-- 
http://parenteses.org/mario


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-11-07 13:17 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-07 11:54 [base][PATCH 1/2] setup-environment: Add support for set distro Fabio Berton
2016-11-07 11:54 ` [base][PATCH 2/2] setup-environment: Move error log from top to bottom Fabio Berton
2016-11-07 13:17 ` [base][PATCH 1/2] setup-environment: Add support for set distro Mario Domenech Goulart

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.