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

Launching a certain steam game crashes labwc #1788

Closed
guzz46 opened this issue May 8, 2024 · 20 comments
Closed

Launching a certain steam game crashes labwc #1788

guzz46 opened this issue May 8, 2024 · 20 comments
Labels
bug Something isn't working
Milestone

Comments

@guzz46
Copy link

guzz46 commented May 8, 2024

I have two steam games (halo and gears) that kick me out to the login screen when I try and launch them, they both unfortunately happen to use microsoft .net framework, I never had any issue with them on X11, I've tried a few different launch options, and proton GE without any success, the error log exceeded the maximum characters so I've attached it as a pdf, thanks.

error.pdf

@Consolatis
Copy link
Member

Please attach a simple text file instead. Also, what version / package of labwc are you using?

@Consolatis Consolatis added this to the 0.7.2 milestone May 8, 2024
@guzz46
Copy link
Author

guzz46 commented May 8, 2024

I'm on version 0.7.1 thanks

error.txt

@Consolatis
Copy link
Member

Consolatis commented May 8, 2024

Thanks, the error is this (line numbers for git checkout 0.7.1):

labwc: ../src/view.c:1941: view_move_to_front: Assertion `root' failed

I am not quite sure how this can happen and if this has already been fixed in the current master branch.

My opinion: we should investigate and fix this before the 0.7.2 release and if necessary delay the release.
Thoughts @ahesford @jlindgren90 @johanmalm @tokyo4j ?

Edit: sounds like 9d51107 so chances are this is already fixed in latest master.
@guzz46 any chance you could test the master branch? (either manually or via the labwc-git package if you are on Arch)

@Consolatis Consolatis added bug Something isn't working help wanted Extra attention is needed investigation required Needs further investigation labels May 8, 2024
@tokyo4j
Copy link
Contributor

tokyo4j commented May 8, 2024

My opinion: we should investigate and fix this before the 0.7.2 release and if necessary delay the release.
Thoughts @ahesford @jlindgren90 @johanmalm @tokyo4j ?

I wonder if there's any criteria for delaying releases to fix bugs. Personally, I think we should delay the release only if there are critical regressions that degrade UX from previous versions. But this is not a strong preference and delaying releases doesn't matter for me as a user of labwc-(im-)git AUR.

Anyway, this issue is seemingly already fixed in the master branch so I guess we have nothing to fix before the release.

@Consolatis
Copy link
Member

Personally I feel uncomfortable to ship a release with a known crash / assert trigger. I agree on the assumption that this is likely already fixed in latest master though, I wrote the comment before digging through the git logs.

@ahesford
Copy link
Member

ahesford commented May 8, 2024

If the fix is known and straightforward to implement, delaying a release is reasonable. If the fix is not obvious or requires complex modifications, I wouldn't recommend delaying a release indefinitely to fix a longstanding issue.

Hopefully, this is already fixed by the noted commit.

@guzz46
Copy link
Author

guzz46 commented May 8, 2024

I'm on nixos, so I'm not sure how to go about installing labwc-git, I'll do some research and see if I can figure it out

@tokyo4j
Copy link
Contributor

tokyo4j commented May 8, 2024

I'm on nixos, so I'm not sure how to go about installing labwc-git, I'll do some research and see if I can figure it out

You can follow https://github.com/labwc/labwc?tab=readme-ov-file#2-build-and-installation

@guzz46
Copy link
Author

guzz46 commented May 8, 2024

Thanks but I don't know if that would work on nixos, its immutable, and I believe the only way to install software is declaring it in the configuration.nix file and then doing a "nixos-rebuild switch"

@tokyo4j
Copy link
Contributor

tokyo4j commented May 8, 2024

You don't need to install systemwide for labwc to run. Just run meson setup build, meson compile -C build then execute build/labwc.

EDIT:
I mean an executable file labwc is created in build directory.

@tokyo4j
Copy link
Contributor

tokyo4j commented May 8, 2024

But if you're using display manager, you need to disable it and directly execute build/labwc from VT or modify the configuration file of display manager to execute build/labwc instead. I'm not sure if it's easy in NixOS.

@guzz46
Copy link
Author

guzz46 commented May 8, 2024

I get an error about not being able to find wayland-server when trying to build using meson

@johanmalm
Copy link
Collaborator

I get an error about not being able to find wayland-server when trying to build using meson

That's a package you have to install. Not sure what it's called on Nix.

@guzz46
Copy link
Author

guzz46 commented May 8, 2024

There doesn't appear to be a wayland-server package on nixos, unless its called something else

@tokyo4j
Copy link
Contributor

tokyo4j commented May 9, 2024

There doesn't appear to be a wayland-server package on nixos, unless its called something else

Isn't it in included in wayland package? In Arch /usr/lib/pkgconfig/wayland-server.pc is provided by wayland package.

@guzz46
Copy link
Author

guzz46 commented May 9, 2024

I have /nix/store/la9ax3jv86b69n6d5ybwqz3yggsl170x-wayland-1.22.0-dev/lib/pkgconfig/wayland-server.pc
But its not in /run/current-system/sw/lib/pkgconfig
I don't know why it can't find it when building with meson, I've tried to build it the nixos way but I can't figure out how to do it.

@tokyo4j
Copy link
Contributor

tokyo4j commented May 9, 2024

https://nixos.wiki/wiki/FAQ/I_installed_a_library_but_my_compiler_is_not_finding_it._Why%3F
I know nothing about nix, but I did a quick search and found this.

@guzz46
Copy link
Author

guzz46 commented May 9, 2024

I tried that but it still gave the same error, someone over on reddit says I need to installing it via flakes, which looks too complicated for me to figure out, I guess I'll just have to wait for 0.7.2 to come out to see if its fixed the problem.

@Consolatis
Copy link
Member

That fact that this happens with xwayland on 0.7.1 and the only way for the root to be NULL (child window of an unmanaged surface) is now handled in the master branch I consider this issue to be fixed, even without confirmation. Thus closing this for now but please confirm when you had a chance to test 0.7.2.

Sorry for the noise everybody.

@Consolatis Consolatis removed help wanted Extra attention is needed investigation required Needs further investigation labels May 9, 2024
@guzz46
Copy link
Author

guzz46 commented May 9, 2024

I finally managed to figure out how to install the master branch, and I can confirm the bug doesn't exist in the master branch, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants