-
Notifications
You must be signed in to change notification settings - Fork 16
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
Append SDE pkgconfig folders to PKG_CONFIG_PATH #319
Conversation
- Check for `pkgconfig` directories in the SDE and append them to PKG_CONFIG_PATH before using pkg-config to find the DPDK module. Signed-off-by: Derek G Foster <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great find! Change LGTM for DPDK. Can we extend it to ES2K and Tofino as well?
Also, can we append the path for dependencies as well?
To the best of my knowledge, this change only applies to finding pkg-config files that are part of the SDE (i.e., DPDK for the DPDK TDI target). |
- Address a CMake module warning by using `DpdkDriver` as the DPDK package name instead of `DPDK`. - Correct error in xxx_LDFLAGS variable name. Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
36d2739
to
b198e54
Compare
Signed-off-by: Derek G Foster <[email protected]>
Signed-off-by: Derek G Foster <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Derek G Foster <[email protected]>
pkgconfig
directories in the SDE and append them to PKG_CONFIG_PATH before using pkg-config to find the DPDK module.This should eliminate the need to set PKG_CONFIG_PATH in the environment and allow the build to "just work".