From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Neel Patel <neel(dot)patel(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Fixed runtime compilation error for Qt4 |
Date: | 2016-02-23 09:12:05 |
Message-ID: | CA+OCxox3a_gffUnjQouSEkYCUqSc=ZfVEM10MR663rBj-fM6ng@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, committed.
On Tue, Feb 23, 2016 at 7:51 AM, Neel Patel <neel(dot)patel(at)enterprisedb(dot)com>
wrote:
> Hi All,
>
> As sandeep from EDB was doing the setup for pgAdmin4 runtimes in CentOS 7
> with Qt4 and found that there were some compilation errors.
>
> Please find the attached patch file which will fix the below compilation
> errors for Qt4.
>
> - QtWebKitWidgets should only be included in Qt5. There is no such
> header in Qt4.
> - QUrl class does not have "url()" function in Qt4 so when we compile
> the code in Qt4 it gives error saying "no such function url()". This is
> fixed by adding common method ( "host()" ) available in both the version of
> Qt.
> - In QTcpSocket class, there is no "bind" method available in Qt4 so
> it gives error during the compilation.
> - "QThread::sleep(1)" is the public method in Qt5 while it is
> protected in Qt4 so we can not use the same method in Qt4. We have written
> delay function to replace this method.
> - Removed "cache()" from Qt project file. Cache file is required by
> qmake to read special setting specified other then project and qmake.conf
> file, as we have not specified any special configuration other then project
> file so we have removed this function.
>
> We have tested in both the versions Qt4 and Qt5 and now it is compiling
> without any errors.
>
> Do review it and let us know in case of any comments/queries.
>
> Thanks,
> Neel Patel
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2016-02-23 10:07:44 | pgAdmin 4 commit: Add support for viewing/editing procedural languages. |
Previous Message | Dave Page | 2016-02-23 09:10:59 | pgAdmin 4 commit: Runtime fixes for QT4 support. |