Skip to content

Update Makefile #7

New issue

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

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

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

supreet-shukla
Copy link

@supreet-shukla supreet-shukla commented Feb 6, 2025

Fix Makefile: Correct HEXL Library Path

This pull request updates the Makefile to correct the path to the HEXL library. Previously, the Makefile referenced ../../third_party/hexl-development/build/hexl/lib64/libhexl.a, but the lib64 directory does not exist in the specified path.

Changes Made

Updated the Makefile to use the correct path to libhexl.a, ensuring compatibility with the build structure.

Impact

This fix ensures successful compilation of kyber1024-demo without manual intervention in the Makefile.

Testing

Verified that the make command runs successfully after updating the path.

### Error while running make

~/lazer/lazer/demos/kyber1024$ make
cc -Wall -Wextra -Wshadow -Wundef -O3 -g -I../.. -o kyber1024-demo kyber1024-demo.c ../../liblazer.a -lmpfr -lgmp -lm ../../third_party/hexl-development/build/hexl/lib64/libhexl.a -lstdc++ 
cc: error: ../../third_party/hexl-development/build/hexl/**lib64**/libhexl.a: No such file or directory
make: *** [Makefile:11: kyber1024-demo] Error 1

> reason is that there is no file named **lib64** in the path /third_party/hexl-development/build/hexl


I did the necessary changes in the makefile.
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.

1 participant