Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

Assertion failed: ps->server_grabbed #29

Closed
apprehensions opened this issue Nov 10, 2023 · 12 comments
Closed

Assertion failed: ps->server_grabbed #29

apprehensions opened this issue Nov 10, 2023 · 12 comments
Assignees
Labels
resolved This issue has a solution!

Comments

@apprehensions
Copy link

<-- 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: f796e60
Build Method: Source (meson setup -Dregex=false build)
WM: dwm

Original picom runs fine.

@allusive-dev
Copy link
Owner

allusive-dev commented Nov 10, 2023

@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
Copy link
Author

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
Copy link
Owner

PCRE is not a official dependency. it is optional.

It is actually. Look at picom-git

@allusive-dev
Copy link
Owner

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

@apprehensions
Copy link
Author

apprehensions commented Nov 10, 2023

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
Copy link
Owner

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
Copy link
Author

yshui/picom@e2b8c3f
PCRE is deprecated. USE PCRE2, or just update this fork...

@apprehensions
Copy link
Author

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
Copy link
Owner

allusive-dev commented Nov 11, 2023

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
Copy link
Author

apprehensions commented Nov 11, 2023

Sorry.

@allusive-dev
Copy link
Owner

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 allusive-dev added the in progress This issue is currently being worked on. label Nov 11, 2023
@allusive-dev allusive-dev modified the milestone: 1.5.0 Nov 11, 2023
Repository owner locked as resolved and limited conversation to collaborators Nov 11, 2023
@allusive-dev
Copy link
Owner

@apprehensions
1.5.0 has released with the updated dependency.

@allusive-dev allusive-dev added resolved This issue has a solution! and removed in progress This issue is currently being worked on. labels Nov 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
resolved This issue has a solution!
Projects
None yet
Development

No branches or pull requests

2 participants