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

GPU hotplug support #8980

Merged
merged 7 commits into from
Jan 23, 2025
Merged

GPU hotplug support #8980

merged 7 commits into from
Jan 23, 2025

Conversation

maround95
Copy link
Contributor

Describe your PR, what does it fix/add?

See this for information: hyprwm/aquamarine#130
The Hyprland part just involved lifecycle managing of the wayland event sources

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

Needs the aquamarine PR above merged before.

Is it ready for merging, or does it need work?

Been daily driving and playing with this for a couple of days, no issues.

@maround95
Copy link
Contributor Author

maround95 commented Jan 20, 2025

Rebased onto main and resolved conflicts

Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

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

rest lgtm

vaxerski
vaxerski previously approved these changes Jan 22, 2025
Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

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

thanks!

@vaxerski
Copy link
Member

missed include it seems for map

vaxerski
vaxerski previously approved these changes Jan 22, 2025
@vaxerski
Copy link
Member

missing clang-format now :P

@maround95
Copy link
Contributor Author

Dammit :p I'm on termux so I don't see

@vaxerski
Copy link
Member

clang-format -i src/**/*{cpp,hpp}

@vaxerski vaxerski merged commit f1bd628 into hyprwm:main Jan 23, 2025
11 checks passed
@littleblack111
Copy link
Contributor

missing clang-format now :P

hmm is the action not working?

@vaxerski
Copy link
Member

yes it fails to post the patch

@vaxerski
Copy link
Member

image

@littleblack111
Copy link
Contributor

littleblack111 commented Jan 23, 2025

seem like permission problem. anyways, lets not talk abt it here, will make new PR to discuss & solve it

EDIT: ref: #9144

littleblack111 pushed a commit to littleblack111/Hyprland that referenced this pull request Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants