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

Allow to configure yara's make tool #1123

Merged
merged 1 commit into from
Dec 6, 2022
Merged

Conversation

catap
Copy link
Contributor

@catap catap commented Nov 13, 2022

This is another approach to remove hardocoded call of make to build yara. This approach uses the same make by default but introduces YARA_MAKE_PROGRAM option which can be used to specify a path to gmake for example.

Thus, when CMake produces a set of makefile, it reuses CMAKE_MAKE_PROGRAM to respect a global -jXXX option.

This is another approach to remove hardocoded call of `make` to build
`yara`. This approach uses the same `make` by default but introduces
`YARA_MAKE_PROGRAM` option which can be used to specify a path to
`gmake` for example.

Thus, when CMake produces a set of makefile, it reuses
`CMAKE_MAKE_PROGRAM` to respect a global `-jXXX` option.
@PeterMatula PeterMatula merged commit 9e52bec into avast:master Dec 6, 2022
@PeterMatula
Copy link
Collaborator

Thanks, although the tests fail at the moment, I merged it - it looks fine, and the tests are broken - we are working on it.

@catap catap deleted the yara-make branch December 6, 2022 11:09
@catap
Copy link
Contributor Author

catap commented Dec 6, 2022

@PeterMatula I've used this patch at MacPorts for a while => it is fine :)

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.

None yet

2 participants