Re: Fixed runtime compilation error for Qt4

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

In response to

Browse pgadmin-hackers by date

  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.