Re: [PATCH 2/2] treewide: run internal library tests in "make check"

David Timber <[email protected]>
Newsgroups dev.linux.lists.exfat
Message-ID <[email protected]>
On 8/20/26 13:28, David Timber wrote:
> The library tests were intended to be used internally for dev purposes,
> but users have suggested that they should be run when "make check" is
> run.
>
> Hook up the test programs to the TESTS target. "./configure
> --enable-tests" now only hooks up the test programs to the
> noinst_PROGRAMS target. "./configure --enable-tests" is not required to
> run the test programs using "make check".
>
> Fix stray "enable_tests=yes" output in ./configure script due to a
> missing comma.
>
> Run make check in the "Test building exfatprogs on arbitrary directory"
> step in the Github Actions CI pipelines.
>
> Please do not close #391 as the fsck test cases should be run as well.
> This is not an easy fix and further work is being done to achieve this.
>
> Link: https://github.com/exfatprogs/exfatprogs/issues/391
> Suggested-by: Sam James <[email protected]>
> Signed-off-by: David Timber <[email protected]>
> ---
>  .github/workflows/c-cpp.yml | 2 +-
>  configure.ac                | 2 +-
>  lib/Makefile.am             | 7 ++++---
>  lib/tests/run-all.sh        | 3 +++
>  4 files changed, 9 insertions(+), 5 deletions(-)
>  mode change 100755 => 100644 lib/tests/run-all.sh
For reasons unknown, my text editor changed the file permissions. Will
reroll tomorrow.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.