Skip to content

Conversation

@nopdotcom
Copy link

I’m just checkpointing progress on Qt5. This should work on OS X 10.9 and later.

Could you check if notifications work? I don’t know how to elicit one.

I’m going to test a build of this on 10.7. I’m not sure 10.6 is going to work, certainly not without a lot of Homebrew rebuild-from-source.

Don't just blindly pull it. I have not tested this.
In Qt5, qt_mac_execute_apple_script got desupported. Luckily,
QSystemTrayIcon appears to handle Notifications Center, and handles
Growl prior to that.

"The Growl notification system must be installed for
QSystemTrayIcon::showMessage() to display messages on Mac OS X prior
to 10.8 (Mountain Lion)." http://doc.qt.io/qt-5/qsystemtrayicon.html
At some point, Apple changed over from libstdc++ to libc++.  Modern
homebrew builds expect libc++. Change the minimum deployment target to
10.9 temporarily; 10.7 support may be possible.

10.6 support would seem to require custom Homebrew builds of (at
least) Boost and libdb_cxx with CXXFLAGS += -stdlib=libstdc++, or the
equivalent deployment target.
@nopdotcom
Copy link
Author

Grr, this is giving me problems on the latest Xcode, which seems to want C++11. I’ll fix.

@LittleDuke
Copy link
Owner

Damn you’re a coding machine!

Notifications will come if you receive coins — unless you’re talking about alerts?

I’ll just fire off another 1,000 coins at you to test when you’re ready — wonder if we should fire up a Slack thing to do the instant messaging?

-dvd

On May 1, 2016, at 13:27, Jay Carlson notifications@github.com wrote:

I’m just checkpointing progress on Qt5. This should work on OS X 10.9 and later.

Could you check if notifications work? I don’t know how to elicit one.

I’m going to test a build of this on 10.7. I’m not sure 10.6 is going to work, certainly not without a lot of Homebrew rebuild-from-source.

You can view, comment on, or merge this pull request online at:

#3

Commit Summary

Various hacks to make this build on clang on OS X 10.11 and Boost 1.60.
On Qt5 on OS X, only use QSystemTray for notifications.
OS X: pick OS deplyment version vs which stdc++ lib
File Changes

M 2GiveCoin.pro (23)
M src/bitcoinrpc.cpp (4)
M src/net.cpp (2)
M src/qt/bitcoingui.h (2)
M src/qt/notificator.cpp (8)
M src/qt/rpcconsole.cpp (2)
M src/rpcblockchain.cpp (20)
M src/rpcdump.cpp (4)
M src/rpcmining.cpp (22)
M src/rpcnet.cpp (10)
M src/rpcrawtransaction.cpp (12)
M src/rpcwallet.cpp (84)
M src/scrypt-x86_64.S (14)
M src/serialize.h (2)
Patch Links:

https://github.com/LittleDuke/2GIVE/pull/3.patch
https://github.com/LittleDuke/2GIVE/pull/3.diff

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

@nopdotcom
Copy link
Author

Doh, homebrew completely broke macqtdeploy for Qt5. I just spent a few hours debugging before finding Homebrew/legacy-homebrew#42191. Upshot: use the upstream Qt5 libraries if you ever want to deploy a Qt5 app on a non-homebrew system.

Lemme finish this bug before I think of anything else.

@nopdotcom
Copy link
Author

I gave up and went back to Homebrew Qt. It works better. So, to build this branch:

/usr/local/opt/qt5/bin/qmake 2GiveCoin.pro USE_UPNP=-
/usr/local/opt/qt5/bin/macdeployqt 2GiveCoin-Qt.app -verbose=2

The executable runs on 10.7. I don’t have Growl installed, so I don’t think I’ll get notifications.

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