Skip to content
New issue

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

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

Already on GitHub? # to your account

Feat: option to install test programs #1079

Merged
merged 7 commits into from
Oct 23, 2024
Merged

Conversation

perazz
Copy link
Member

@perazz perazz commented Sep 23, 2024

Address #1070 @everythingfunctional.

This PR proposes to add the option to also install the test programs in the installation directory.
Two methods are made possible, similar to what can be done for a library install:

  1. Via manifest

User can now specify a test boolean key in fpm.toml to request that test programs are installed as well

[install]
library = true
test = true
  1. Via CLI

User can request tests to be installed via the --test flag.

The default installation subdirectory is test/, but the user can request a custom subdirectory via --testdir "testdir".
In both cases, this is a subdirectory created within the installation $PREFIX.

fpm install --prefix=/path --test --testdir "my_test_dir"

Full description is reported in fpm help install.

cc: @everythingfunctional @urbanjost @fortran-lang/fpm

Copy link
Member

@everythingfunctional everythingfunctional left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for putting this together.

@perazz
Copy link
Member Author

perazz commented Sep 27, 2024

Thanks a lot Brad. @everythingfunctional. Let's wait a bit longer, and then merge if there are no further comments.

@perazz perazz merged commit 1f08686 into fortran-lang:main Oct 23, 2024
22 checks passed
@perazz perazz deleted the install_tests branch October 23, 2024 13:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants