Skip to content

Added CMake build process to the project#1

Open
richardlayman wants to merge 14 commits intoheeen:masterfrom
richardlayman:master
Open

Added CMake build process to the project#1
richardlayman wants to merge 14 commits intoheeen:masterfrom
richardlayman:master

Conversation

@richardlayman
Copy link

I made a few changes that would effect the qmake build.
I changed the location of the qpa header which I need to fix.
I also changed the location of the shaders search path - which really isn't needed, it was just put there for aesthetics.

#include <qpa/qplatformnativeinterface.h>
//#include <qpa/qplatformnativeinterface.h>
// TODO - I need to fix this below line so that you can build using the above line
#include <QtGui/5.6.0/QtGui/qpa/qplatformnativeinterface.h>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, I won't merge it if it ties the whole build to this qt version

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you need qplatformnativeinterface for?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@heeen
Copy link
Owner

heeen commented Apr 28, 2016

Hi,
thank you for your work! please see my comments.

Florian

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants