reddcoin-project / reddcoin-3.10

Reddcoin: the digital social currency
http://www.reddcoin.com
MIT License
287 stars 114 forks source link

Update Litecoin #93

Closed oritwoen closed 4 years ago

oritwoen commented 7 years ago

This project is a fork Litecoin but I see that it is not the current version. Among other things, the following error has been corrected in Litecoin (appears when compiling):

https://github.com/litecoin-project/litecoin/commit/2ede6b7142cb57c422126722f845b36da7d3c4ef

In file included from net.cpp: 24: 0:
/usr/include/miniupnpc/miniupnpc.h:62:1: note: DECLARED here

I will be grateful for updated fork or patch so that the compilation work correctly :)

reddink commented 7 years ago

The current release is now a fork of Bitcoin, Unfortunately, one is not able to update the github details to reflect that :)

The current build was tested using miniupnpc 1.6 (original)

To resolve the error looks like a trivial fix.

Let me take a look

reddink commented 7 years ago

I have just pushed the following commit (81b71123e94ec06c5733cc25c2bb5c9624113259) onto master branch (which also contains a few extra fixes)

oritwoen commented 7 years ago

This error no longer displays but now appear other: x Screenshot terminal attached.

zrzut ekranu z 2016-11-26 07-31-18

reddink commented 7 years ago

This is an issue with the newer versions of boost and compiling with c++11 Referencing against the issue on bitcoin. https://github.com/bitcoin/bitcoin/commit/a2b04ddfe6452e7d6274f4096bf3f2aee695a6d9

I can patch this in tonight,

reddink commented 7 years ago

I have pushed this update to my personal Reddcoin repo first. https://github.com/reddink/reddcoin Can you test it against your system?

oritwoen commented 7 years ago

It works a little better but still in one place, now a little later, there are problems with the make.

System: Fedora 25

reddink commented 7 years ago

What is the error now reported?

oritwoen commented 7 years ago

./autogen.sh

zrzut ekranu z 2016-11-28 13-15-55

make

zrzut ekranu z 2016-11-28 13-20-20

reddink commented 7 years ago

Thanks for that. The first screen is ok.. these warnings are expected

Second screen, The warning is triggered in the version of boost you have installed (1.61?) https://github.com/libbitcoin/libbitcoin/issues/582

It is a known issue and should be resolved in boost 1.62

These should be warnings only, there must be an error also in the output

oritwoen commented 7 years ago

You're right, there is a fault. Nothing else appears.

zrzut ekranu z 2016-11-29 11-53-33

reddink commented 7 years ago

Ok, compiling with QT5 support. The error relates to QT5 build hardening can you run configure with the following CXXFLAGS="-fPIC" ./configure CXXFLAGS="-fPIC"

in the current Reddcoin configure, it is using -fPIE as default. I will need to pull the updated configure scripts from bitcoin to pull into our build

oritwoen commented 7 years ago

Hmm, I checked this. Not work. I used ./configure CXXFLAGS="-fPIC". Makefile contains fPIC instead fPIE but error is the same.

reddink commented 7 years ago

Ok.. thanks for testing. I have found the relevant commits on bitcoin.. They will need a bit of work to pull into Reddcoin.. stand-by

reddink commented 7 years ago

Pulling the relevant build updates is going to take longer than expected. Can you try running: make clean ./configure CXXFLAGS="-fPIC" CFLAGS="-fPIC" make

oritwoen commented 7 years ago

Clean git repository https://github.com/reddink/reddcoin and ./configure CXXFLAGS="-fPIC" CFLAGS="-fPIC" the error this same.

No hurry about it. Take care of more important matters, and to find the time to come back to this, I prefer to compile own wallets but currently I use the downloaded binaries for Linux so it's okay.

oritwoen commented 7 years ago

Problem solved.

./configure CXXFLAGS="-fPIC" CFLAGS="-fPIC" – add fPIC on root folder Makefile but not change ./src/qt/Makefile.

I edited ./src/qt/Makefile CXXFLAGS and manually change fPIE to fPIC.

Btw, please update official branch with previous commit: https://github.com/reddink/reddcoin/commit/2ab654109b69a69c1f4197d4727011afab7d7440

And maybe update ./configure to work with all makefiles not only root?

reddink commented 7 years ago

Thanks, glad you got it working. Will make these changes

davidmar0003 commented 7 years ago

Thanks I hope I can get this to work. If not I'll be back.

Sylk commented 7 years ago

This ticket should be closed.