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

[Bug] Flatpak 1.18.2 crashes on open file in Fedora 36 #561

Closed
1 task
meeas opened this issue Jun 29, 2022 · 17 comments
Closed
1 task

[Bug] Flatpak 1.18.2 crashes on open file in Fedora 36 #561

meeas opened this issue Jun 29, 2022 · 17 comments

Comments

@meeas
Copy link

meeas commented Jun 29, 2022

Operating System

Linux

What's the issue you encountered?

Note sure if you control the Flatpak build or if it is a third party.....finding this out is a major flatpak weakness. Please let me know if this is a third party build and I will try to hunt them down.

However it installs cleanly on Fedora 36 from flathub, and starts the GUI just fine. However when you open a file or try to create a new file, the app crashes with:

Gtk-Message: 01:31:34.030: Failed to load module "canberra-gtk-module"
Gtk-Message: 01:31:34.030: Failed to load module "pk-gtk-module"
Gtk-Message: 01:31:34.031: Failed to load module "canberra-gtk-module"
Gtk-Message: 01:31:34.031: Failed to load module "pk-gtk-module"
/usr/include/c++/11.3.0/bits/stl_vector.h:1045: std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](std::vector<_Tp, _Alloc>::size_type) [with _Tp = unsigned char; _Alloc = std::allocator<unsigned char>; std::vector<_Tp, _Alloc>::reference = unsigned char&; std::vector<_Tp, _Alloc>::size_type = long unsigned int]: Assertion '__n < this->size()' failed.

I'm guessing it is missing dependencies in the Flatpak build.

How can the issue be reproduced?

Just install from flathub, start app, and try opening a file. To see the error message, start from a commandline with flatpak run.

ImHex Version

1.18.2

ImHex Build Type

  • Nightly or built from sources

Additional context?

  • Additional information about your environment.
  • If possible and useful, please upload the binary you've been editing when the bug occured.
@iTrooz
Copy link
Collaborator

iTrooz commented Jun 29, 2022

Hey, could you try using the AppImage version ? (Please take it from the github actions builds, not the release)

@meeas
Copy link
Author

meeas commented Jun 29, 2022

The AppImage opens the app fine, but when I click Open File is crashed before even showing the file picker, and errors with:

(imhex:61873): GLib-GIO-ERROR **: 02:02:05.004: Settings schema 'org.gnome.settings-daemon.plugins.xsettings' does not contain a key named 'antialiasing'
fish: Job 1, 'imhex.AppImage' terminated by signal SIGTRAP (Trace or breakpoint trap)

@meeas
Copy link
Author

meeas commented Jun 29, 2022

That was running with Wayland. I also tried it with X11 and it threw Awaita Icon errors.

@iTrooz
Copy link
Collaborator

iTrooz commented Jun 29, 2022

Okay, what DE do you use on Fedora ?

@iTrooz
Copy link
Collaborator

iTrooz commented Jul 2, 2022

FlatPak ImHex works for me on Fedora 35 XFCE

@btzy
Copy link
Contributor

btzy commented Jul 2, 2022

It seems that nativefiledialog-extended is the only reason for needing GTK. If you don't need to have a custom default path in the dialog, perhaps try NFD_PORTAL=ON in the CMake configure command so that portals are used instead of GTK dialogs? Especially since this is a build targetting Flatpak.

@iTrooz
Copy link
Collaborator

iTrooz commented Jul 2, 2022

WerWolf pushed a new version to flatpak. Could you try now ?

@WerWolv
Copy link
Owner

WerWolv commented Jul 2, 2022

I have not pushed a new flatpak release. I just fixed added the NFD_PORTAL=ON thing to the cmake script.
There will be a new flatpak release in the next few days though when I release a new version of ImHex

@jonathanspw
Copy link
Contributor

You could try the new native packages built for Fedora too: https://github.com/WerWolv/ImHex/actions/runs/2767434620#artifacts

@iTrooz
Copy link
Collaborator

iTrooz commented Aug 4, 2022

Note : We are waiting for Flatpak to use GCC 12 in their build environnments for the update to be made. (This should be sometime in August)

@iTrooz
Copy link
Collaborator

iTrooz commented Sep 11, 2022

Flatpak now has the latest version : https://flathub.org/apps/details/net.werwolv.ImHex
Could you try with this one ?

@mich181189
Copy link

mich181189 commented Sep 16, 2022

Just installed on Fedora 35 from FlatPak - I get the same issue the original report shows (Assertion '__n < this->size()' failed)

I don't see any Gnome messages in the log though

@meeas
Copy link
Author

meeas commented Sep 16, 2022

Sorry for the silence. I just tried the latest version in FlatHub version 1.23.0, same issue. I also tried your latest official Fedora package version 1.23.0 from the GitHub releases page.

Same issues. Both crash while trying to open a file. I'm running a full patched stable version of Fedora 36, with the default Gnome/Wayland DE.

From RPM build:

❯ imhex                                                                                                                                                                                                ─╯
[17:12:55] [INFO]  [main] Welcome to ImHex!
[17:12:55] [INFO]  [main] Native scaling set to: 1.0
/usr/include/c++/12/bits/stl_vector.h:1123: constexpr std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](size_type) [with _Tp = hex::prv::Provider*; _Alloc = std::allocator<hex::prv::Provider*>; reference = hex::prv::Provider*&; size_type = long unsigned int]: Assertion '__n < this->size()' failed.
[17:13:24] [FATAL] [main] Terminating with signal 6
fish: Job 1, 'imhex' terminated by signal SIGABRT (Abort)

From Flatpack build:

❯ flatpak run net.werwolv.ImHex                                                                                                                                                                        ─╯
[17:17:30] [INFO]  [main] Welcome to ImHex!
[17:17:30] [INFO]  [main] Native scaling set to: 1.0
/usr/include/c++/12.1.0/bits/stl_vector.h:1123: constexpr std::vector<_Tp, _Alloc>::reference std::vector<_Tp, _Alloc>::operator[](size_type) [with _Tp = hex::prv::Provider*; _Alloc = std::allocator<hex::prv::Provider*>; reference = hex::prv::Provider*&; size_type = long unsigned int]: Assertion '__n < this->size()' failed.
[17:17:43] [FATAL] [main] Terminating with signal 6

@jonathanspw
Copy link
Contributor

The official Fedora RPM build for 1.23 crashes on trying to open a file as well. It was reported at https://bodhi.fedoraproject.org/updates/FEDORA-2022-679b84ef3d

@WerWolv
Copy link
Owner

WerWolv commented Sep 17, 2022

Latest commit should address this once again, sorry.

@iTrooz
Copy link
Collaborator

iTrooz commented Sep 17, 2022

1.23.2 is out, should have fixed it :)

@meeas
Copy link
Author

meeas commented Oct 2, 2022

Confirmed, fixed. Thank You!!!

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

No branches or pull requests

6 participants