fwts V0.25.04 release review

[Colin Ian King]

Keng-Yu,

I've cloned both fwts and fwts-test repos, did a clean build of fwts
and run it against the tests and I get no failures. See attached log.

How are you building and testing?



On 21/06/12 08:45, Keng-Yu Lin wrote:
> Dear all:
>    I will start from this release to send out the fwts-test log against
> the version of fwts about to release for reviewing.
>    It is a hope that at the stage that everyone focuses on seeing the
> failure, giving comments, and fixing them if necessarily.
>
>    We slipped the V0.25.03 because it cannot pass the fwts-test tests.
> So I hope this "guard procedure" can help us prevent this from
> happening again.
>
>    I will expect ACKs too. I will propose two ACKs as the normal patch
> acceptance standard. But any comment/suggestion to the procedure is
> welcomed as always.
>
>    fwts patches (V0.25.03..V0.25.04-rc1)
>
>    lib: fwts_acpi_tables: load tables in deterministically
>    lib: fwts_log: handle special logfile names
>    lib: log: move line counting to logging back-ends
>    lib: fwts_log: remove fwts_log_line_number(), no longer used
>    lib: fwts_framework: output all logfile names being written to
>    lib: fwts_log: add fwts_log_get_filenames()
>    lib: fwts_summary: remove references to log line number
>
>
>    fwts-test patches (V0.25.03..V0.25.04-rc1)
>
>    arg-table-path-0001: update log to now that table loading is done
> deterministically
>    arg-results-0001: use more sensible log name
>    dmesg_common-0001: update due to summary output change
>    klog-0001: update due to summary output change
>    oops-0001: minor changes to summary output
>    arg-log-format-0001: update to match new log fields
>
>    fwts-test failure.log is in the attchment.
>
>    cheers,
> -kengyu
>
>
>

-------------- next part --------------
A non-text attachment was scrubbed...
Name: results.log
Type: text/x-log
Size: 5389 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20120621/c5c44a70/attachment.bin>