search

allusive-dev / compfy

A Compositor for X11 based on Picom. No Longer Maintained in favor of Picom v12!
200 stars 24 forks source link

Assertion failed: ps->server_grabbed #29

Closed apprehensions closed 11 months ago

apprehensions commented 11 months ago

<-- Please fill in all the forms below that are relevant then delete this line -->

What is the issue Assertion failure within this project

What did you expect Program runs

Relevant Images or Videos:

Assertion failed: ps->server_grabbed (../src/c2.c: c2_match: 1671)
Aborted

Your configuration Default, no configuration file is present

Package Version: f796e60a0bafd10e7a956e6c791b2198e7f06141 Build Method: Source (meson setup -Dregex=false build) WM: dwm

Original picom runs fine.

allusive-dev commented 11 months ago

@apprehensions You may not have all the required dependencies. I ran the same command as you, on a fresh install of Arch with the dependencies and their were no issues.

apprehensions commented 11 months ago

PCRE is not a official dependency. it is optional. Please update your README to reflect that.

Why is PCRE required for this though? Regular picom functions fine without it.

allusive-dev commented 11 months ago

PCRE is not a official dependency. it is optional.

It is actually. Look at picom-git

allusive-dev commented 11 months ago

Either way I can confirm PCRE is required for this fork of picom to my knowledge.

apprehensions commented 11 months ago

No. It is a OPTIONAL dependency.

https://github.com/allusive-dev/picom-allusive/blob/f796e60a0bafd10e7a956e6c791b2198e7f06141/src/meson.build#L46

It can be disabled. If you truly believe it is required then make it a forced dependency with no options.

allusive-dev commented 11 months ago

If you truly believe it is required then make it a forced dependency with no options.

I'll do that in a second then. Can you please try building with PCRE installed.

apprehensions commented 11 months ago

https://github.com/yshui/picom/commit/e2b8c3fd1e5eb06c137daf03dcb6c11f93aa2cd4 PCRE is deprecated. USE PCRE2, or just update this fork...

apprehensions commented 11 months ago

Building with PCRE does not help:

Assertion failed: ps->server_grabbed (../src/c2.c: c2_match: 1672)
Aborted

c2 is the original file for regex (which if you went through the source code to begin with, you would have known). Please update your fork and fix this issue

allusive-dev commented 11 months ago

I would appreciate you be a bit more courteous with your wording.

(which if you went through the source code to begin with, you would have known)

Please don't insinuate that I am not putting effort into this project, I'm trying very hard to fix all the issues people have had with picom and make it better.

I will make the updates however in the future please consider the choosing of your words more carefully.

apprehensions commented 11 months ago

Sorry.

allusive-dev commented 11 months ago

I'm testing the updates now but it might take quite a while since I need to change the dependencies on NixOS for me to be able to build the package.

When you see 1.5.0 release that will be the patched update.

allusive-dev commented 11 months ago

@apprehensions 1.5.0 has released with the updated dependency.