Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Automake DIST_SUBDIRS #61

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kamalesh-babulal
Copy link
Contributor

This patchset, has two patches. Where the first one introduces DIST_SUBDIRS
to ship the directories/sub-directories as part of the tarball created using make dist*
and use SUBDIRS to add build conditions.

The second patch is built upon the usage of DIST_SUBDIRS, where the building
and testing SUBDIRS directories (ftests/gunit) are guarded using WITH_TESTS in
the top-level Makefile.am. The WITH_TESTS is passed when ./configure is passed
--enable-tests to the Makefile.

As per automake manual[1], DIST_SUBDIRS should be used to recurse into
the subdirectories those we want to ship as part of make dist and use
SUBDIRS for conditionally building the directories. Adopt this rule
across the source.

[1] https://www.gnu.org/software/automake/manual/html_node/SUBDIRS-vs-DIST_005fSUBDIRS.html

Signed-off-by: Kamalesh Babulal <[email protected]>
Control the building and testing (make check) of the tests using the
if/endif WITH_TESTS guard based on the configuration options passed.
If the --enable-tests (default on) is passed during ./configure step,
the SUBDIRS directories (ftests/gunit) are recused into build and
make check are run.

Signed-off-by: Kamalesh Babulal <[email protected]>
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.

1 participant