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

Remove CMake requirement for builds #297

Closed
camshaft opened this issue Dec 14, 2023 · 2 comments
Closed

Remove CMake requirement for builds #297

camshaft opened this issue Dec 14, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@camshaft
Copy link
Collaborator

Problem:

CMake is not a ubiquitous build tool; for example it is not automatically installed with the build-essential. By requiring CMake, aws-lc-rs puts additional build requirements on environments, which will potentially hinder adoption.

Solution

Instead, the library should have a build system that uses more ubiquitous tools. I think make would be a step in the right direction, but I think ideally the only requirement should be a C compiler and assembler. This is the approach that was taken in s2n-tls-sys which makes it quite easy to build.

@justsmth
Copy link
Contributor

Related to #152

@justsmth
Copy link
Contributor

As of release v1.7.0, CMake is no longer required to build aws-lc-rs for Linux/MacOS on x86-64/aarch64. For these platforms, only the minimal Rust & C toolchains should be required for the build to succeed.

I'm going to close this issue now. Feel free to open another issue to request extending support for our no-cmake build to other platforms (e.g., Windows).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants