Re: [tisdk-build-scripts][PATCH 5/5] arago-core-tisdk-dylan-puget: update to use new CLI

[Denys Dmytriyenko <denys@ti.com>]

On Tue, Mar 18, 2014 at 04:00:18PM -0400, Cooper Jr., Franklin wrote:
> 
> 
> > On Mar 18, 2014, at 9:29 AM, "Dmytriyenko, Denys" <denys@ti.com> wrote:
> > 
> >> On Tue, Mar 18, 2014 at 10:25:43AM -0400, Denys Dmytriyenko wrote:
> >>> On Tue, Mar 18, 2014 at 02:20:28PM +0000, Cooper Jr., Franklin wrote:
> >>> 
> >>> 
> >>>> -----Original Message-----
> >>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >>>> bounces@arago-project.org] On Behalf Of Denys Dmytriyenko
> >>>> Sent: Monday, March 03, 2014 5:36 PM
> >>>> To: meta-arago@arago-project.org
> >>>> Subject: [meta-arago] [tisdk-build-scripts][PATCH 5/5] arago-core-tisdk-dylan-
> >>>> puget: update to use new CLI
> >>>> 
> >>>> From: Denys Dmytriyenko <denys@ti.com>
> >>>> 
> >>>> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> >>>> ---
> >>>> configs/arago-core-tisdk-dylan-puget.txt |    4 ++--
> >>>> 1 file changed, 2 insertions(+), 2 deletions(-)
> >>>> 
> >>>> diff --git a/configs/arago-core-tisdk-dylan-puget.txt b/configs/arago-core-tisdk-
> >>>> dylan-puget.txt
> >>>> index f4ccb26..e24ab70 100644
> >>>> --- a/configs/arago-core-tisdk-dylan-puget.txt
> >>>> +++ b/configs/arago-core-tisdk-dylan-puget.txt
> >>>> @@ -164,11 +164,11 @@ TEST_LOCAL_IFACE="eth1"
> >>>> 
> >>>> # This variable contains the tarball name for the opentest CLI which is  #
> >>>> required to do target side testing.
> >>>> -OPENTEST_CLI="$SCRIPTS_ROOT/opentest_cli_0.7.tar.gz"
> >>>> +OPENTEST_CLI="$SCRIPTS_ROOT/linux-devtest.tgz"
> >>> [Franklin] Would it make sense to make this update to all the different configs?
> >> 
> >> Shouldn't be a problem. At the time of submitting these patches, there were 
> >> other issues with Opentest and no tests were working out of the box. Hence I 
> >> only updated my config for further testing. Most of the issues have been fixed 
> >> since then, so we can update other configs as well. But I made the changes to 
> >> be incremental,
> > 
> > Let me rephrase this to be more clear:
> > 
> >> so it should work with the old Opentest CLI, if you setup 
> >> requires it. Unfortunately, it no longer works with TIGT...
> > 
> > If your setup requires the use of the old Opentest CLI, it should still work. 
> > The old CLI no longer works in TIGT. But I'm fine with updating all configs to 
> > switch to the new CLI.
> 
> Thanks for explaining. I'm not sure of the status of our TEE in Dallas so 
> I'm ok with this patch as is. I'll update my team's config if I need to 
> later

Franklin, have you had a chance to look over the rest of the patches in this 
series?


> >>>> # This variable specifies the IP address of the TEE to be used with OpenTest  #
> >>>> Defaults to 128.247.105.11 if not set -OPENTEST_TEE_IP="10.218.102.142"
> >>>> +OPENTEST_TEE_IP="none"
> >>>> 
> >>>> # This variable specifies the hardware capabilities to be used with OpenTest
> >>>> # Capabilities are separated by underscores, some excamples are here:
> >>>> --
> >>>> 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