guix-devel
[Top][All Lists]
Advanced

[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: Fri, 28 Oct 2016 16:44:00 +0200
User-agent: mu4e 0.9.17; emacs 25.1.1

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).

Kind regards,
Roel Janssen



reply via email to

[Prev in Thread] Current Thread [Next in Thread]