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

add_custom_command(TARGET) uses unsupported DEPENDS keyword #5073

Closed
kou opened this issue Jan 31, 2025 · 0 comments · Fixed by #5074
Closed

add_custom_command(TARGET) uses unsupported DEPENDS keyword #5073

kou opened this issue Jan 31, 2025 · 0 comments · Fixed by #5074

Comments

@kou
Copy link
Contributor

kou commented Jan 31, 2025

Problem:

s2n-tls/CMakeLists.txt

Lines 446 to 452 in 3e57b05

add_custom_command(
TARGET ${PROJECT_NAME} POST_BUILD
DEPENDS libcrypto.symbols
COMMAND
bash -c "${CMAKE_AR} -r lib/libs2n.a s2n_libcrypto/*.o"
VERBATIM
)
uses DEPENDS. But we can't use DEPENDS for add_custom_command(TARGET).

If we use DEPENDS for add_custom_command(TARGET), the following warning is reported:

The following keywords are not supported when using
add_custom_command(TARGET): DEPENDS.

-DS2N_INTERN_LIBCRYPTO=ON uses the add_custom_command(TARGET).

Solution:

We can remove the DEPENDS.

  • Does this change what S2N sends over the wire? No.
  • Does this change any public APIs? No.
  • Which versions of TLS will this impact? TLS versions are not related.

Requirements / Acceptance Criteria:

What must a solution address in order to solve the problem? How do we know the solution is complete?

  • RFC links: Not related
  • Related Issues: No relevant issue
  • Will the Usage Guide or other documentation need to be updated? No
  • Testing: We can check whether the warning isn't reported with -DS2N_INTERN_LIBCRYPTO=ON
    • Will this change trigger SAW changes? No.
    • Should this change be fuzz tested? No.

Out of scope:

Is there anything the solution will intentionally NOT address?

No.

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 a pull request may close this issue.

1 participant