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

Also consider profile properties when looking for disable mirror keys #3384

Merged
merged 2 commits into from
Jan 19, 2024

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Jan 19, 2024

I also created

as currently Tycho does ignore any mirrors (by modify the repository under the hood), but the mirror list is still loaded in such a case!

Replaces:

Copy link

Test Results

  579 files  ±0    579 suites  ±0   3h 36m 47s ⏱️ - 33m 5s
  384 tests ±0    377 ✅ ±0   7 💤 ±0  0 ❌ ±0 
1 152 runs  ±0  1 130 ✅ ±0  22 💤 ±0  0 ❌ ±0 

Results for commit d31c3d9. ± Comparison against base commit 253632e.

@laeubi
Copy link
Member Author

laeubi commented Jan 19, 2024

@vogella @Phillipus please validate

@Phillipus
Copy link

@laeubi I've tested this with both options tycho.disableP2Mirrors and eclipse.p2.mirrors in my settings.xml file and profile and it is working as expected.

Thanks for this, sorry I didn't take it further in the original PR, your implementation is much better than what I would have done!

@laeubi laeubi merged commit 5974572 into eclipse-tycho:main Jan 19, 2024
10 checks passed
@laeubi
Copy link
Member Author

laeubi commented Jan 19, 2024

@vogella
Copy link
Contributor

vogella commented Jan 19, 2024

Thanks @laeubi for fixing it and @Phillipus for identifying the issue and helping with testing it.

You guys are cool.

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.

3 participants