Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Determine whether it's acceptable to remain pinned to a particular version of syn, or choose a different workaround #1088

Open
2 tasks
Tracked by #671
joshlf opened this issue Apr 2, 2024 · 0 comments
Labels
blocking-next-release This issue should be resolved before we release on crates.io

Comments

@joshlf
Copy link
Member

joshlf commented Apr 2, 2024

To work around #1085 until we have a more complete solution, we plan to pin our dependency on syn to a specific version (#1089). We need to remove this workaround and implement a more complete solution before releasing 0.8.

Progress

  • Determine whether continuing to remain pinned to a particular version of syn will be problematic for 0.8; consider both dependency resolution and build-time performance
    • An alternative: could we pin to a particular version only on certain Rust toolchains, but leave it unpinned for more recent toolchains? We would need to consider whether we run the risk of future MSRV bumps, or whether we can structure the dependency in such a way that it just always works. This would resolve the build-time performance issues for most users.
  • If we decide not to continue pinning, consider this workaround:
    • Declare that zerocopy-derive's MSRV is just whatever syn's is (TODO: How do we pin this in CI for testing?)
    • Document that zerocopy's derive feature is exempt from our MSRV
    • After 0.8, work to resolve this - in the most optimistic scenario, we can return to having both crates have the same MSRV, but at least we won't be blocked on it
@joshlf joshlf mentioned this issue Apr 2, 2024
51 tasks
joshlf added a commit that referenced this issue Apr 2, 2024
This is a temporary work-around for #1085. Per #1088, we will remove
this and implement a more complete fix before releasing 0.8 (#671).
joshlf added a commit that referenced this issue Apr 2, 2024
This is a temporary work-around for #1085. Per #1088, we will remove
this and implement a more complete fix before releasing 0.8 (#671).
github-merge-queue bot pushed a commit that referenced this issue Apr 2, 2024
This is a temporary work-around for #1085. Per #1088, we will remove
this and implement a more complete fix before releasing 0.8 (#671).
@joshlf joshlf added the blocking-next-release This issue should be resolved before we release on crates.io label May 30, 2024
@joshlf joshlf changed the title Un-pin dependency on specific version of syn Determine whether it's acceptable to remain pinned to a particular version of syn, or choose a different workaround May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocking-next-release This issue should be resolved before we release on crates.io
Projects
None yet
Development

No branches or pull requests

1 participant