Re: C++ help needed (Was: Help with cmake / Qt5 migration needed)

2017-10-21 Thread Juhani Numminen
Hi Andreas, Andreas Tille kirjoitti 21.10.2017 klo 09:22: Hi Juhani, thanks a lot for your help. I'm happy to help you. On Fri, Oct 20, 2017 at 08:41:35PM +0300, Juhani Numminen wrote: I think that's the build system sorted, even though the compilation still fails. Now you probably need t

Re: C++ help needed (Was: Help with cmake / Qt5 migration needed)

2017-10-20 Thread Andreas Tille
Hi Juhani, thanks a lot for your help. On Fri, Oct 20, 2017 at 08:41:35PM +0300, Juhani Numminen wrote: > > I think that's the build system sorted, even though the compilation still > fails. Now you probably need to add some includes. I was able to add the said includes but somehow this project

Re: C++ help needed (Was: Help with cmake / Qt5 migration needed)

2017-10-20 Thread Juhani Numminen
Hi Andreas, Andreas Tille kirjoitti 20.10.2017 klo 18:41: Hi Juhani, On Fri, Oct 20, 2017 at 06:14:02PM +0300, Juhani Numminen wrote: -TARGET_LINK_LIBRARIES( qtbeads ${QT_LIBRARIES} ... +TARGET_LINK_LIBRARIES( qtbeads Qt::Widgets ...>> I can't see that line in the patch at [1] https://anon

Re: C++ help needed (Was: Help with cmake / Qt5 migration needed)

2017-10-20 Thread Andreas Tille
Hi Juhani, On Fri, Oct 20, 2017 at 06:14:02PM +0300, Juhani Numminen wrote: > > I think I fixed this by removing the code copy of CImg.h and using the > > Debian packaged version instead. But the QDebug issue came back and > > may be I need another trick to tweak src/qtbeads/CMakeLists.txt that >

Re: C++ help needed (Was: Help with cmake / Qt5 migration needed)

2017-10-20 Thread Juhani Numminen
Andreas Tille kirjoitti 20.10.2017 klo 15:29: > Hi again, > > On Fri, Oct 20, 2017 at 11:55:22AM +0200, Wout B wrote: >> It looks like the initial #include was actually correct and that >> it indeed doesn't find it somehow. Manually adding Qt5::Core and >> Qt5::Widgets to target_link_libraries

C++ help needed (Was: Help with cmake / Qt5 migration needed)

2017-10-20 Thread Andreas Tille
Hi again, On Fri, Oct 20, 2017 at 11:55:22AM +0200, Wout B wrote: > It looks like the initial #include was actually correct and that > it indeed doesn't find it somehow. Manually adding Qt5::Core and > Qt5::Widgets to target_link_libraries seems to work though. I can confirm that this helps - th