Skip to content

Conversation

@terasakisatoshi
Copy link
Member

Summary

This PR reorganizes the project structure by moving the Rust library from the root directory into , making it a proper dependency of the Julia package.

Changes

1. Directory Structure Reorganization

  • Moved →
  • All Rust source files, configuration files, and generated headers moved

2. Build Scripts and Path References

  • Updated to use instead of
  • Updated default path to
  • Updated and library search paths

3. CI/CD Updates

  • Updated all GitHub Actions workflows to use paths
  • Updated cache paths and keys to reflect new location
  • Updated integration test paths

4. Documentation Updates

  • Updated all path references in documentation
  • Updated project structure descriptions
  • Updated build instructions and troubleshooting guides

Benefits

  • Better project organization: is now clearly a dependency of
  • Cleaner root directory structure
  • More intuitive for Julia package development workflow
  • Follows Julia package conventions for embedded dependencies

Testing

  • All path references updated
  • Build scripts tested with new paths
  • CI workflows updated
  • Documentation reviewed for accuracy

This reorganizes the project structure by moving the calcpi-rs Rust library
into CalcPi.jl/deps/calcpi-rs, making it a dependency of the Julia package
rather than a sibling directory.
- Update build.jl to use deps/calcpi-rs instead of ../calcpi-rs
- Update generate_C_API.jl default path to ../deps/calcpi-rs
- Update prologue.jl and C_API.jl to search for library in deps/calcpi-rs/target
- Update all working-directory paths from calcpi-rs to CalcPi.jl/deps/calcpi-rs
- Update cache paths and keys to reflect new location
- Update integration test paths
- Update all path references in documentation from calcpi-rs to CalcPi.jl/deps/calcpi-rs
- Update project structure descriptions
- Update build instructions and troubleshooting guides
@terasakisatoshi terasakisatoshi merged commit 2c03a97 into main Jan 8, 2026
23 checks passed
@terasakisatoshi terasakisatoshi deleted the move-calcpi-rs-to-deps branch January 8, 2026 07:27
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