Skip to content

add mirror source switching functionality during slow database sync#4376

Open
xhdndmm wants to merge 3 commits intoarchlinux:masterfrom
xhdndmm:master
Open

add mirror source switching functionality during slow database sync#4376
xhdndmm wants to merge 3 commits intoarchlinux:masterfrom
xhdndmm:master

Conversation

@xhdndmm
Copy link
Copy Markdown

@xhdndmm xhdndmm commented Apr 5, 2026

  • This fix issue:

PR Description:

add mirror source switching functionality during slow database sync

When the mirror is slow , use the TUI to switch to a new mirror

Tests and Checks

  • I have tested the code!

@xhdndmm xhdndmm requested a review from Torxed as a code owner April 5, 2026 22:31
@svartkanin
Copy link
Copy Markdown
Collaborator

Can you add a explanation and description for this PR.
Also this usrs input and not the TUI menu.

@xhdndmm
Copy link
Copy Markdown
Author

xhdndmm commented Apr 7, 2026

Can you add a explanation and description for this PR. Also this usrs input and not the TUI menu.

okay

@svartkanin
Copy link
Copy Markdown
Collaborator

In addition, this should not be in the main.py

@xhdndmm
Copy link
Copy Markdown
Author

xhdndmm commented Apr 7, 2026

In addition, this should not be in the main.py

So do I need to make some changes?

@svartkanin
Copy link
Copy Markdown
Collaborator

The tui should not get included into the main.py, so I would suggest to extract all that functionality out of the main

@xhdndmm
Copy link
Copy Markdown
Author

xhdndmm commented Apr 7, 2026

The tui should not get included into the main.py, so I would suggest to extract all that functionality out of the main

Okay, I changed it.

@svartkanin
Copy link
Copy Markdown
Collaborator

svartkanin commented Apr 11, 2026

This is still using

		return input(prompt).strip().lower() in ('y', 'yes', '')

also why was the check_online moved?

If you're just using an LLM, please don't as these PRs are on the very bottom of my priority list.

@xhdndmm
Copy link
Copy Markdown
Author

xhdndmm commented Apr 11, 2026

This is still using

		return input(prompt).strip().lower() in ('y', 'yes', '')

also why was the check_online moved?

If you're just using an LLM, please don't as these PRs are on the very bottom of my priority list.

okay, I'll check them, sorry

@xhdndmm
Copy link
Copy Markdown
Author

xhdndmm commented Apr 11, 2026

I fixed it, and now can use tui to switch

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.

2 participants