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

Unable to build with Qt6.8 #464

Open
txtsd opened this issue Nov 16, 2024 · 8 comments
Open

Unable to build with Qt6.8 #464

txtsd opened this issue Nov 16, 2024 · 8 comments

Comments

@txtsd
Copy link

txtsd commented Nov 16, 2024

Steps to reproduce

  1. Try to build with Qt6.8

Expected behaviour

It should build

Actual behaviour

It throws errors about the #error macro being removed in Qt6.8

These errors show up even when all the #error macro lines are replaced with static_assert().

Operating system

Arch Linux

Version of Kotatogram Desktop

1.5.0.4.r1005.g0581eb6

Installation source

Other (unofficial) source: git

Crash ID

No response

Logs

No response

@ilya-fedin
Copy link
Contributor

A tip: you can still build with Qt 5

@txtsd
Copy link
Author

txtsd commented Dec 9, 2024

A tip: you can still build with Qt 5

Yes, definitely, but I'm in charge of packaging https://aur.archlinux.org/packages/kotatogram-desktop-git so this is here as an upstream tracker.

@ilya-fedin
Copy link
Contributor

These errors show up even when all the #error macro lines are replaced with static_assert().

This #error is in Qt sources, you have to replace them in Qt, not kotato

Yes, definitely, but I'm in charge of packaging https://aur.archlinux.org/packages/kotatogram-desktop-git so this is here as an upstream tracker.

I'm highly doubt it will be solved any time soon, so switch the package to Qt 5 for the time being? So that people can still use it at least.

@txtsd
Copy link
Author

txtsd commented Dec 10, 2024

These errors show up even when all the #error macro lines are replaced with static_assert().

This #error is in Qt sources, you have to replace them in Qt, not kotato

Ah I should've realised!

Yes, definitely, but I'm in charge of packaging https://aur.archlinux.org/packages/kotatogram-desktop-git so this is here as an upstream tracker.

I'm highly doubt it will be solved any time soon, so switch the package to Qt 5 for the time being? So that people can still use it at least.

That's not a bad idea. I will do it soon.

@dreirund
Copy link

dreirund commented Feb 23, 2025

This #error is in Qt sources, you have to replace them in Qt, not kotato

Ah I should've realised!

By the way: Is there any upstream report to Qt already? If so, where?

I'm highly doubt it will be solved any time soon, so switch the package to Qt 5 for the time being? So that people can still use it at least.

That's not a bad idea. I will do it soon.

Looking forward to it.

Regards!

@ilya-fedin
Copy link
Contributor

By the way: Is there any upstream report to Qt already? If so, where?

No. Do you think they will revert the change? It looks pretty intentional.

@dreirund
Copy link

By the way: Is there any upstream report to Qt already? If so, where?

No. Do you think they will revert the change? It looks pretty intentional.

I understood the comment posted by @ilya-fedin:

This #error is in Qt sources, you have to replace them in Qt, not kotato

that it is an Qt issue (Qt sources beeing inconsistend with themselved).

So, it is not a Qt issue but a kotatogtam issue?

Regards!

@ilya-fedin
Copy link
Contributor

ilya-fedin commented Feb 23, 2025

Qt basically removed the API kotatogram is using and returns a error on an attempt to use it when building. From my understanding, that's not an issue at all, just incompatibility, there seem to be no guarantee by Qt that sources using it will build without changes with next versions. There seem to be only binary guarantees (you can build the sources with the Qt version it was made for and then update Qt, it should still work).

In other words, you can build only with old Qt and that seem to be absolutely intended.

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

3 participants