Skip to content

Feature add: modify dlog* message output to more clean up - #12

Merged
xiulipan merged 1 commit into
thesofproject:masterfrom
Bin-QA:logging_ctl
Jan 6, 2020
Merged

xiulipan merged 1 commit into
thesofproject:masterfrom
Bin-QA:logging_ctl

Conversation

@Bin-QA

@Bin-QA Bin-QA commented Dec 24, 2019

Copy link
Copy Markdown
Contributor

Current we support the test-case scripot load other test-case script
to finish the test-case, but from the test-case output information
the current script output will mix with sub-test-case script

Now add the 'Sub-Test' flag to the sub-test-case
when end-user catch the terminal output without confuse
for the script process

it will be helpful for maintenance the script

Signed-off-by: Wu, BinX binx.wu@intel.com

Comment thread case-lib/logging_ctl.sh Resolved
Comment thread case-lib/logging_ctl.sh Resolved
@Bin-QA

Bin-QA commented Jan 3, 2020

Copy link
Copy Markdown
Contributor Author

Force update for rebase

@Bin-QA

Bin-QA commented Jan 6, 2020

Copy link
Copy Markdown
Contributor Author

@libinyang @xiulipan @aiChaoSONG @keqiaozhang
Please help to review this patch

@xiulipan xiulipan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@aiChaoSONG @keqiaozhang @libinyang Please help to review.

Comment thread case-lib/logging_ctl.sh Resolved
Current we support the test-case scripot load other test-case script
to finish the test-case, but from the test-case output information
the current script output will mix with sub-test-case script

Now add the 'Sub-Test' flag to the sub-test-case
when end-user catch the terminal output without confuse
for the script process

it will be helpful for maintenance the script

Signed-off-by: Wu, BinX <binx.wu@intel.com>
@xiulipan
xiulipan merged commit 167a1c2 into thesofproject:master Jan 6, 2020
@Bin-QA
Bin-QA deleted the logging_ctl branch January 14, 2020 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants