Created attachment 171337 [details]
Maintainer informed via mail
Report was written from bad email address,
but I wrote it as maintainer
Can you please update your bugzilla email address to match that which is in the MAINTAINER line of the dxirc port so we can confirm you are indeed the maintainer.
Sorry, my fault. Instead change email address (I didn't know this option) I created new account with email address matched maintainer line.
As maintainer of dxirc I agree with this update.
Created attachment 176847 [details]
Final diff file. Remove BROKEN= Unfetchable (google code has gone away)
Created attachment 177299 [details]
irc/dxirc: Update to 1.30.0
Would you have a look at this patch? I just finished this when I realized there was already an open PR. Your patch is missing some things as it is. I think it makes more sense to just have the user just pick one toolkit and I defaulted it to QT4 since that is probably the most likely one that the user would already have installed. I didn't see much point in having trivial options that only FOX uses like the tray icon since it doesn't require extra dependencies and can be toggled in the application. It also simplifies things since there are multiple toolkit options and it will make a consistent experience across all of them.
All of the toolkit options work. I did build and run tests with each one. The Qt flavors were failing due to a missing header in CMakeLists.txt and subsequently it wasn't generating the moc code for it, so I patched for that. I also tried to make library detection a little more CMake-ish instead of relying solely on pkgconfig and needing CPPFLAGS/LDFLAGS, etc. in the port Makefile. The better solution would be Find*.cmake modules for everything.
your patch is good work, thanks for it. About default toolkit, Historically dxirc was started with Fox toolkit. dxirc 1.10.0, which last in ports now, has only Fox. As developer of dxirc could say Qt was added on user request. Main effort in development is on dxirc-fox. But I agree that Qt could users installed more often than Fox. But Qt4 specially has a few testing etc. So If we make Qt as default I prefer Qt5, which is default in linux packages.
(In reply to David Vachulka from comment #7)
Fair enough. We can make QT5 the default. I was mostly concerned with using multiple toolkits at the same time. If you're fine with everything else, please set the approval flag on my patch to '+' and I'll commit it. Thanks!
A commit references this bug:
Date: Wed Nov 23 19:21:20 UTC 2016
New revision: 426961
Update to 1.30.0
Switch to CMake build
Convert to options helpers
Add support for Qt and set Qt 5.x as the default toolkit
Submitted by: David Vachulka <email@example.com> (maintainer)
New version committed to HEAD.
Quarterly branch was fixed to fetch previous version in r426753.