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

Fix CMake configuration for hide-formfields build #35

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

Conversation

fless-lab
Copy link
Contributor

Description:

This PR fixes the CMake configuration for the hide-formfields module, ensuring that the build works properly on systems with Qt5 and Poppler dependencies.

Changes:

  • Updated CMakeLists.txt to properly locate and link Qt5 and Poppler libraries.
  • Adjusted include directories to resolve build issues.

Why this change?
The previous configuration caused compatibility issues on certain systems where dependencies like Qt5 and Poppler were not properly detected or linked. This fix ensures a smooth and consistent build process across environments.

- Updated `CMakeLists.txt` to properly find and link Qt5 and Poppler libraries.
- Set correct include directories to resolve build issues.
- Ensures compatibility and successful build on systems with Qt5 and Poppler dependencies.
@brendanbond
Copy link
Contributor

brendanbond commented Nov 22, 2024

@AlexeyNikipelau does this enable us to build this binary? Is there still a use for it? Tagging you in here

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