-
Notifications
You must be signed in to change notification settings - Fork 359
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
Depext: fix alpine tagged repositories handling #4758
Conversation
I don't think this should be the behaviour in this case. For example:
Both should be marked as installed if they have the same version number |
Oh, I made a mistake in the example. In the code the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor changes to simplify and harden the codebase, but overall it looks ok.
I haven't tested it yet, I'll do that tomorrow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested successfully
Some examples of the new behavior
libx11
as installedlibx11@edge
as availablebubblewrap
andbubblewrap@edge
as installedcapnproto-dev@edgecommunity
as available