Handle library paths with spaces

[brondani]

Changes

• Add quotes to library paths when it contains spaces
• Add test cases

Checklist

• 🤖 This change is covered by unit tests (if applicable).
• 🤹 Manual testing has been performed (if necessary).
• 🛡️ Security impacts have been considered (if relevant).
• 📖 Documentation updates are complete (if required).
• 🧠 Third-party dependencies and TPIP updated (if required).

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[qltysh]

Coverage Impact

⬆️ Merging this pull request will increase total coverage on main by 0.31%.

Modified Files with Diff Coverage (1)

Rating	File	% Diff	Uncovered Line #s
	pkg/maker/buildcontent.go	80.0%	392, 410-411
	Total	80.0%

🤖 Increase coverage with AI coding...

In the `handle-library-paths-spaces` branch, add test coverage for this new code:

- `pkg/maker/buildcontent.go` -- Lines 392 and 410-411

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[github-actions]

Test Results

  6 files  ±0   18 suites  ±0   4s ⏱️ +3s
110 tests +1  110 ✅ +1  0 💤 ±0  0 ❌ ±0 
660 runs  +6  660 ✅ +6  0 💤 ±0  0 ❌ ±0 

Results for commit e621d83. ± Comparison against base commit 99ccdb9.

[Copilot]

Pull request overview

This PR updates the CMake generation logic to correctly handle library paths containing spaces by quoting affected target_link_libraries(...) entries, and adds test coverage plus fixture data to validate the behavior.

Changes:

• Quote target_link_libraries entries that contain spaces during CMake content generation.
• Add a unit test covering mixed quoted/unquoted library entries with/without spaces and varying indentation.
• Extend test fixtures to include a library file whose filename contains a space and update the expected generated CMakeLists.txt.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
test/data/solutions/blanks/ref/project X.AC6 X+ARMCM0 X/CMakeLists.txt	Updates expected CMake output to include a quoted library path containing spaces.
test/data/solutions/blanks/project/project X.cproject.yml	Adds a library file with a space in its filename to the project input fixture.
test/data/solutions/blanks/project/project X.AC6 X+ARMCM0 X.cbuild.yml	Adds the space-containing library file as a library category input fixture.
test/data/solutions/blanks/project/libr ary.a	Adds a dummy library file fixture whose filename includes a space.
pkg/maker/buildcontent.go	Introduces quoteEntry and applies it when emitting target_link_libraries entries.
pkg/maker/buildcontent_test.go	Adds a unit test validating quoting behavior for library entries with spaces.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.