Skip to content

Update Makefile : Fix Library Path for HEXL in Makefile #8

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

This PR updates the Makefile to correct the path of the HEXL library. The previous version referenced lib64/libhexl.a, which has been modified to lib/libhexl.a to ensure compatibility with the latest build structure.

Reason for Change:

The correct path for libhexl.a is lib/ instead of lib64/.
Ensures compatibility with the current build setup.

Impact:

Fixes build issues related to incorrect library paths.
Ensures that the linking process completes successfully with the correct HEXL library.

This PR updates the Makefile to correct the path of the HEXL library. The previous version referenced lib64/libhexl.a, which has been modified to lib/libhexl.a to ensure compatibility with the latest build structure.

Reason for Change:

    The correct path for libhexl.a is lib/ instead of lib64/.
    Ensures compatibility with the current build setup.

Impact:

    Fixes build issues related to incorrect library paths.
    Ensures that the linking process completes successfully with the correct HEXL library.
@p-steuer
Copy link
Contributor

On some systems HEXL will be installed to lib64/. So this will fix the build problem for systems, where HEXL is installed to lib/ but break the build for systems, where HEXL is installed to lib64/.

Ideally, the build system would detect the correct path instead of hard coding it.

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