[PATCH 2/2] live-image/fwts-frontend-text: conditional added to done_tests_finish() which creates a symbolic link to the latest results.html at the top level WORK_DIR regardless of booting via casper (fwts-live) or running post-installation. The

Chris Van Hoof vanhoof at canonical.com
Wed Jun 13 19:28:29 UTC 2012


On 06/13/2012 06:12 AM, Keng-Yu Lin wrote:
> On Wed, Jun 13, 2012 at 5:58 PM, Colin Ian King
> <colin.king at canonical.com> wrote:
>> On 12/06/12 00:02, Chris Van Hoof wrote:
>>>
>>> Signed-off-by: Chris Van Hoof <vanhoof at canonical.com>
>>> ---
>>>  live-image/fwts-frontend-text |    7 +++++++
>>>  1 file changed, 7 insertions(+)
>>>
>>> diff --git a/live-image/fwts-frontend-text b/live-image/fwts-frontend-text
>>> index 90c0fc7..932c068 100755
>>> --- a/live-image/fwts-frontend-text
>>> +++ b/live-image/fwts-frontend-text
>>> @@ -195,6 +195,13 @@ no_tests_finish()
>>>  #
>>>  done_tests_finish()
>>>  {
>>> +    if [ -n "${CASPER_DETECTED:+x}" ]; then
>>> +        cd $WORK_DIR
>>> +        ln -sf ./results.html ../../results.html
>>> +    else
>>> +        ln -sf $WORK_DIR/results.html /fwts/results.html
>>> +    fi
>>> +
>>>        dialog  --backtitle "$FWTS" --title "Testing Complete" --yesno \
>>>  "The results can be found on the USB stick in the\n
>>>  the directory: /fwts/$FWTS_DATE/$FWTS_TIME/results.log\n\n
>>>
>>
>> Perhaps this patch can be re-formatted before it's included into the fwts
>> git repo with a shorter subject and the rest of the subject line in the
>> patch description.  Keng-Yu are you OK to do this?
>>
>> Colin
>>
> 
> Yes, I can do this before applying. I am also okay if Chris like to
> send his own version.
> Both works for me, so,

Keng-yu -- Feel free to edit it since I'm heading out of town soon:

"live-image/fwts-frontend-text: conditional added to done_tests_finish()
to link latests results.html to WORK_DIR"

... or something like that will work.

> Acked-by: Keng-Yu Lin <kengyu at canonical.com>
> 





More information about the fwts-devel mailing list