Skip to content

0.7.1#496

Merged
ca333 merged 71 commits intomasterfrom
beta
Sep 22, 2021
Merged

0.7.1#496
ca333 merged 71 commits intomasterfrom
beta

Conversation

@ca333
Copy link
Copy Markdown

@ca333 ca333 commented Sep 22, 2021

No description provided.

jmjatlanta and others added 20 commits August 16, 2021 07:57
remove dead dns seeders + add fixed seeds nodes
Since we added mini-gmp.c as a part of libbitcoin_server_a_SOURCES,
-fPIC is required to avoid the linkage error:

/usr/bin/ld: libbitcoin_server.a(libbitcoin_server_a-mini-gmp.o): relocation R_X86_64_32 against `.rodata.str1.1' can not be used when making a shared object; recompile with -fPIC
libbitcoin_server.a: error adding symbols: Bad value
collect2: error: ld returned 1 exit status

TODO: In future we should totally (!) get rid of mini-gmp.c, for example,
same way as i did here - DeckerSU@bf14e82 ,
bcz actually we using only two functions from this mini-gmp.c - it's
bitcoin_base58decode and bitcoin_base58encode, which are perfectly
could be replaced with C++ analogs DecodeBase58, EncodeBase58 from
base58.h
Copy link
Copy Markdown

@smk762 smk762 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Launch and sync tests ok on Window/Linux/Mac

Copy link
Copy Markdown

@DeckerSU DeckerSU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

p.s. And thx everyone for the hard work, related to this release.

Copy link
Copy Markdown
Collaborator

@dimxy dimxy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great refactoring

@ca333 ca333 merged commit 7df102f into master Sep 22, 2021
who-biz pushed a commit to who-biz/komodo that referenced this pull request May 9, 2023
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.

8 participants