Skip to content

refactor!: flatten crate organization to use workspace properly - #321

Merged
cedricchevalier19 merged 5 commits into
LIHPC-Computational-Geometry:masterfrom
imrn99:flatten-workspace
Jan 19, 2026
Merged

cedricchevalier19 merged 5 commits into
LIHPC-Computational-Geometry:masterfrom
imrn99:flatten-workspace

Conversation

@imrn99

@imrn99 imrn99 commented Jan 16, 2026

Copy link
Copy Markdown
Member

I consider this a breaking change since if people specified the dependency using a path, it will break.

This PR reorganize the crates of the repository to:

  1. remove multiple nesting level
  2. use the root manifest as a workspace manifest, unifying dependency spec across members

I could also move mesh-io/mesh-io-ffi to the tools crate to only keep coupe and tools visible at the root.

With this done, we can setup auto deps updates without PRs being too confusing.

Old structure

>tree --gitignore -L 2
.
├── Cargo.lock
├── Cargo.toml
├── Dockerfile
├── ..
├── benches
│   └── rcb_cartesian.rs
├── examples
│   └── simple.rs
├── ffi
│   ├── Cargo.toml
│   ├── src
│   └── ..
├── src
│   ├── lib.rs
│   ├── main.rs
│   └── ..
└── tools
    ├── Cargo.toml
    ├── ..
    ├── mesh-io
    ├── num-part
    ├── report
    └── src

New structure

>tree --gitignore -L 2
├── Cargo.lock
├── Cargo.toml
├── ..
├── coupe
│   ├── Cargo.toml
│   ├── benches
│   ├── examples
│   └── src
├── coupe-ffi
│   ├── Cargo.toml
│   ├── src
│   └── ..
├── mesh-io
│   ├── Cargo.toml
│   └── src
├── mesh-io-ffi
│   ├── Cargo.toml
│   ├── meshio.h
│   └── src
└── tools
    ├── Cargo.toml
    ├── ..
    ├── bins
    ├── lib
    ├── num-part
    └── report

some subcrates were using outdated deps, which required code updates
to build. `cargo build --all-targets` now works, with many warnings
@imrn99
imrn99 marked this pull request as ready for review January 16, 2026 16:10

@cedricchevalier19 cedricchevalier19 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it makes a lot of sense to reorganize the repo like this.
Thank you

@cedricchevalier19
cedricchevalier19 merged commit 6abd08f into LIHPC-Computational-Geometry:master Jan 19, 2026
4 checks passed
@imrn99
imrn99 deleted the flatten-workspace branch January 19, 2026 12:39
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