nobody/odyssey-xbps-gui
1
0
Fork 1
Code Issues 5 Pull requests 1 Projects Releases Packages Wiki Activity Actions

main #1

Open
zenobit wants to merge 3 commits from zenobit/odyssey-xbps-gui:main into main
pull from: zenobit/odyssey-xbps-gui:main
merge into: nobody:main
Conversation 1 Commits 3 Files changed 1 +5 -5
zenobit commented 2026-05-19 21:06:28 +01:00
First-time contributor
Copy link

squash...

squash...
nobody approved these changes 2026-05-23 22:23:02 +01:00
nobody left a comment
Copy link

Thanks zenobit — merged. This is a real fix, not just a cleanup: xbps-gui does use QtWidgets and QtGui, so declaring python3-pyqt6-widgets and python3-pyqt6-gui as explicit dependencies is correct — without them the app could install but fail to start on a clean system. I verified both names against the Void repos, they're exact. Bumping the revision to 7 was the right move too. Nicely done.

Follow-up on my side: I'll do a full pass over the imports soon to catch any other Qt6 submodules the app pulls in (network, svg, printsupport, etc.), since those need to be explicit in depends as well — they just don't show up as missing on a dev machine that already has everything installed. That'll go in as a separate commit, revision 8.

Thanks zenobit — merged. This is a real fix, not just a cleanup: xbps-gui does use QtWidgets and QtGui, so declaring python3-pyqt6-widgets and python3-pyqt6-gui as explicit dependencies is correct — without them the app could install but fail to start on a clean system. I verified both names against the Void repos, they're exact. Bumping the revision to 7 was the right move too. Nicely done. Follow-up on my side: I'll do a full pass over the imports soon to catch any other Qt6 submodules the app pulls in (network, svg, printsupport, etc.), since those need to be explicit in depends as well — they just don't show up as missing on a dev machine that already has everything installed. That'll go in as a separate commit, revision 8.
This pull request has changes conflicting with the target branch.
  • template
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u main:zenobit-main
git switch zenobit-main

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff zenobit-main
git switch zenobit-main
git rebase main
git switch main
git merge --ff-only zenobit-main
git switch zenobit-main
git rebase main
git switch main
git merge --no-ff zenobit-main
git switch main
git merge --squash zenobit-main
git switch main
git merge --ff-only zenobit-main
git switch main
git merge zenobit-main
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
nobody
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
nobody/odyssey-xbps-gui!1
No description provided.