Skip to content

Update on Readme.md & INSTALL - for GCC & openssl compatibility #33

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

onlylonly
Copy link

Changes 1 -
Modified INSTALL to git clone from openSSL branch "OpenSSL_1_1_0-stable".

  • Since the last tested and verified working version is openssl 1.1.0, I'll suggest to git clone from OpenSSL_1_1_0-stable for maximum compatibility

Changes 2 -
Modify README.md to git clone from openSSL branch "OpenSSL_1_1_0-stable".

  • Same reason as Changes 1

Changes 3 -
Modify Makefile to include -fcommon in the CFLAGS

  • SInce GCC 10, it's defaults to -fno-common(instead of -fcommon) , which doesn't allow mulltiple declaration. This breaks the compilation of Revsh. To fix this, I'll suggest to include -fcommon in the CFLAGS.

Modify install script to git clone from OpenSSL branch 1.1.0_stable
Update Readme to git clone from OpenSSL branch 1.1.0_stable
*Modify MakefIle to include  -fcommon in the
    CFLAGS, GCC 10 have default to -fno-common
    which does not allow duplicate deeclaration
fix the missing - on git clone --branch OpenSSL_1_1_0-stable https://github.com/openssl/openssl.git
added the missing - on line
git clone --branch OpenSSL_1_1_0-stable https://github.com/openssl/openssl.git
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant