[yocto] yocto_installer test script update

[Ke, Liping]

> -----Original Message-----
> From: Zhang, Jessica
> Sent: Thursday, December 02, 2010 8:19 AM
> To: Ke, Liping; Lu, Lianhao; Tian, Kevin
> Cc: yocto at yoctoproject.org
> Subject: RE: yocto_installer test script update
> 
> Liping,
> 
> I've modified couple files mainly for the messaging, styling, etc.
> Also, globaly changed SDK to ADT (Application Development Tools, which
> is the new name that came out refering to the toolchains, etc.).
> Please see attachment.
> 
> Other comments:
> 1. I'd expect after your do the code clean up, the code logic flow
> should be more clearer, now the functions and main installation  code
> are mingled together 
Yes, functions will be put on the header and the mainbody will be put behind surely!


2. There're couple places that I commented off
> with my initial JZ which I don't think there're the needs for the code,
> we can walk through them tonight...

Ok.

> 3. For target sysroot setup, I'd expect similar prompt to the user
> about the specified configuration and confirmation

OK, the image to be extracted, user_nfs_home_dir and the architecture will
be printed and ask whether user will go on extracting... Also if the folder
exists there, other prompt will show. Is it enough?


 4. Also for target
> sysroot setup, in the yocto_config file, you specify
> ROOTFS_powerpc="sdk"
> TARGET_SYSROOT_IMAGE_powerpc="sato"

> I'd expect the installer will run into error since the download images
> only contain SDK no sato, sysroot can't be setup using sato, which I
> think it's missing from the current code...
A function called validate_config will be added to validate the user configurations
before installation. I will cover this requirement in the validation function.

> 
> In general it looks good and main flow that needs to be covered are
> there...
> 
> Thanks,
> Jessica
> 
> 
> -----Original Message-----
> From: Ke, Liping
> Sent: Wednesday, December 01, 2010 2:06 AM
> To: Zhang, Jessica; Lu, Lianhao; Tian, Kevin
> Cc: yocto at yoctoproject.org
> Subject: yocto_installer test script update
> 
> Hi, Jessica
> 
> The newer version of the installer is in lianhao's location. You can
> directly run ./yocto_installer http://llu-piketon.sh.intel.com/ Today
> we tested both on 32 bit and 64 bit ubuntu host.
> 
> It's very initial and rough, I will move many non-root required ops
> into the outer wrapper script later.
> 
> Any early comments are welcomed, just let me know.
> 
> Thanks& Regards,
> criping
>