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

RFE: fmf plugin/dialect #236

Open
LecrisUT opened this issue May 21, 2024 · 0 comments · May be fixed by #250
Open

RFE: fmf plugin/dialect #236

LecrisUT opened this issue May 21, 2024 · 0 comments · May be fixed by #250

Comments

@LecrisUT
Copy link
Contributor

LecrisUT commented May 21, 2024

Description

Right now there is only one "dialect" of the fmf files being tmt test files, while if we want to use fmf in other context, they would have an ambiguity if a fmf tree is supposed to be processed by tmt or another program. An example of this is tmt lint.

Proposal

Define a standard .fmf/plugins file which would be read as a requirements.txt file, e.g.

# .fmf/plugins
tmt
tmt-cmake @ git+https://github.com/LecrisUT/tmt-cmake@main

Fmf responsibility here would be just to:

  • Check that the plugin requirements are satisfied
    • name and specifier
    • maybe extras and marker
    • ignore url
  • Print out a requirements.txt of the unsatisfied plugins when executing cli
  • Print out all the .fmf/plugins as is (in case this will evolve later). Useful if the user would want to always install the plugins

Impact in tmt/testing-farm

  • tmt would have to suggest adding a .fmf/plugins file with tmt line
  • testing-farm would check the .fmf/plugins and if there is a package there that it does not recognize with python3dist() as pre-installed, then fail early
@LecrisUT LecrisUT linked a pull request Jun 17, 2024 that will close this issue
9 tasks
@LecrisUT LecrisUT changed the title Store plugin requirements RFE: fmf plugin/dialect Nov 11, 2024
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 a pull request may close this issue.

1 participant