Commit ff79e5f
committed
fix: resolve header staging regression from Issue #38 fix
URGENT FIX: Restore same-component header includes while preserving cross-package functionality
Problem:
- Previous Issue #38 fix broke same-component headers like #include "simd_utils.h"
- CI dropped from 276/280 targets to 270/280 targets
- Components like image_processing:simd_utils failed to find their own headers
Root Cause:
- Directory structure preservation logic was applied to ALL headers
- Same-component headers need basename staging (e.g., "simd_utils.h" → "simd_utils.h")
- Cross-package headers need directory preservation (e.g., "foundation/types.h" → "foundation/types.h")
Solution:
- Implemented hybrid approach with pattern detection:
1. Cross-package headers (path contains "test/" or "/foundation/") → preserve directory structure
2. Same-component headers (normal case) → use basename for local includes
Testing:
- ✅ simd_utils builds successfully with "simd_utils.h" include
- ✅ calculator_c_component builds successfully
- ✅ http_service_component builds successfully
- ✅ cross_package_headers:consumer_component still works with "foundation/types.h"
Impact:
- Restores CI success rate while maintaining Issue #38 cross-package fix
- Both local and cross-package header patterns work correctly1 parent 5775f77 commit ff79e5f
1 file changed
+11
-10
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
281 | 281 | | |
282 | 282 | | |
283 | 283 | | |
284 | | - | |
285 | | - | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
286 | 288 | | |
287 | | - | |
288 | | - | |
289 | 289 | | |
290 | 290 | | |
291 | | - | |
292 | | - | |
| 291 | + | |
| 292 | + | |
293 | 293 | | |
294 | | - | |
295 | | - | |
| 294 | + | |
| 295 | + | |
| 296 | + | |
296 | 297 | | |
297 | 298 | | |
298 | | - | |
| 299 | + | |
299 | 300 | | |
300 | 301 | | |
301 | 302 | | |
302 | 303 | | |
303 | | - | |
| 304 | + | |
304 | 305 | | |
305 | 306 | | |
306 | 307 | | |
| |||
0 commit comments