[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add qtwebkit.
From: |
Roel Janssen |
Subject: |
Re: [PATCH] gnu: Add qtwebkit. |
Date: |
Wed, 02 Nov 2016 13:35:21 +0100 |
User-agent: |
mu4e 0.9.17; emacs 25.1.1 |
Roel Janssen writes:
> Thomas Danckaert writes:
>
>> Hi,
>>
>> this patch adds QtWebKit 5.7 as a separate package (hopefully this will
>> reduce security concerns: only packages which really need QtWebKit will
>> include it).
>>
>> QtWebKit is not officially supported anymore as of Qt 5.6, so this is a
>> “community release” (not exactly sure what that entails, or if this includes
>> any updates at all since the Qt 5.5 version). I suppose many projects are
>> switching to Qt WebEngine, and there are some efforts to create an updated
>> version of QtWebKit (https://github.com/annulen/webkit), but in the mean
>> time, this package might help build packages which still rely on the old
>> QtWebKit.
>>
>> By default, the qmake build system for qtwebkit insists on installing into
>> the same prefix as qtbase, and it seems no command line parameters will
>> change that. The solution I came up with, was to substitute all the
>> necessary paths in the generated Makefiles. This makes the patch a bit
>> lengthy.
>>
>> I've also attached a minimal example.
>>
>> Thomas
>> QT += widgets
>> QT += webkit
>> QT += webkitwidgets
>> SOURCES = example.cpp
>>
>> #include <QWebView>
>> #include <QApplication>
>>
>> int main(int argc, char** argv) {
>> QApplication app(argc, argv);
>> QWebView view;
>> view.show();
>> view.setUrl(QUrl("https://www.gnu.org/software/guix"));
>> return app.exec();
>> }
>
> Thanks a lot for this patch. I can confirm it builds fine, yet I have
> to test it on some Qt packages that need QtWebkit (Texmaker).
I think this patch works fine. Unfortunately for me, it does not fix
Texmaker's build because it needs QtWebkitWidgets which is apparently a
different thing.
Kind regards,
Roel Janssen
- Re: [PATCH] gnu: Add qtwebkit.,
Roel Janssen <=