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

Split OpamClient content #5890

Open
rjbou opened this issue Mar 19, 2024 · 0 comments
Open

Split OpamClient content #5890

rjbou opened this issue Mar 19, 2024 · 0 comments

Comments

@rjbou
Copy link
Collaborator

rjbou commented Mar 19, 2024

OpamClient module is more and more huge, it contains in its first part all opam commands first level function, and in its second part only opam init functions (~700 lines on a ~1900 lines file). Initialisation functions are more and more huge with Windows support (Cygwin, Git, MSYS2, etc.).
It can be separated into OpamClient and OpamClientInit, the second module/file containing only init related functions. We can keep OpamClient.init functions that is just a redirection to OpamClientInit.init to not break all projects using opam libraries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant