Skip to content

Optimize Boost headers packaging to exclude unnecessary files#11

Merged
fumitoh merged 1 commit intomainfrom
claude/reduce-download-files-x8ig4
Feb 25, 2026
Merged

Optimize Boost headers packaging to exclude unnecessary files#11
fumitoh merged 1 commit intomainfrom
claude/reduce-download-files-x8ig4

Conversation

@fumitoh
Copy link
Member

@fumitoh fumitoh commented Feb 25, 2026

Summary

Modified the Boost headers packaging logic to copy only the boost/ subdirectory instead of the entire Boost include directory, reducing package size by excluding unnecessary files from the full Boost source distribution.

Key Changes

  • Changed from copying the entire $BoostIncludeDir to selectively copying only the boost/ subdirectory
  • Added explicit staging directory creation before copying files
  • Updated comments to clarify that only the boost/ headers are included, excluding libs/, tools/, doc/, and other unnecessary directories

Implementation Details

  • Creates the boost staging directory explicitly with New-Item before copying
  • Uses -Force flag to ensure directory creation succeeds
  • Targets the specific boost/ subdirectory path to avoid copying unrelated Boost distribution files
  • This approach reduces the final package size while maintaining all necessary header files for QuantLib compilation

https://claude.ai/code/session_01BHkqy2bRJgJdgTNNeHNVHt

The packaging step was copying the entire extracted Boost source tree
(libs/, tools/, doc/, status/, etc.) into the distribution zip. Only
the boost/ headers subdirectory is needed for compiling against QuantLib.
This eliminates the majority of files from the downloadable.

https://claude.ai/code/session_01BHkqy2bRJgJdgTNNeHNVHt
@fumitoh fumitoh marked this pull request as ready for review February 25, 2026 14:40
@fumitoh fumitoh merged commit 38ff047 into main Feb 25, 2026
2 checks passed
@fumitoh fumitoh deleted the claude/reduce-download-files-x8ig4 branch February 25, 2026 14:46
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.

2 participants