-
Notifications
You must be signed in to change notification settings - Fork 82
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
stop using repo2solv #534
Comments
Since we are currently rewriting code to be executed async we'd need to build our own repo2solv binary ( libsolv has no thread support )... which probably would not be smaller than the one we are currently shipping.. |
Once we are there we can try and build a binary that only does what we want and links libsolv directly and see if its smaller. |
We're currently using Maybe @mlschroe likes to split a few things into a basic libsolv-tools and move the stuff also covered by repo2solv into an extra package (updateinfoxml2solv, susetags2solv, rpmmd2solv, rpms2solv, helix2solv, deltainfoxml2solv..) |
libsolv could also expose those features as library function |
libzypp depends on libsolv-tools due to a single line of code where
repo2solv
is invoked:libzypp/zypp/RepoManager.cc
Line 452 in fb7deaf
Would it be possible to drop this external call and use parts of libsolv directly? I know that https://github.com/openSUSE/libsolv/blob/master/tools/repo2solv.c is not exactly small, but maybe not all of its functionality is absolutely required?
This would allow us to make our containers a bit smaller by dropping the libsolv-tools dependency entirely.
The text was updated successfully, but these errors were encountered: