search

deeponion / deeponion-legacy

Official Source Repo for DeepOnion - Anonymous Cryptocurrency on TOR Network (legacy)
https://deeponion.org
MIT License
416 stars 113 forks source link

Fixed issue: undefined reference to qInitResources_bitcoin() #107

Closed aitorp6 closed 6 years ago

aitorp6 commented 6 years ago

Hi.

This fixes the following issues: https://github.com/deeponion/deeponion/issues/75 and https://github.com/deeponion/deeponion/issues/106

It have been tested on Linux and MAC, test that it works fine on Windows.

deeponion commented 6 years ago

nice and interesting... I've been using this to fix the issue: https://github.com/deeponion/deeponion/issues/75

aitorp6 commented 6 years ago

I used the same trick, but this commit fixes that issue. As I said, it should be tested on Windows.

chisNaN commented 6 years ago

Hello @aitorp6 Thanks for the PR, will test that ASAP on a macbook 12 3rd gen

chisNaN commented 6 years ago

I confirm the src/qt/DeepOnion-qt was successfully generated thanks to changes by @aitorp6 but now i don't understand why the DeepOnion.conf file requires some additional paramaters like rpcuser and rpcpassword sot that the wallet keep running on macOs?

On windows my config file just contains node addresses and it run "correctly". My questions are: Are these variables set somewhere else on windows? If not is it bad practice not to have them set on the DeepOnion.conf file ? (the consequences) Thanks

fuss132 commented 6 years ago

@aitorp6 Are you sure this will work forever? (Haven't tried it yet, would be nice if it does) I currently have to edit the file to change the filehash every now and then. But only at 20% of the time

aitorp6 commented 6 years ago

I hope so, but I'm not sure.

deeponion commented 6 years ago

on windows i never see this issue, the issue only occurs in mac build or linux build.