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

CI broken, due to MSRV of home crate. #1283

Closed
michaelkirk opened this issue Dec 19, 2024 · 4 comments · Fixed by #1290
Closed

CI broken, due to MSRV of home crate. #1283

michaelkirk opened this issue Dec 19, 2024 · 4 comments · Fixed by #1290

Comments

@michaelkirk
Copy link
Member

CI is currently broken, due to an MSRV bump of the home crate.

Our direct dependency proj has a transitive dependency on the which crate, which in turn depends on this home crate.

Someone recently opened harryfei/which-rs#104, which well summarizes the issue

home is not intended for external use (outside of cargo). home has cargo's automatic MSRV-3 policy, which causes build failures for downstream dependants who check against their MSRV and where the package resolver is not MSRV-aware.

They proposed a fix just yesterday: harryfei/which-rs#105

env-home is a better solution:

As a side note, I just learned that rust 2024 intends to have an MSRV aware package resolution, yay!

We can either:

  • wait to see if this fix gets merged soon
  • pin to an older version of home
  • abandon our 6+ month MSRV support window (not my preferred option)

All this to read a couple of env vars 😆

This was referenced Dec 19, 2024
@urschrei
Copy link
Member

Let's give it a few days, but I can't see a fix landing before Christmas

@michaelkirk
Copy link
Member Author

#1285 was merged, which works around the problem, but I'm going to leave this issue open as a reminder that ultimately we should remove the dev-dependency specification for home.

Plausible resolutions are:

@Xaeroxe
Copy link

Xaeroxe commented Dec 20, 2024

which 7.0.1 is now released which should resolve this problem.

@michaelkirk
Copy link
Member Author

michaelkirk commented Dec 20, 2024

Oh geeze, turns out bindgen removed the which crate a while back.

Sorry for the noise @Xaeroxe, but thanks for the quick response anyway.

For us, the next steps are:

  • update bindgen in proj-sys: update bindgen proj#210
  • release proj-sys
  • update proj's proj-sys dependency
  • release proj
  • update geo's proj dependency
  • Finally, delete our optional home dependency in geo.

github-merge-queue bot pushed a commit to georust/proj that referenced this issue Dec 20, 2024
The immediate instigation for this was
georust/geo#1283.

Turns out bindgen [dropped the `which`
dependency](rust-lang/rust-bindgen@c03964c)
a while back.

But also, good to stay on top of the dependency updates.
@michaelkirk michaelkirk mentioned this issue Dec 23, 2024
2 tasks
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 a pull request may close this issue.

3 participants