Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Travis build for QT 5 fails on build #98 after modularization #11

Closed
Mr-Kumar-Abhishek opened this issue May 6, 2016 · 17 comments
Closed
Assignees
Labels

Comments

@Mr-Kumar-Abhishek
Copy link
Owner

Mr-Kumar-Abhishek commented May 6, 2016

Travis build for QT 5 fails on build #98.

Build error is specifically for QT 5:

In file included from about_app.h:3:0,

                 from about_app.cpp:1:

sub_win.h:4:19: fatal error: QWidget: No such file or directory

compilation terminated.

And.. others such as this which inherits sub_win class. However the build is clean for QT 4

Check here for the build log

Refer here as a starting point to find a solution on how to make it compatible for both QT 4 and QT 5 with modularization

@Mr-Kumar-Abhishek Mr-Kumar-Abhishek self-assigned this May 6, 2016
@Mr-Kumar-Abhishek Mr-Kumar-Abhishek changed the title Travis build for QT 5 fails on build #98 Travis build for QT 5 fails on build #98 after modularization May 6, 2016
@Mr-Kumar-Abhishek
Copy link
Owner Author

This is one of those times when somebody yells..

I don't care if it builds on your machine ! We are not shipping your machine !!

:trollface:

on the other hand , keeping meta-object compilers was probably a good idea 💭

@Mr-Kumar-Abhishek
Copy link
Owner Author

This is ready to be worked on 😃

@Mr-Kumar-Abhishek
Copy link
Owner Author

Okay, so working on it. Testing the builds on test branch..

@Mr-Kumar-Abhishek
Copy link
Owner Author

Nope, having meta-object compilers doesn't makes much of a difference in the build

Check the log for details..

@Mr-Kumar-Abhishek
Copy link
Owner Author

Well, I suppose I have look into it after this build , I wonder why commits aren't referred here...

@Mr-Kumar-Abhishek
Copy link
Owner Author

Alright, I seem to have missed this, hmm...

sudo apt-get install -qq g++ qt4-qmake libqt4-dev qt5-qmake qtbase5-dev" failed and exited with 100

well... now all my QT projects needs to be updated. 😒

@Mr-Kumar-Abhishek
Copy link
Owner Author

I suppose I was not much updated due other priorities. Travis CI was upgraded to container-based infrastructure. Refer to these docs for migrating

@Mr-Kumar-Abhishek
Copy link
Owner Author

Build logs could be compared by referring Travis build 103 and Travis build 97 which are for the same commits but in different times.

@Mr-Kumar-Abhishek
Copy link
Owner Author

Mr-Kumar-Abhishek commented May 14, 2016

Hmm...changes are not required much if any for migrating.. sudo works in Travis CI for installing Ubuntu packages.. however packages are changed so some options are not available such has
QT 4.8 doesn't have qmake -qt=qt4 option... check the build here

@Mr-Kumar-Abhishek
Copy link
Owner Author

packages qt5-qmake and qtbase5-dev are not available now... I so need to be in ubuntu system to resolve this quickly .. instead of pushing random commits for checking it.

$ sudo apt-get install -qq g++ qt5-qmake qtbase5-dev

E: Unable to locate package qt5-qmake

E: Unable to locate package qtbase5-dev

check this build here

@Mr-Kumar-Abhishek
Copy link
Owner Author

Mr-Kumar-Abhishek commented May 14, 2016

in between.. does these packages where white-listed after Travis CI was upgraded to container-based infrastructure ??

@Mr-Kumar-Abhishek
Copy link
Owner Author

Mr-Kumar-Abhishek commented May 14, 2016

Well, qtbase5-dev package isn't white-listed ..

@Mr-Kumar-Abhishek
Copy link
Owner Author

We will use beineri ppa which is white-listed. It gives tests and builds on different version of QT (more than the previous configuration)

Mr-Kumar-Abhishek added a commit that referenced this issue May 16, 2016
@Mr-Kumar-Abhishek
Copy link
Owner Author

Mr-Kumar-Abhishek commented May 16, 2016

Got the Travis CI build 108 only the build of Qt 4.8 passed, others failed. Presumably we are back to main issue of making it compatible with Qt 5.x versions after modularization.

@Mr-Kumar-Abhishek
Copy link
Owner Author

The docs only gives way to transition from QT 4 .x to QT 5.x, however, it can't be made to be compatible on both with this modularization. So either we have to keep this modularization and be in Qt 4.x or remove most of the modularization and make it compatible for both Qt 4.x and Qt 5.x versions.

@Mr-Kumar-Abhishek
Copy link
Owner Author

It would be better to keep this in 4.x and leave transitioning from QT 4.x to QT 5.x in another issue on a later date. For, now just Travis CI configuration file needs to be adjusted so that it only builds for QT 4

@Mr-Kumar-Abhishek
Copy link
Owner Author

No error in builds now. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant