-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
The terminal does not appear. #1397
Comments
add screencast Screencast.from.2024-01-01.10-01-57.1.mp4 |
Can you run |
|
this is the part I find interesting, or most concerning. I am not sure why it even thinks that |
Hey, i'm trying to build on openSUSE Tumbleweed myself and realized that our install-deps.sh script is still only installing qt5 dependencies. How did you build it for SuSE? Using Qt5 or Qt6 dependencies? What are you dependencies that you used? (I'm trying to get it running on a local openSuSE VM myself :) |
I'm having issues finding the package that provides |
Here is my OBS spec, which is not different from the Fedora spec.
|
Sadly, at v0.4, OBS failed to build with qt5 and gave priority only to qt6, whereas at v0.3, both successfully built. |
thx for your quick response, @mantarimay. yeah, i know about OBS having hard-moved to qt6. I'd prefer that too, but we still need to somehow support Ubuntu 18.04 and 20.04 (wich both don't come with Qt 6). We try to default and build with Qt 6 where-ever possible. Also, we have some features that only work if built with qt6 (like color schemes automatically adapting to dark/light mode switching).
that should be at least Catch2 version 3 (3.4 or higher) with Contour >= 0.4.0 (0.4.1 as of last night)
As of Contour >= 0.4.1, we've dropped
Not sure if this line is addressing it, but we have the changelog embedded into the metainfo.xml. If not, you could use xmllint to extract the changelog info as well. That's what we do for the release CI :) |
I'd highly recommend to add libsixel-tools to it (is it called like this?) (https://github.com/libsixel/libsixel), because of the CLI tool |
|
@mantarimay I know this is the worst location to point out at potential bugs in the openSuSE packaging, but I think there's a problem with Somehow somebody believed that on ARM64 one only needs OpenGL ES, and thus, the above package does not provide any other GL function headers for qt6 (qt5-opengl-devel does provide them btw). I am on an ARM64 laptop (Apple M1) on various Linux distros. So far openSuSE (Tumbleweed) is the first one that does not provide more headers, such as the one we need Can you maybe point me to the place were I could report it in a way it is heard, or you could drop that info to the relevant people? (sorry for the comms abuse :) ) I'll try to spin up a SuSE distro on my x86-64 desktop pc once i'm near it to repro your findings :) see: aarch64 contents vs x86-64 contents |
@Marenz You have Contour build from source on openSuSE Tumbleweed. Do you happen to remember what you added (dependencies installed) to get it working with Qt 6? |
Hi there!
|
Hey @uunnxx . I am really sorry for this experience. There seems to be dependencies missing. We are going to fix this ASAP. -- if nobody was faster, I will be trying to fix that after I am back from work, tonight . Sorry for the inconveniences . |
@christianparpart after installing these dependencies, I've successfully running
|
@uunnxx what distribution are you on? I could not find a |
No, this module is not for qt5, this is due to lack of some features from qt5 in qt6 qml, Blur for instance |
@mantarimay Hey, my linked PR #1399 seems to work for Ubuntu 22.04 packaging. Looking at the diff, maybe that helps creating a .rpm package for SuSE Tumbleweed as well? Is there anything we can do on our end to ease packaging? Mind, with Contour 0.4, we've transitioned from from Qt Widgets (default Qt 5) to Qt QML (default Qt 6). This is rumbling, and I hope we can ease the inconveniences to a minimum to all as much as possible. EDIT: (I tried to add SuSE Linux Tumbleweed to our CI, but failed in finding time so far...) |
@uunnxx your comment aboved helped me A LOT! Many thanks for that. 🥇 |
@mantarimay ok i think i've found the last missing dependencies on SuSE Linux for Contour >= 0.4, which are:
So in total (in our install-deps.sh script, for quick building from source), we have the following section for SuSE identified systems: contour/scripts/install-deps.sh Lines 435 to 448 in 1df9f70
|
Good morning. @mantarimay when you confirm, that you've got it working on the SuSE side, I think we can close this ticket. No hurries though, take your time :) |
@mantarimay I really wonder where this |
not specifically no, as it was a constant add this delete this etc so at the end I had no idea which part was really required. I can check what packages are installed now at least, but which ones of those are actually required I can't say. Let me know if you want that list (as it is on an other computer I have to boot then) |
I think it's not a package issue per se. we should try to figure out why we see this one warning there, stating |
I compiled contour on opensuse from (#1399) and everything works for me |
@Yaraslaut I think we need to repro that by using @mantarimay's .spec files - and these on a fresh SuSE image :) |
@mantarimay can you please help me a bit, how to test this |
I ran into a similar issue on Fedora 39: √ ; contour debug all
[2024-01-17 20:16:57.686968366.686968] [config] Loading configuration from file: /home/yg/.config/contour/contour.yml
[2024-01-17 20:16:57.689668714.689668] [config] Missing key .logging.enabled. Using default: false.
[2024-01-17 20:16:57.689676344.689676] [config] Missing key .logging.file. Using default: "".
[2024-01-17 20:16:57.689755864.689755] [config] Missing key background_image.path. Using default: "".
[2024-01-17 20:16:57.689784594.689784] [config] Missing key .shell. Using default: "".
[2024-01-17 20:16:57.689816535.689816] [config] Missing key .refresh_rate. Using default: 0.
[2024-01-17 20:16:57.689850435.689850] [config] Defaulting TERM to contour.
[2024-01-17 20:16:57.698985173.698985] [gui.display] Color theme mode at startup: Light
Warning: QQmlApplicationEngine failed to load component ((null):0, (null))
Warning: qrc:/contour/ui/main.qml:30:5: Type Terminal unavailable (qrc:/contour/ui/main.qml:30, (null))
Warning: qrc:/contour/ui/Terminal.qml:8:1: module "Qt5Compat.GraphicalEffects" is not installed (qrc:/contour/ui/Terminal.qml:8, (null)) ; sudo dnf5 install contour-terminal
Updating and loading repositories:
Fedora 39 - x86_64 - Updates 100% | 113.2 KiB/s | 22.0 KiB | 00m00s
Repositories loaded.
Package Arch Version Repository Size
Installing:
contour-terminal x86_64 0.4.1.6292-1.fc39 updates 3.1 MiB |
Looks like fedora package is missing some dependencies, can you please check if installing qt6-qt5compat package will resolve this issue for you @kierun? So @topazus can update package to fix it |
@Yaraslaut I think you don't need to build locally because it is already built on obs. You just install using At least I put the instructions to install manually here:
|
So qt6-qt5compat is runtime dependency? |
@topazus , @Yaraslaut I installed this package and
But it does not seem to affect the terminal at all. |
@kierun IIUC that message is because QT is trying to use hardware acceleration but you are probably missing a(nother) library. Apart from performance I don't think it will affect anything - I only just silenced a similar warning on my system today - yet to notice any performance improvements. |
@cqexbesd Thanks for the help. Any chance you can remember which package that is? |
Everything I know is from https://wiki.archlinux.org/title/Hardware_video_acceleration so I think it depends on your hardware. |
@mantarimay so few packages are missing as a dependency :( i managed to open contour after installing qt6-qt5compat-imports and qt6-multimedia-imports |
I just checked. We've added these already to our install-deps.sh script in 1df9f70 (Jan 4th '24). @mantarimay: As soon as you confirm that you CAN build a package for SuSE Linux again, we'll be releasing 0.4.2. So once confirmed by you, you may want to wait a very little bit and go straight for 0.4.2 then. It will contain bug fixes only (quite important ones IMHO), if you don't mind. Many thanks, and sorry for the inconveniences. :) |
@Yaraslaut Glad if it works, missing dependency will I fix it in a new release. At least I can update on main repo even it doesn't work on my machine. Thank you for testing. |
@mantarimay i'd like your machine to work though, as well... |
@mantarimay any updates? |
@Yaraslaut not yet. I decided to remove it from the main repo because v0.3 build failed took over 6 weeks and v0.4 did not properly work on my machine until now. I still think it's worth it if I push even though it does not work on my machine. |
@Yaraslaut I am glad now that I know the problem. It's because I use |
Oh, great that everything is working now :) |
@mantarimay FYI: I sadly need to release again, because last night release contains a mutex thread locking condition. So I'll release now, containing only the fix, but I sadly have to bump the numbers for this, to 0.4.3 |
Contour Terminal version
0.4.1.6292
Installer source
Github: source code cloned
Operating System
openSUSE Tumbleweed
Architecture
x86-64
Other Software
libunicode version : 0.4.0-21.1
boxed-cpp-devel version : 1.1.0-4.2
gnome-shell version : 45.0+172 git.1dda3393-220.1
labwc version : 0.7.0+28 git.21234a5-33.1
Steps to reproduce
Expected Behavior
Terminal appears and is functional, like version 0.3.
Actual Behavior
I tried version 0.3.12.262 and had no problem, but after upgrading to 0.4, terminal did not appear.
Additional notes
tested on wayland at gnome-shell & labwc (wlroots).
I wish I could push and update to the main repo on openSUSE, but sadly, it's not open properly.
Build info:
https://build.opensuse.org/project/show/home:mantarimay:branches:X11:terminals
The text was updated successfully, but these errors were encountered: