Skip to content

Conversation

@ngaro
Copy link

@ngaro ngaro commented Nov 6, 2016

  • Adds 2 missing headers needed for compiling
  • Removes a definition already defined earlier (not necessary)
  • Fixes problems with incorrect gcc options
  • Automatically download and build the correct version of openssl
  • Add possibility for cleaning the build environment

@Ri1a
Copy link

Ri1a commented Apr 24, 2018

I changed the lines you committed, but I still got one error:

"gcc: error: ../heartleech/heartleech.c: No such file or directory"

I also reinstalled it and modified the path, but this didn't help. Did I do sth wrong?

@ngaro
Copy link
Author

ngaro commented Aug 4, 2018

I tried it again here, works fine. You must be compiling it wrong.
I can compile my rep like this:
git clone git@github.com:ngaro/heartleech.git && cd heartleech && make

Copy link

@Ri1a Ri1a left a comment

Choose a reason for hiding this comment

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

To entirely fix the Linux build problems you should also add the -I flag to openssl/libssl.a and openssl/libcrypto.a

Copy link

@Ri1a Ri1a left a comment

Choose a reason for hiding this comment

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

added -I flags to openssl/libssl.a and openssl/libcrypto.a

gcc -I../openssl/include -L../openssl -lcrypto -lssl -lcrypto -ldl -lpthread -o heartleech heartleech.c

gcc heartleech.c -Iopenssl -Iopenssl/include -Iopenssl/crypto/include -Iopenssl/ssl openssl/libssl.a openssl/libcrypto.a -ldl -lpthread -o heartleech

Copy link

Choose a reason for hiding this comment

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

Suggested change
gcc heartleech.c -I openssl -I openssl/include -I openssl/crypto/include -I openssl/ssl -I openssl/libssl.a -I openssl/libcrypto.a -ldl -lpthread -o heartleech

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.

2 participants