Skip to content

Conversation

@gambr
Copy link
Contributor

@gambr gambr commented May 26, 2013

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

Without QT, CMake fails:

CMake Error at /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:97 (MESSAGE):
  Could NOT find Qt4 (missing: QT_QMAKE_EXECUTABLE QT_MOC_EXECUTABLE
  QT_RCC_EXECUTABLE QT_INCLUDE_DIR QT_LIBRARY_DIR QT_QTCORE_INCLUDE_DIR
  QT_QTCORE_LIBRARY QT_QTGUI_INCLUDE_DIR QT_QTGUI_LIBRARY
  QT_QTTEST_INCLUDE_DIR QT_QTTEST_LIBRARY QT_UIC_EXECUTABLE)
Call Stack (most recent call first):
  /usr/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:288 (_FPHSA_FAILURE_MESSAGE)
  /usr/share/cmake-2.8/Modules/FindQt4.cmake:1200 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  examples/CalcQt/CMakeLists.txt:3 (find_package)


-- Configuring incomplete, errors occurred!

Can you make it optional? You can simply skip it if QT libs are not found or prompt a warning message and continue.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree. I will fix it asap.

@paoloambrosio
Copy link
Member

Thanks for the contribution!

A few notes...

IMO the wait times should be configurable (perhaps from an environment variable?) so that the tests can run at full speed in an integration server and with wait times for manual and demo purposes.

Please check that the C++ code style matches that of the rest of the project.

Gianni added 2 commits June 2, 2013 20:03
…or each step.

Skip compilation when FindQt4 fails.
Cleaned indentation and curly brackets. Removed commented code and useless includes.
paoloambrosio added a commit that referenced this pull request Jun 3, 2013
GUI Calculator example with Qt libraries
@paoloambrosio paoloambrosio merged commit 56c8e05 into cucumber:master Jun 3, 2013
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.

2 participants