Skip to content

Fix: Change PUBLIC to INTERFACE for interface library in error message#1400

Open
Isla-jq wants to merge 1 commit intoFreeRTOS:mainfrom
Isla-jq:fix/cmake-interface-public-error
Open

Fix: Change PUBLIC to INTERFACE for interface library in error message#1400
Isla-jq wants to merge 1 commit intoFreeRTOS:mainfrom
Isla-jq:fix/cmake-interface-public-error

Conversation

@Isla-jq
Copy link
Copy Markdown

@Isla-jq Isla-jq commented Apr 11, 2026

Correct target_compile_definitions scope in CMake error message for INTERFACE library

Description

This CMake error shows incorrect usage of PUBLIC scope with an INTERFACE library. This fix corrects the message code.

Test Steps

Verify the message now shows correct CMake syntax with INTERFACE instead of PUBLIC
Run CMake configuration to confirm it no longer generates scope errors

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sonarqubecloud
Copy link
Copy Markdown

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.

1 participant