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

#6098: Add Config Flag to hyprctl systeminfo #6160

Merged
merged 36 commits into from
May 25, 2024

Conversation

The-Briel-Deal
Copy link
Contributor

@The-Briel-Deal The-Briel-Deal commented May 19, 2024

Closes #6098

Describe your PR, what does it fix/add?

Frequently I will look through issues on Github and be unable to repro the issue because there is no config attached to the issue. My hope is that by adding a --config/-c flag to systeminfo and asking for config files will significantly improve this problem and hopefully save developer time from time wasted trying to repro unclear issues.

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

Should be fine. Let me know if I should correct wording in the template.

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

Should be ready.

src/debug/HyprCtl.cpp Outdated Show resolved Hide resolved
The-Briel-Deal and others added 22 commits May 24, 2024 21:58
if sliding and resizing, include the slide in the resize to avoid off-screen surfaces.

fixes hyprwm#6150
- meson
. fix run_command() check warning
. drop lines for compatability, as it's already using c++23

- cmake
. generate `compile_commands.json` by default
. position independent build: __FILE__
When `region_size` is set in the config (non-empty
`boundBox`), cursor is mapped to wrong coordinate because
`CBox::translate` mutates `TAB->boundBox`, making all subsequent coordinate
calculations wrong.

This also fixes the edge case where user sets `region_position` but
not `region_size`.
Added a simple way to get basic info about the GPU on ARM based systems
on exit of hyprland the CMonitor destroy signal comes after the
compositor has been destructed, causing a heap use after free. add if
check to ensure compositor exist and isnt shutting down when its
triggered.
.github/ISSUE_TEMPLATE/bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug.yml Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/bug.yml Outdated Show resolved Hide resolved
src/config/ConfigManager.cpp Outdated Show resolved Hide resolved
src/config/ConfigManager.cpp Outdated Show resolved Hide resolved
src/config/ConfigManager.cpp Outdated Show resolved Hide resolved
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 vaxerski merged commit 553232a into hyprwm:main May 25, 2024
10 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.

Add Config to systeminfo or bug reports.