Adam Williamson
f036b42298
All checks were successful
CI via Tox / tox (pull_request) Successful in 1m50s
Packaging two Python scripts, one of which is a wrapper for the other, is pretty awkward. But more importantly, we can make the EL "wrapper" work better with a refactor, too. It's now a stand alone script called `rdc-compose`. The original script is renamed to `rdc-repos`. The CLI scripts now share a lot of bits via `shared.py`. We put handy run-it-locally wrappers for each CLI script at the top level; you *can* install the package properly and you'll get `rdc-repos` and `rdc-compose` executables but we'll likely rarely do that. `rdc-compose` will now run faster because it will reuse the same metadata between variants (previously it got re-downloaded with every execution of `rmdepcheck.py`) and it can now nicely collect up all the repoclosure strings before using the same parse-and- exit code as rmdepcheck, which solves a lot of awkward issues around output. We also add tests covering `rdc-compose`, as part of which it made sense to factor out the Pungi config URL discovery (which I meant to do anyway). We fix a bug that was exposed by writing the tests: we have to check variants that depend on variants we have packages for, even if we don't have packages for those variants. This isn't great git hygiene, sorry. Should have made the refactor, the addition of tests and the bug fix into separate commits. But it's awkward and time-consuming to unpick now and I don't think it's a good use of time. Signed-off-by: Adam Williamson <awilliam@redhat.com>
14 lines
369 B
Python
Executable file
14 lines
369 B
Python
Executable file
#!/usr/bin/python3
|
|
|
|
"""Convenience wrapper for executing rdc-repos from a git checkout."""
|
|
|
|
import os
|
|
import sys
|
|
|
|
# add src subdirectory directory to module import path
|
|
sys.path.insert(0, os.path.join(os.path.dirname(os.path.realpath(__file__)), "src"))
|
|
|
|
from rmdepcheck.repos import main # pylint: disable=wrong-import-position
|
|
|
|
if __name__ == "__main__":
|
|
main()
|