Douglas Roark
Douglas Roark
Ready to go. Verified that the Armory code removes the warnings. LMDB and Crypto++ still have the warnings.
@goatpig - I don't think this breaks anything that wasn't already broken. After getting the tests going again (PR #173), I noticed that nothing was broken that wasn't already broken....
@goatpig - Okay. I've been keeping it rebased as I go. I'll test it once OSX builds again.
@goatpig - The PR should be ready to go. I did add some code that probably should be discussed, though. I added some configure.ac changes that enable shadow warnings by...
Crypto++ is under the Boost license now, as I recall. I seem to recall Alan saying awhile back that this prohibited us from migrating. I'd have to go look at...
@goatpig - On second thought, Boost and MIT seem to be fine. I went through some old chat logs and can't find any explicit reason why the Boost license spooked...
Gaaaaaaah. Rebased and didn't realize I was rebasing on the wrong branch. Will try again in the morning.
Rebased. Would still like to see this get into Armory 0.97. The cleaner we can make the C++ code, the better off we are in the long run. :) (That...
Hmmm. Is there another way to do this? I don't like the idea of forcing people to add LRELEASE_PATH to their configs. macOS users would have to do it. I...
On macOS, this patch fails. ``` checking for lrelease... no configure: error: missing lrelease in path, make sure qt4-linguist-tools is installed or specify LRELEASE_PATH ``` Granted, I am running this...