Re: QtWebengine version

2022-05-20 Thread Thiago Macieira
On Friday, 20 May 2022 15:27:00 PDT Albert Astals Cid wrote:
> One solution that may make sense is:
>  * Us creating a kde/5.15 branch on 5.15 and make it so the generated cmake
> file doesn't exactly require 5.15.x but just generally require 5.15

That sounds like what upstream should be have been doing all along.

-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
   Software Architect - Intel DPG Cloud Engineering





Re: QtWebengine version

2022-05-20 Thread Albert Astals Cid
El divendres, 20 de maig de 2022, a les 17:14:16 (CEST), Jonathan Riddell va 
escriure:
> I'm building Qt 5 from the KDE branches with the new 5.15.4 rebase and
> QtWebengine is up to 5.15.10, I understand this is because there are tags
> in Qt Git for it.

We do *not* maintain a kde/5.15 branch for qtwebengine, what you get is the 
actual upstream 5.15 branch, that module is still maintained publicly.
 
> I can patch the version number in QtWebEngine to match but that feels
> wrong.  What's the right solution?
> https://invent.kde.org/neon/qt/qtwebengine/-/blob/Neon/release/debian/patches/0001-lower-version-to-.3-to-align-with-the-rest-of-the-pa.patch

For me that looks like the right solution (I have not looked at the patch, i 
mean it from the theoretical point of view), you're indeed mixing two versions, 
it's just that they happen to work better so relaxing the requirement to be 
exactly matching versions makes sense.

Other [wrong] solutions would be:
 * Use actual qtwebengine 5.15.4 instead 5.15.10 <- that's stupid, you'll only 
get more security bugs
 * Us creating a kde/5.15 branch on top of 5.15 and faking the version number 
to call it 5.15.4 <- that's wrong, it's not 5.15.4

One solution that may make sense is:
 * Us creating a kde/5.15 branch on 5.15 and make it so the generated cmake 
file doesn't exactly require 5.15.x but just generally require 5.15

I think that would probably be an acceptable solution (maybe i'm wrong, it's 
late), but do we have someone with the time to work on such a patch?

Cheers,
  Albert

> 
> Jonathan
> 






Re: QtWebengine version

2022-05-20 Thread Antonio Rojas

El viernes, 20 de mayo de 2022 17:14:16 (CEST), Jonathan Riddell escribió:

I can patch the version number in QtWebEngine to match but that feels
wrong.  What's the right solution?
https://invent.kde.org/neon/qt/qtwebengine/-/blob/Neon/release/debian/patches/0001-lower-version-to-.3-to-align-with-the-rest-of-the-pa.patch


You need to patch the generated cmake config file

https://www.qt.io/blog/building-qt-webengine-against-other-qt-versions


QtWebengine version

2022-05-20 Thread Jonathan Riddell
I'm building Qt 5 from the KDE branches with the new 5.15.4 rebase and
QtWebengine is up to 5.15.10, I understand this is because there are tags
in Qt Git for it.

However building, for example, messagelib I get complains about QtQuick not
being the right version

"""

*12:27:47* CMake Error at
/usr/lib/x86_64-linux-gnu/cmake/Qt5WebEngineCore/Qt5WebEngineCoreConfig.cmake:111
(find_package):*12:27:47*   Could not find a configuration file for
package "Qt5Quick" that is*12:27:47*   compatible with requested
version "5.15.10".*12:27:47* *12:27:47*   The following configuration
files were considered but not accepted:*12:27:47* *12:27:47*
/usr/lib/x86_64-linux-gnu/cmake/Qt5Quick/Qt5QuickConfig.cmake,
version: 5.15.4*12:27:47* *12:27:47* Call Stack (most recent call
first):*12:27:47*
/usr/lib/x86_64-linux-gnu/cmake/Qt5WebEngineWidgets/Qt5WebEngineWidgetsConfig.cmake:111
(find_package)*12:27:47*
/usr/lib/x86_64-linux-gnu/cmake/Qt5/Qt5Config.cmake:28
(find_package)*12:27:47*   CMakeLists.txt:66 (find_package)*12:27:47*
"""

I can patch the version number in QtWebEngine to match but that feels
wrong.  What's the right solution?
https://invent.kde.org/neon/qt/qtwebengine/-/blob/Neon/release/debian/patches/0001-lower-version-to-.3-to-align-with-the-rest-of-the-pa.patch

Jonathan