All of lore.kernel.org
 help / color / mirror / Atom feed
* [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
@ 2015-03-13 13:49 Jacob Stiffler
  2015-03-13 13:50 ` Cooper Jr., Franklin
  2015-03-13 13:57 ` Denys Dmytriyenko
  0 siblings, 2 replies; 9+ messages in thread
From: Jacob Stiffler @ 2015-03-13 13:49 UTC (permalink / raw)
  To: meta-arago

Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
 create-sdcard.sh |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/create-sdcard.sh b/create-sdcard.sh
index ca02ad2..17ae564 100644
--- a/create-sdcard.sh
+++ b/create-sdcard.sh
@@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then
 fi
 
 THEPWD=$EXEPATH
-PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
+PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
 
 if [ "$PARSEPATH" != "" ] ; then
 PATHVALID=1
@@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 
 	#check that in the right directory
 
-	THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
+	THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
 
 	if [ $PATHVALID -eq 1 ]; then
 	echo "now installing:  $THEEVMSDK"
@@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 			then
 				echo "Directory exists"
 				echo ""
-				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
+				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*processor-sdk.*.[0-9]/'`
 				#echo $PARSEPATH
 
 				if [ "$PARSEPATH" != "" ] ; then
@@ -731,7 +731,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 				#echo $PATHVALID
 				if [ $PATHVALID -eq 1 ] ; then
 
-				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
+				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'processor-sdk-.*[0-9]'`
 				echo "Is this the correct SDK: $THEEVMSDK"
 				echo ""
 				read -p 'Is this correct? [y/n] : ' ISRIGHTPATH
@@ -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 					*)  echo "Please enter y or n";ENTERCORRECTLY=0;;
 					esac
 				else
-				echo "Invalid SDK path make sure to include ti-sdk-xxxx"
+				echo "Invalid SDK path make sure to include processor-sdk-xxxx"
 				ENTERCORRECTLY=0
 				fi
 
@@ -804,7 +804,7 @@ cat << EOM
       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
 
   If files are located in a directory write the directory path
-      e.x. $: /ti-sdk/board-support/prebuilt-images/
+      e.x. $: /processor-sdk/board-support/prebuilt-images/
 
   NOTE: Not all platforms will have an MLO file and this file can
         be ignored for platforms that do not support an MLO.
@@ -937,7 +937,7 @@ cat << EOM
       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
 
   If files are located in a directory write the directory path
-      e.x. $: /ti-sdk/targetNFS/
+      e.x. $: /processor-sdk/targetNFS/
 
 ################################################################################
 
-- 
1.7.9.5



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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-13 13:49 [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk Jacob Stiffler
@ 2015-03-13 13:50 ` Cooper Jr., Franklin
  2015-03-13 13:56   ` Stiffler, Jacob
  2015-03-13 13:57 ` Denys Dmytriyenko
  1 sibling, 1 reply; 9+ messages in thread
From: Cooper Jr., Franklin @ 2015-03-13 13:50 UTC (permalink / raw)
  To: Stiffler, Jacob; +Cc: meta-arago@arago-project.org

Send that patch to me as an attachment so I can commit it.


> On Mar 13, 2015, at 8:49 AM, Stiffler, Jacob <j-stiffler@ti.com> wrote:
> 
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
> create-sdcard.sh |   14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/create-sdcard.sh b/create-sdcard.sh
> index ca02ad2..17ae564 100644
> --- a/create-sdcard.sh
> +++ b/create-sdcard.sh
> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then
> fi
> 
> THEPWD=$EXEPATH
> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
> 
> if [ "$PARSEPATH" != "" ] ; then
> PATHVALID=1
> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
> 
>    #check that in the right directory
> 
> -    THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +    THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
> 
>    if [ $PATHVALID -eq 1 ]; then
>    echo "now installing:  $THEEVMSDK"
> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>            then
>                echo "Directory exists"
>                echo ""
> -                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
> +                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*processor-sdk.*.[0-9]/'`
>                #echo $PARSEPATH
> 
>                if [ "$PARSEPATH" != "" ] ; then
> @@ -731,7 +731,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>                #echo $PATHVALID
>                if [ $PATHVALID -eq 1 ] ; then
> 
> -                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'processor-sdk-.*[0-9]'`
>                echo "Is this the correct SDK: $THEEVMSDK"
>                echo ""
>                read -p 'Is this correct? [y/n] : ' ISRIGHTPATH
> @@ -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>                    *)  echo "Please enter y or n";ENTERCORRECTLY=0;;
>                    esac
>                else
> -                echo "Invalid SDK path make sure to include ti-sdk-xxxx"
> +                echo "Invalid SDK path make sure to include processor-sdk-xxxx"
>                ENTERCORRECTLY=0
>                fi
> 
> @@ -804,7 +804,7 @@ cat << EOM
>       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
> 
>   If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
> 
>   NOTE: Not all platforms will have an MLO file and this file can
>         be ignored for platforms that do not support an MLO.
> @@ -937,7 +937,7 @@ cat << EOM
>       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
> 
>   If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/targetNFS/
> +      e.x. $: /processor-sdk/targetNFS/
> 
> ################################################################################
> 
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-13 13:50 ` Cooper Jr., Franklin
@ 2015-03-13 13:56   ` Stiffler, Jacob
  0 siblings, 0 replies; 9+ messages in thread
From: Stiffler, Jacob @ 2015-03-13 13:56 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org

[-- Attachment #1: Type: text/plain, Size: 3533 bytes --]



-----Original Message-----
From: Cooper Jr., Franklin 
Sent: Friday, March 13, 2015 9:51 AM
To: Stiffler, Jacob
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.

Send that patch to me as an attachment so I can commit it.


> On Mar 13, 2015, at 8:49 AM, Stiffler, Jacob <j-stiffler@ti.com> wrote:
> 
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
> create-sdcard.sh |   14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/create-sdcard.sh b/create-sdcard.sh index 
> ca02ad2..17ae564 100644
> --- a/create-sdcard.sh
> +++ b/create-sdcard.sh
> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then fi
> 
> THEPWD=$EXEPATH
> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
> 
> if [ "$PARSEPATH" != "" ] ; then
> PATHVALID=1
> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
> 
>    #check that in the right directory
> 
> -    THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +    THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
> 
>    if [ $PATHVALID -eq 1 ]; then
>    echo "now installing:  $THEEVMSDK"
> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>            then
>                echo "Directory exists"
>                echo ""
> -                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
> +                PARSEPATH=`echo $SDKFILEPATH | grep -o 
> + '.*processor-sdk.*.[0-9]/'`
>                #echo $PARSEPATH
> 
>                if [ "$PARSEPATH" != "" ] ; then @@ -731,7 +731,7 @@ if 
> [ $FILEPATHOPTION -eq 1 ] ; then
>                #echo $PATHVALID
>                if [ $PATHVALID -eq 1 ] ; then
> 
> -                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +                THEEVMSDK=`echo $SDKFILEPATH | grep -o 
> + 'processor-sdk-.*[0-9]'`
>                echo "Is this the correct SDK: $THEEVMSDK"
>                echo ""
>                read -p 'Is this correct? [y/n] : ' ISRIGHTPATH @@ 
> -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>                    *)  echo "Please enter y or n";ENTERCORRECTLY=0;;
>                    esac
>                else
> -                echo "Invalid SDK path make sure to include ti-sdk-xxxx"
> +                echo "Invalid SDK path make sure to include processor-sdk-xxxx"
>                ENTERCORRECTLY=0
>                fi
> 
> @@ -804,7 +804,7 @@ cat << EOM
>       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
> 
>   If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
> 
>   NOTE: Not all platforms will have an MLO file and this file can
>         be ignored for platforms that do not support an MLO.
> @@ -937,7 +937,7 @@ cat << EOM
>       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
> 
>   If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/targetNFS/
> +      e.x. $: /processor-sdk/targetNFS/
> 
> ######################################################################
> ##########
> 
> --
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

[-- Attachment #2: 0001-create-sdcard-Change-SDK-install-prefix-from-ti-sdk-.patch --]
[-- Type: application/octet-stream, Size: 2725 bytes --]

From 29c5da759e24434945a25d3e24bd805c522a235b Mon Sep 17 00:00:00 2001
From: Jacob Stiffler <j-stiffler@ti.com>
Date: Fri, 13 Mar 2015 09:47:01 -0400
Subject: [PATCH] create-sdcard: Change SDK install prefix from ti-sdk to
 processor-sdk.

Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
---
 create-sdcard.sh |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/create-sdcard.sh b/create-sdcard.sh
index ca02ad2..17ae564 100644
--- a/create-sdcard.sh
+++ b/create-sdcard.sh
@@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then
 fi
 
 THEPWD=$EXEPATH
-PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
+PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
 
 if [ "$PARSEPATH" != "" ] ; then
 PATHVALID=1
@@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 
 	#check that in the right directory
 
-	THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
+	THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
 
 	if [ $PATHVALID -eq 1 ]; then
 	echo "now installing:  $THEEVMSDK"
@@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 			then
 				echo "Directory exists"
 				echo ""
-				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
+				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*processor-sdk.*.[0-9]/'`
 				#echo $PARSEPATH
 
 				if [ "$PARSEPATH" != "" ] ; then
@@ -731,7 +731,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 				#echo $PATHVALID
 				if [ $PATHVALID -eq 1 ] ; then
 
-				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
+				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'processor-sdk-.*[0-9]'`
 				echo "Is this the correct SDK: $THEEVMSDK"
 				echo ""
 				read -p 'Is this correct? [y/n] : ' ISRIGHTPATH
@@ -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
 					*)  echo "Please enter y or n";ENTERCORRECTLY=0;;
 					esac
 				else
-				echo "Invalid SDK path make sure to include ti-sdk-xxxx"
+				echo "Invalid SDK path make sure to include processor-sdk-xxxx"
 				ENTERCORRECTLY=0
 				fi
 
@@ -804,7 +804,7 @@ cat << EOM
       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
 
   If files are located in a directory write the directory path
-      e.x. $: /ti-sdk/board-support/prebuilt-images/
+      e.x. $: /processor-sdk/board-support/prebuilt-images/
 
   NOTE: Not all platforms will have an MLO file and this file can
         be ignored for platforms that do not support an MLO.
@@ -937,7 +937,7 @@ cat << EOM
       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
 
   If files are located in a directory write the directory path
-      e.x. $: /ti-sdk/targetNFS/
+      e.x. $: /processor-sdk/targetNFS/
 
 ################################################################################
 
-- 
1.7.9.5


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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-13 13:49 [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk Jacob Stiffler
  2015-03-13 13:50 ` Cooper Jr., Franklin
@ 2015-03-13 13:57 ` Denys Dmytriyenko
  2015-03-13 14:02   ` Stiffler, Jacob
  1 sibling, 1 reply; 9+ messages in thread
From: Denys Dmytriyenko @ 2015-03-13 13:57 UTC (permalink / raw)
  To: Jacob Stiffler; +Cc: meta-arago

This is going to affect all the SDKs - ti-sdk was chosen on purpose to be 
generic enough to cover all our SDKs, not some specific one...


On Fri, Mar 13, 2015 at 09:49:04AM -0400, Jacob Stiffler wrote:
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  create-sdcard.sh |   14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/create-sdcard.sh b/create-sdcard.sh
> index ca02ad2..17ae564 100644
> --- a/create-sdcard.sh
> +++ b/create-sdcard.sh
> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then
>  fi
>  
>  THEPWD=$EXEPATH
> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
>  
>  if [ "$PARSEPATH" != "" ] ; then
>  PATHVALID=1
> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  
>  	#check that in the right directory
>  
> -	THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +	THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
>  
>  	if [ $PATHVALID -eq 1 ]; then
>  	echo "now installing:  $THEEVMSDK"
> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  			then
>  				echo "Directory exists"
>  				echo ""
> -				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
> +				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*processor-sdk.*.[0-9]/'`
>  				#echo $PARSEPATH
>  
>  				if [ "$PARSEPATH" != "" ] ; then
> @@ -731,7 +731,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  				#echo $PATHVALID
>  				if [ $PATHVALID -eq 1 ] ; then
>  
> -				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'processor-sdk-.*[0-9]'`
>  				echo "Is this the correct SDK: $THEEVMSDK"
>  				echo ""
>  				read -p 'Is this correct? [y/n] : ' ISRIGHTPATH
> @@ -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  					*)  echo "Please enter y or n";ENTERCORRECTLY=0;;
>  					esac
>  				else
> -				echo "Invalid SDK path make sure to include ti-sdk-xxxx"
> +				echo "Invalid SDK path make sure to include processor-sdk-xxxx"
>  				ENTERCORRECTLY=0
>  				fi
>  
> @@ -804,7 +804,7 @@ cat << EOM
>        e.x. $:  /home/user/MyCustomTars/boot.tar.gz
>  
>    If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
>  
>    NOTE: Not all platforms will have an MLO file and this file can
>          be ignored for platforms that do not support an MLO.
> @@ -937,7 +937,7 @@ cat << EOM
>        e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
>  
>    If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/targetNFS/
> +      e.x. $: /processor-sdk/targetNFS/
>  
>  ################################################################################
>  
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-13 13:57 ` Denys Dmytriyenko
@ 2015-03-13 14:02   ` Stiffler, Jacob
  2015-03-13 14:43     ` Cooper Jr., Franklin
  0 siblings, 1 reply; 9+ messages in thread
From: Stiffler, Jacob @ 2015-03-13 14:02 UTC (permalink / raw)
  To: Dmytriyenko, Denys; +Cc: meta-arago@arago-project.org

I'm not sure if we can do that now that the versioning has been reset to 1.0.0.0.

However, I do plan on submitting an updated patch after this first release. This method of autodetecting the SDK installation directory is somewhat broken with installbuilder since the user may choose any directory which may or may not have the correct format.

 It seems that the sdk_install.sh script should do some modifications to get the explicit directory within the script, but it appears this would take more changes which I haven't yet had a chance to look into.

-  Jake

-----Original Message-----
From: Dmytriyenko, Denys 
Sent: Friday, March 13, 2015 9:58 AM
To: Stiffler, Jacob
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.

This is going to affect all the SDKs - ti-sdk was chosen on purpose to be generic enough to cover all our SDKs, not some specific one...


On Fri, Mar 13, 2015 at 09:49:04AM -0400, Jacob Stiffler wrote:
> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> ---
>  create-sdcard.sh |   14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/create-sdcard.sh b/create-sdcard.sh
> index ca02ad2..17ae564 100644
> --- a/create-sdcard.sh
> +++ b/create-sdcard.sh
> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then
>  fi
>  
>  THEPWD=$EXEPATH
> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
>  
>  if [ "$PARSEPATH" != "" ] ; then
>  PATHVALID=1
> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  
>  	#check that in the right directory
>  
> -	THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +	THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
>  
>  	if [ $PATHVALID -eq 1 ]; then
>  	echo "now installing:  $THEEVMSDK"
> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  			then
>  				echo "Directory exists"
>  				echo ""
> -				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
> +				PARSEPATH=`echo $SDKFILEPATH | grep -o '.*processor-sdk.*.[0-9]/'`
>  				#echo $PARSEPATH
>  
>  				if [ "$PARSEPATH" != "" ] ; then
> @@ -731,7 +731,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  				#echo $PATHVALID
>  				if [ $PATHVALID -eq 1 ] ; then
>  
> -				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
> +				THEEVMSDK=`echo $SDKFILEPATH | grep -o 'processor-sdk-.*[0-9]'`
>  				echo "Is this the correct SDK: $THEEVMSDK"
>  				echo ""
>  				read -p 'Is this correct? [y/n] : ' ISRIGHTPATH
> @@ -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>  					*)  echo "Please enter y or n";ENTERCORRECTLY=0;;
>  					esac
>  				else
> -				echo "Invalid SDK path make sure to include ti-sdk-xxxx"
> +				echo "Invalid SDK path make sure to include processor-sdk-xxxx"
>  				ENTERCORRECTLY=0
>  				fi
>  
> @@ -804,7 +804,7 @@ cat << EOM
>        e.x. $:  /home/user/MyCustomTars/boot.tar.gz
>  
>    If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
>  
>    NOTE: Not all platforms will have an MLO file and this file can
>          be ignored for platforms that do not support an MLO.
> @@ -937,7 +937,7 @@ cat << EOM
>        e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
>  
>    If files are located in a directory write the directory path
> -      e.x. $: /ti-sdk/targetNFS/
> +      e.x. $: /processor-sdk/targetNFS/
>  
>  ################################################################################
>  
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-13 14:02   ` Stiffler, Jacob
@ 2015-03-13 14:43     ` Cooper Jr., Franklin
  2015-03-13 17:06       ` Stiffler, Jacob
  0 siblings, 1 reply; 9+ messages in thread
From: Cooper Jr., Franklin @ 2015-03-13 14:43 UTC (permalink / raw)
  To: Stiffler, Jacob; +Cc: meta-arago@arago-project.org

Well I guess Denys is right and we should play nice with others.

Sdk_install.sh part would be easy.

Replace  "*ti-sdk.*.[0-9]/" with the name of the directory since you already know it.

Then it works for anyone 




> On Mar 13, 2015, at 9:02 AM, Stiffler, Jacob <j-stiffler@ti.com> wrote:
> 
> I'm not sure if we can do that now that the versioning has been reset to 1.0.0.0.
> 
> However, I do plan on submitting an updated patch after this first release. This method of autodetecting the SDK installation directory is somewhat broken with installbuilder since the user may choose any directory which may or may not have the correct format.
> 
> It seems that the sdk_install.sh script should do some modifications to get the explicit directory within the script, but it appears this would take more changes which I haven't yet had a chance to look into.
> 
> -  Jake
> 
> -----Original Message-----
> From: Dmytriyenko, Denys 
> Sent: Friday, March 13, 2015 9:58 AM
> To: Stiffler, Jacob
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
> 
> This is going to affect all the SDKs - ti-sdk was chosen on purpose to be generic enough to cover all our SDKs, not some specific one...
> 
> 
>> On Fri, Mar 13, 2015 at 09:49:04AM -0400, Jacob Stiffler wrote:
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>> ---
>> create-sdcard.sh |   14 +++++++-------
>> 1 file changed, 7 insertions(+), 7 deletions(-)
>> 
>> diff --git a/create-sdcard.sh b/create-sdcard.sh
>> index ca02ad2..17ae564 100644
>> --- a/create-sdcard.sh
>> +++ b/create-sdcard.sh
>> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then
>> fi
>> 
>> THEPWD=$EXEPATH
>> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
>> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
>> 
>> if [ "$PARSEPATH" != "" ] ; then
>> PATHVALID=1
>> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>> 
>>    #check that in the right directory
>> 
>> -    THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
>> +    THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
>> 
>>    if [ $PATHVALID -eq 1 ]; then
>>    echo "now installing:  $THEEVMSDK"
>> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>>            then
>>                echo "Directory exists"
>>                echo ""
>> -                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
>> +                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*processor-sdk.*.[0-9]/'`
>>                #echo $PARSEPATH
>> 
>>                if [ "$PARSEPATH" != "" ] ; then
>> @@ -731,7 +731,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>>                #echo $PATHVALID
>>                if [ $PATHVALID -eq 1 ] ; then
>> 
>> -                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
>> +                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'processor-sdk-.*[0-9]'`
>>                echo "Is this the correct SDK: $THEEVMSDK"
>>                echo ""
>>                read -p 'Is this correct? [y/n] : ' ISRIGHTPATH
>> @@ -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>>                    *)  echo "Please enter y or n";ENTERCORRECTLY=0;;
>>                    esac
>>                else
>> -                echo "Invalid SDK path make sure to include ti-sdk-xxxx"
>> +                echo "Invalid SDK path make sure to include processor-sdk-xxxx"
>>                ENTERCORRECTLY=0
>>                fi
>> 
>> @@ -804,7 +804,7 @@ cat << EOM
>>       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
>> 
>>   If files are located in a directory write the directory path
>> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
>> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
>> 
>>   NOTE: Not all platforms will have an MLO file and this file can
>>         be ignored for platforms that do not support an MLO.
>> @@ -937,7 +937,7 @@ cat << EOM
>>       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
>> 
>>   If files are located in a directory write the directory path
>> -      e.x. $: /ti-sdk/targetNFS/
>> +      e.x. $: /processor-sdk/targetNFS/
>> 
>> ################################################################################
>> 
>> -- 
>> 1.7.9.5
>> 
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-13 14:43     ` Cooper Jr., Franklin
@ 2015-03-13 17:06       ` Stiffler, Jacob
  2015-03-17 18:23         ` Stiffler, Jacob
  0 siblings, 1 reply; 9+ messages in thread
From: Stiffler, Jacob @ 2015-03-13 17:06 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org

Yes, but there is a lot of extra logic which would not be needed.

-----Original Message-----
From: Cooper Jr., Franklin 
Sent: Friday, March 13, 2015 10:43 AM
To: Stiffler, Jacob
Cc: Dmytriyenko, Denys; meta-arago@arago-project.org
Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.

Well I guess Denys is right and we should play nice with others.

Sdk_install.sh part would be easy.

Replace  "*ti-sdk.*.[0-9]/" with the name of the directory since you already know it.

Then it works for anyone 




> On Mar 13, 2015, at 9:02 AM, Stiffler, Jacob <j-stiffler@ti.com> wrote:
> 
> I'm not sure if we can do that now that the versioning has been reset to 1.0.0.0.
> 
> However, I do plan on submitting an updated patch after this first release. This method of autodetecting the SDK installation directory is somewhat broken with installbuilder since the user may choose any directory which may or may not have the correct format.
> 
> It seems that the sdk_install.sh script should do some modifications to get the explicit directory within the script, but it appears this would take more changes which I haven't yet had a chance to look into.
> 
> -  Jake
> 
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Friday, March 13, 2015 9:58 AM
> To: Stiffler, Jacob
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
> 
> This is going to affect all the SDKs - ti-sdk was chosen on purpose to be generic enough to cover all our SDKs, not some specific one...
> 
> 
>> On Fri, Mar 13, 2015 at 09:49:04AM -0400, Jacob Stiffler wrote:
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>> ---
>> create-sdcard.sh |   14 +++++++-------
>> 1 file changed, 7 insertions(+), 7 deletions(-)
>> 
>> diff --git a/create-sdcard.sh b/create-sdcard.sh index 
>> ca02ad2..17ae564 100644
>> --- a/create-sdcard.sh
>> +++ b/create-sdcard.sh
>> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then fi
>> 
>> THEPWD=$EXEPATH
>> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
>> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
>> 
>> if [ "$PARSEPATH" != "" ] ; then
>> PATHVALID=1
>> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>> 
>>    #check that in the right directory
>> 
>> -    THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
>> +    THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
>> 
>>    if [ $PATHVALID -eq 1 ]; then
>>    echo "now installing:  $THEEVMSDK"
>> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>>            then
>>                echo "Directory exists"
>>                echo ""
>> -                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
>> +                PARSEPATH=`echo $SDKFILEPATH | grep -o 
>> + '.*processor-sdk.*.[0-9]/'`
>>                #echo $PARSEPATH
>> 
>>                if [ "$PARSEPATH" != "" ] ; then @@ -731,7 +731,7 @@ 
>> if [ $FILEPATHOPTION -eq 1 ] ; then
>>                #echo $PATHVALID
>>                if [ $PATHVALID -eq 1 ] ; then
>> 
>> -                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
>> +                THEEVMSDK=`echo $SDKFILEPATH | grep -o 
>> + 'processor-sdk-.*[0-9]'`
>>                echo "Is this the correct SDK: $THEEVMSDK"
>>                echo ""
>>                read -p 'Is this correct? [y/n] : ' ISRIGHTPATH @@ 
>> -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>>                    *)  echo "Please enter y or n";ENTERCORRECTLY=0;;
>>                    esac
>>                else
>> -                echo "Invalid SDK path make sure to include ti-sdk-xxxx"
>> +                echo "Invalid SDK path make sure to include processor-sdk-xxxx"
>>                ENTERCORRECTLY=0
>>                fi
>> 
>> @@ -804,7 +804,7 @@ cat << EOM
>>       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
>> 
>>   If files are located in a directory write the directory path
>> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
>> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
>> 
>>   NOTE: Not all platforms will have an MLO file and this file can
>>         be ignored for platforms that do not support an MLO.
>> @@ -937,7 +937,7 @@ cat << EOM
>>       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
>> 
>>   If files are located in a directory write the directory path
>> -      e.x. $: /ti-sdk/targetNFS/
>> +      e.x. $: /processor-sdk/targetNFS/
>> 
>> #####################################################################
>> ###########
>> 
>> --
>> 1.7.9.5
>> 
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-13 17:06       ` Stiffler, Jacob
@ 2015-03-17 18:23         ` Stiffler, Jacob
  2015-03-17 19:28           ` Cooper Jr., Franklin
  0 siblings, 1 reply; 9+ messages in thread
From: Stiffler, Jacob @ 2015-03-17 18:23 UTC (permalink / raw)
  To: Cooper Jr., Franklin; +Cc: meta-arago@arago-project.org

I've begun looking into the changes needed to modifying the prefix during installation, and I'm not sure quite what to do. Why does this detection need to happen? 

I guess for installing images from the SDK, I can get that patch from installation, but if user wants to enter another SDK location, is there any need to verify that the directory has the assumed format? Can the script return an error if the images cannot be found within that directory?

-----Original Message-----
From: meta-arago-bounces@arago-project.org [mailto:meta-arago-bounces@arago-project.org] On Behalf Of Stiffler, Jacob
Sent: Friday, March 13, 2015 1:07 PM
To: Cooper Jr., Franklin
Cc: meta-arago@arago-project.org
Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.

Yes, but there is a lot of extra logic which would not be needed.

-----Original Message-----
From: Cooper Jr., Franklin
Sent: Friday, March 13, 2015 10:43 AM
To: Stiffler, Jacob
Cc: Dmytriyenko, Denys; meta-arago@arago-project.org
Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.

Well I guess Denys is right and we should play nice with others.

Sdk_install.sh part would be easy.

Replace  "*ti-sdk.*.[0-9]/" with the name of the directory since you already know it.

Then it works for anyone 




> On Mar 13, 2015, at 9:02 AM, Stiffler, Jacob <j-stiffler@ti.com> wrote:
> 
> I'm not sure if we can do that now that the versioning has been reset to 1.0.0.0.
> 
> However, I do plan on submitting an updated patch after this first release. This method of autodetecting the SDK installation directory is somewhat broken with installbuilder since the user may choose any directory which may or may not have the correct format.
> 
> It seems that the sdk_install.sh script should do some modifications to get the explicit directory within the script, but it appears this would take more changes which I haven't yet had a chance to look into.
> 
> -  Jake
> 
> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Friday, March 13, 2015 9:58 AM
> To: Stiffler, Jacob
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
> 
> This is going to affect all the SDKs - ti-sdk was chosen on purpose to be generic enough to cover all our SDKs, not some specific one...
> 
> 
>> On Fri, Mar 13, 2015 at 09:49:04AM -0400, Jacob Stiffler wrote:
>> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
>> ---
>> create-sdcard.sh |   14 +++++++-------
>> 1 file changed, 7 insertions(+), 7 deletions(-)
>> 
>> diff --git a/create-sdcard.sh b/create-sdcard.sh index
>> ca02ad2..17ae564 100644
>> --- a/create-sdcard.sh
>> +++ b/create-sdcard.sh
>> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then fi
>> 
>> THEPWD=$EXEPATH
>> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
>> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
>> 
>> if [ "$PARSEPATH" != "" ] ; then
>> PATHVALID=1
>> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>> 
>>    #check that in the right directory
>> 
>> -    THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
>> +    THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
>> 
>>    if [ $PATHVALID -eq 1 ]; then
>>    echo "now installing:  $THEEVMSDK"
>> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>>            then
>>                echo "Directory exists"
>>                echo ""
>> -                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
>> +                PARSEPATH=`echo $SDKFILEPATH | grep -o 
>> + '.*processor-sdk.*.[0-9]/'`
>>                #echo $PARSEPATH
>> 
>>                if [ "$PARSEPATH" != "" ] ; then @@ -731,7 +731,7 @@ 
>> if [ $FILEPATHOPTION -eq 1 ] ; then
>>                #echo $PATHVALID
>>                if [ $PATHVALID -eq 1 ] ; then
>> 
>> -                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
>> +                THEEVMSDK=`echo $SDKFILEPATH | grep -o 
>> + 'processor-sdk-.*[0-9]'`
>>                echo "Is this the correct SDK: $THEEVMSDK"
>>                echo ""
>>                read -p 'Is this correct? [y/n] : ' ISRIGHTPATH @@
>> -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
>>                    *)  echo "Please enter y or n";ENTERCORRECTLY=0;;
>>                    esac
>>                else
>> -                echo "Invalid SDK path make sure to include ti-sdk-xxxx"
>> +                echo "Invalid SDK path make sure to include processor-sdk-xxxx"
>>                ENTERCORRECTLY=0
>>                fi
>> 
>> @@ -804,7 +804,7 @@ cat << EOM
>>       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
>> 
>>   If files are located in a directory write the directory path
>> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
>> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
>> 
>>   NOTE: Not all platforms will have an MLO file and this file can
>>         be ignored for platforms that do not support an MLO.
>> @@ -937,7 +937,7 @@ cat << EOM
>>       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
>> 
>>   If files are located in a directory write the directory path
>> -      e.x. $: /ti-sdk/targetNFS/
>> +      e.x. $: /processor-sdk/targetNFS/
>> 
>> #####################################################################
>> ###########
>> 
>> --
>> 1.7.9.5
>> 
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
_______________________________________________
meta-arago mailing list
meta-arago@arago-project.org
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


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

* Re: [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk.
  2015-03-17 18:23         ` Stiffler, Jacob
@ 2015-03-17 19:28           ` Cooper Jr., Franklin
  0 siblings, 0 replies; 9+ messages in thread
From: Cooper Jr., Franklin @ 2015-03-17 19:28 UTC (permalink / raw)
  To: Stiffler, Jacob; +Cc: meta-arago@arago-project.org



> -----Original Message-----
> From: Stiffler, Jacob
> Sent: Tuesday, March 17, 2015 1:23 PM
> To: Cooper Jr., Franklin
> Cc: meta-arago@arago-project.org
> Subject: RE: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard:
> Change SDK install prefix from ti-sdk to processor-sdk.
> 
> I've begun looking into the changes needed to modifying the prefix during
> installation, and I'm not sure quite what to do. Why does this detection need
> to happen?
[Franklin] It probably wants to give you feedback early that you are not using the script within the SDK directory aka most likely doesn't contain the files you need.. The script probably needs some restructuring if you want to wait until the last moment to verify that the correct directory is used and if it fails jump back to the portion of the script to allow the user to try again or specify another path.

> 
> I guess for installing images from the SDK, I can get that patch from
> installation, but if user wants to enter another SDK location, is there any
> need to verify that the directory has the assumed format? Can the script
> return an error if the images cannot be found within that directory?
[Franklin] The script already has the ability to allow you to manually specify the directory to grab tarball and binaries. 
However, if you choose the option to use the prebuilt images from the SDK then the assumption is that you want the prebuilt from the current sdk. If not run the create sd card script located in the other SDK or select the option within the script that will ask you the individual folder name of the rootfs, uboot,kernel and dtb.

Honestly I've heard probably three times in three years someone runs into an issue using the script since they change the name of the SDK directory during installation.  So it is indeed a problem but it is so infrequent that myself and others haven't remembered to fix it. Which is essentially the problem your having.


> 
> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Stiffler, Jacob
> Sent: Friday, March 13, 2015 1:07 PM
> To: Cooper Jr., Franklin
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard:
> Change SDK install prefix from ti-sdk to processor-sdk.
> 
> Yes, but there is a lot of extra logic which would not be needed.
> 
> -----Original Message-----
> From: Cooper Jr., Franklin
> Sent: Friday, March 13, 2015 10:43 AM
> To: Stiffler, Jacob
> Cc: Dmytriyenko, Denys; meta-arago@arago-project.org
> Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard:
> Change SDK install prefix from ti-sdk to processor-sdk.
> 
> Well I guess Denys is right and we should play nice with others.
> 
> Sdk_install.sh part would be easy.
> 
> Replace  "*ti-sdk.*.[0-9]/" with the name of the directory since you already
> know it.
> 
> Then it works for anyone
> 
> 
> 
> 
> > On Mar 13, 2015, at 9:02 AM, Stiffler, Jacob <j-stiffler@ti.com> wrote:
> >
> > I'm not sure if we can do that now that the versioning has been reset to
> 1.0.0.0.
> >
> > However, I do plan on submitting an updated patch after this first release.
> This method of autodetecting the SDK installation directory is somewhat
> broken with installbuilder since the user may choose any directory which may
> or may not have the correct format.
> >
> > It seems that the sdk_install.sh script should do some modifications to get
> the explicit directory within the script, but it appears this would take more
> changes which I haven't yet had a chance to look into.
> >
> > -  Jake
> >
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Friday, March 13, 2015 9:58 AM
> > To: Stiffler, Jacob
> > Cc: meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [tisdk-setup-scripts][PATCH] create-sdcard:
> Change SDK install prefix from ti-sdk to processor-sdk.
> >
> > This is going to affect all the SDKs - ti-sdk was chosen on purpose to be
> generic enough to cover all our SDKs, not some specific one...
> >
> >
> >> On Fri, Mar 13, 2015 at 09:49:04AM -0400, Jacob Stiffler wrote:
> >> Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> >> ---
> >> create-sdcard.sh |   14 +++++++-------
> >> 1 file changed, 7 insertions(+), 7 deletions(-)
> >>
> >> diff --git a/create-sdcard.sh b/create-sdcard.sh index
> >> ca02ad2..17ae564 100644
> >> --- a/create-sdcard.sh
> >> +++ b/create-sdcard.sh
> >> @@ -74,7 +74,7 @@ if [ "$AMIROOT" != "root" ] ; then fi
> >>
> >> THEPWD=$EXEPATH
> >> -PARSEPATH=`echo $THEPWD | grep -o '.*ti-sdk.*.[0-9]/'`
> >> +PARSEPATH=`echo $THEPWD | grep -o '.*processor-sdk.*.[0-9]/'`
> >>
> >> if [ "$PARSEPATH" != "" ] ; then
> >> PATHVALID=1
> >> @@ -703,7 +703,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
> >>
> >>    #check that in the right directory
> >>
> >> -    THEEVMSDK=`echo $PARSEPATH | grep -o 'ti-sdk-.*[0-9]'`
> >> +    THEEVMSDK=`echo $PARSEPATH | grep -o 'processor-sdk-.*[0-9]'`
> >>
> >>    if [ $PATHVALID -eq 1 ]; then
> >>    echo "now installing:  $THEEVMSDK"
> >> @@ -720,7 +720,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
> >>            then
> >>                echo "Directory exists"
> >>                echo ""
> >> -                PARSEPATH=`echo $SDKFILEPATH | grep -o '.*ti-sdk.*.[0-9]/'`
> >> +                PARSEPATH=`echo $SDKFILEPATH | grep -o
> >> + '.*processor-sdk.*.[0-9]/'`
> >>                #echo $PARSEPATH
> >>
> >>                if [ "$PARSEPATH" != "" ] ; then @@ -731,7 +731,7 @@
> >> if [ $FILEPATHOPTION -eq 1 ] ; then
> >>                #echo $PATHVALID
> >>                if [ $PATHVALID -eq 1 ] ; then
> >>
> >> -                THEEVMSDK=`echo $SDKFILEPATH | grep -o 'ti-sdk-.*[0-9]'`
> >> +                THEEVMSDK=`echo $SDKFILEPATH | grep -o
> >> + 'processor-sdk-.*[0-9]'`
> >>                echo "Is this the correct SDK: $THEEVMSDK"
> >>                echo ""
> >>                read -p 'Is this correct? [y/n] : ' ISRIGHTPATH @@
> >> -741,7 +741,7 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
> >>                    *)  echo "Please enter y or n";ENTERCORRECTLY=0;;
> >>                    esac
> >>                else
> >> -                echo "Invalid SDK path make sure to include ti-sdk-xxxx"
> >> +                echo "Invalid SDK path make sure to include processor-sdk-xxxx"
> >>                ENTERCORRECTLY=0
> >>                fi
> >>
> >> @@ -804,7 +804,7 @@ cat << EOM
> >>       e.x. $:  /home/user/MyCustomTars/boot.tar.gz
> >>
> >>   If files are located in a directory write the directory path
> >> -      e.x. $: /ti-sdk/board-support/prebuilt-images/
> >> +      e.x. $: /processor-sdk/board-support/prebuilt-images/
> >>
> >>   NOTE: Not all platforms will have an MLO file and this file can
> >>         be ignored for platforms that do not support an MLO.
> >> @@ -937,7 +937,7 @@ cat << EOM
> >>       e.x. $:  /home/user/MyCustomTars/rootfs.tar.gz
> >>
> >>   If files are located in a directory write the directory path
> >> -      e.x. $: /ti-sdk/targetNFS/
> >> +      e.x. $: /processor-sdk/targetNFS/
> >>
> >>
> ##########################################################
> ###########
> >> ###########
> >>
> >> --
> >> 1.7.9.5
> >>
> >> _______________________________________________
> >> meta-arago mailing list
> >> meta-arago@arago-project.org
> >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


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

end of thread, other threads:[~2015-03-17 19:28 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-13 13:49 [tisdk-setup-scripts][PATCH] create-sdcard: Change SDK install prefix from ti-sdk to processor-sdk Jacob Stiffler
2015-03-13 13:50 ` Cooper Jr., Franklin
2015-03-13 13:56   ` Stiffler, Jacob
2015-03-13 13:57 ` Denys Dmytriyenko
2015-03-13 14:02   ` Stiffler, Jacob
2015-03-13 14:43     ` Cooper Jr., Franklin
2015-03-13 17:06       ` Stiffler, Jacob
2015-03-17 18:23         ` Stiffler, Jacob
2015-03-17 19:28           ` Cooper Jr., Franklin

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.