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

Create seemingly necessary subcompositor #73

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gwimm
Copy link

@gwimm gwimm commented Jan 27, 2023

Both alacritty and foot want them, tinywl does it, so kiwmi now does it.

Previous name: "Make diplay info info"

I believe that to be a far more concise and succinct message. Buffet
voiced their concerns about informational content, which I must
vehemently disagree with. The inclusion of "logging" as a concept in the
commit message I see as an entirely superfluous and honestly insulting
statement to the programmer's intelligence. To assume that someone would
fail to infer that "making something info" is to log it as info is
blatantly derogatory. Nonetheless I believe this commit to be important
enough to future developer experience to contribute this despite the
terrible overreach by the original developer of kiwmi.

To hope that this incident is not to be forgotten.

Acked-by: buffet <[email protected]>
@kuruczgy
Copy link

kuruczgy commented Apr 1, 2023

The subcompositor not being created is a breaking change that wlroots 0.16.0 introduced: https://gitlab.freedesktop.org/wlroots/wlroots/-/releases/0.16.0:

wlr_subcompositor is not created automatically anymore on calling wlr_compositor_create(); use wlr_subcompositor_create().

As such, this should not be a separate PR, but included together with #71

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