Skip to content

Fix Windows QNX dependency builds for paths containing spaces - #8

Merged
pablorcum merged 5 commits into
mainfrom
fix/qnx-dependency-path-spaces
Oct 18, 2026
Merged

pablorcum merged 5 commits into
mainfrom
fix/qnx-dependency-path-spaces

Conversation

@pablorcum

Copy link
Copy Markdown
Member

No description provided.

Add dependencyRoot for install, buildtrees, and package roots and return host and target library paths. Reject QNX roots containing whitespace before compilation, preserve forwarded root handling, and document consumer settings.

Generated-by: Codex
@pablorcum pablorcum self-assigned this Oct 18, 2026
Comment thread toolbox/+vcpkg/buildVcpkgDeps.m Fixed
@github-actions

github-actions Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Test Results

 10 files  ± 0   14 suites  ±0   6m 30s ⏱️ + 1m 41s
 51 tests + 5   51 ✅ + 5  0 💤 ±0  0 ❌ ±0 
185 runs  +19  185 ✅ +19  0 💤 ±0  0 ❌ ±0 

Results for commit 2c68628. ± Comparison against base commit 539b9d7.

♻️ This comment has been updated with latest results.

Replace dependencyRoot with temporary drive mappings for affected vcpkg roots while preserving physical installation paths. Clean up mappings on success and failure, return stable paths, and cover implicit and forwarded roots with integration tests.

Generated-by: Codex
@pablorcum
pablorcum marked this pull request as ready for review October 18, 2026 11:18
Use filePermissions.AbsolutePath on R2025a and newer, and preserve older MATLAB compatibility with scoped current-folder resolution.

Generated-by: Codex
Include target integration in default builds and omit it only when GITHUB_ACTIONS is true.

Generated-by: Codex
@pablorcum
pablorcum merged commit f5260d5 into main Oct 18, 2026
7 checks passed
@pablorcum
pablorcum deleted the fix/qnx-dependency-path-spaces branch October 18, 2026 11:49
@pablorcum pablorcum added the bug Something isn't working label Oct 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants