Skip to content
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

Merged
merged 6 commits into from
Oct 24, 2023
Merged

Conversation

ffoulkes
Copy link
Contributor

  • Modify FindDpdkDriver module to check for 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".

- 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]>
@ffoulkes ffoulkes added minor effort Minimal effort required cmake Affects CMake build system labels Oct 20, 2023
Copy link
Collaborator

@nupuruttarwar nupuruttarwar left a 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?

@ffoulkes
Copy link
Contributor Author

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]>
@ffoulkes ffoulkes force-pushed the dpdk-pkg-config-path branch from 36d2739 to b198e54 Compare October 21, 2023 13:06
Copy link
Collaborator

@5abeel 5abeel left a comment

Choose a reason for hiding this comment

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

LGTM

@ffoulkes ffoulkes merged commit 98c0741 into main Oct 24, 2023
4 checks passed
@ffoulkes ffoulkes deleted the dpdk-pkg-config-path branch October 24, 2023 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmake Affects CMake build system minor effort Minimal effort required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants