diff mbox series

[1/1] doc: old: Update TST_SR_IGNORE_ERR description

Message ID 20250515070217.453359-1-pvorel@suse.cz
State Accepted
Headers show
Series [1/1] doc: old: Update TST_SR_IGNORE_ERR description | expand

Checks

Context Check Description
ltpci/debian_stable_s390x-linux-gnu-gcc_s390x success success
ltpci/debian_stable_aarch64-linux-gnu-gcc_arm64 success success
ltpci/debian_stable_powerpc64le-linux-gnu-gcc_ppc64el success success
ltpci/debian_stable_gcc success success
ltpci/debian_stable_gcc success success
ltpci/ubuntu_jammy_gcc success success
ltpci/fedora_latest_clang success success
ltpci/debian_testing_gcc success success
ltpci/debian_oldstable_clang success success
ltpci/quay-io-centos-centos_stream9_gcc success success
ltpci/alpine_latest_gcc success success
ltpci/opensuse-leap_latest_gcc success success
ltpci/debian_testing_clang success success
ltpci/ubuntu_bionic_gcc success success
ltpci/debian_oldstable_gcc success success
ltpci/opensuse-archive_42-2_gcc success success

Commit Message

Petr Vorel May 15, 2025, 7:02 a.m. UTC 
TST_SR_IGNORE_ERR has a broader meaning, it ignores all errors (e.g.
when file does not exists during reading).

Reported-by: Aaron Knobloch <aarondevelops@gmail.com>
Closes: https://github.com/linux-test-project/ltp/issues/1239
Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
Hi all,

@Li I plan to add include/tst_sys_conf.h to our doc and use these
description in kerneldoc, but better to update the old doc in case I
forget.

Kind regards,
Petr

 doc/old/C-Test-API.asciidoc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Li Wang May 15, 2025, 7:40 a.m. UTC | #1 
On Thu, May 15, 2025 at 3:02 PM Petr Vorel <pvorel@suse.cz> wrote:
>
> TST_SR_IGNORE_ERR has a broader meaning, it ignores all errors (e.g.
> when file does not exists during reading).
>
> Reported-by: Aaron Knobloch <aarondevelops@gmail.com>
> Closes: https://github.com/linux-test-project/ltp/issues/1239
> Signed-off-by: Petr Vorel <pvorel@suse.cz>

Reviewed-by: Li Wang <liwang@redhat.com>

> @Li I plan to add include/tst_sys_conf.h to our doc and use these
> description in kerneldoc, but better to update the old doc in case I
> forget.

Thanks for doing this, Petr.
Cyril Hrubis May 16, 2025, 8:39 a.m. UTC | #2 
Hi!
Reviewed-by: Cyril Hrubis <chrubis@suse.cz>
Petr Vorel May 16, 2025, 12:29 p.m. UTC | #3 
Hi all,

> Hi!
> Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

Merged, thanks!

Kind regards,
Petr
diff mbox series

Patch

diff --git a/doc/old/C-Test-API.asciidoc b/doc/old/C-Test-API.asciidoc
index fe26e26067..72fd2731d3 100644
--- a/doc/old/C-Test-API.asciidoc
+++ b/doc/old/C-Test-API.asciidoc
@@ -1651,7 +1651,7 @@  with 'TBROK'. This behavior can be changed using tst_path_val.flags:
 * 'TST_SR_TBROK_RO' – End test with 'TBROK' if the file is read-only
 * 'TST_SR_TCONF_RO' – End test with 'TCONF' if the file is read-only
 * 'TST_SR_SKIP_RO' – Continue without saving the file if it is read-only
-* 'TST_SR_IGNORE_ERR' – Ignore errors when writing new value into the file
+* 'TST_SR_IGNORE_ERR' – Ignore all errors during reading and writing the file
 
 Common flag combinations also have shortcuts: