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

back-port: Remember referenced repositories as system repositories in… #543

Merged

Conversation

gireeshpunathil
Copy link

…stead of user

Currently there is a quite surprising behavior, that if a user adds an updatesite that contains a referenced repository, that after an update-check additional repositories are visible. Even worse these are now used additionally to check for updates and if they contain other references these are also added and so on. This can result not only in a long list of sites the user never has added and has no clue where they are coming from but even pulling in unwanted or conflicting updates.

This now distinguishes two cases:

  1. A repository is discovered by the RepositoryAction it is handled as a user added repository and becomes visible
  2. A repository is discovered by a reference in that case it is handled as a system repository and not becomes visible

that way the list of user visible repositories stay clean from referenced repositories and unexpected side effects on update checks.

@gireeshpunathil
Copy link
Author

back port of #512. I have a customer asking for it.

tivervac and others added 3 commits September 19, 2024 16:27
This abstracts the bit flag checking for repository
references.
Currently if a discovery event would define a nickname it is not passed
by the add repository method resulting in an updatesite to have an empty
name.

This adds a new method that allows to pass a custom RepositoryInfo so
the nickname can be set.
…stead of user

Currently there is a quite surprising behavior, that if a user adds an
updatesite that contains a referenced repository, that after an
update-check additional repositories are visible. Even worse these are
now used additionally to check for updates and if they contain other
references these are also added and so on. This can result not only in a
long list of sites the user never has added and has no clue where they
are coming from but even pulling in unwanted or conflicting updates.

This now distinguishes two cases:

1) A repository is discovered by the RepositoryAction it is handled as a
user added repository and becomes visible
2) A repository is discovered by a reference in that case it is handled
as a system repository and not becomes visible

that way the list of user visible repositories stay clean from
referenced repositories and unexpected side effects on update checks.
@gireeshpunathil
Copy link
Author

i had to backport #506 and #511 to make this work. however, cannot figure out what is the reason for the build-break. there are several of those, so not sure if it is a flake. I ran locally and it works fine. any thoughts?

Copy link
Contributor

@tjwatson tjwatson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tjwatson tjwatson merged commit 5af003c into eclipse-equinox:R4_31_maintenance Sep 23, 2024
1 of 2 checks passed
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 this pull request may close these issues.

4 participants