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

beta regression: $crate may not be imported #69190

Closed
Mark-Simulacrum opened this issue Feb 15, 2020 · 6 comments
Closed

beta regression: $crate may not be imported #69190

Mark-Simulacrum opened this issue Feb 15, 2020 · 6 comments
Assignees
Labels
P-high High priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@Mark-Simulacrum Mark-Simulacrum added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. regression-from-stable-to-beta Performance or correctness regression from stable to beta. labels Feb 15, 2020
@petrochenkov
Copy link
Contributor

Introduced in #67775 ("Make "use $crate" a hard error ").

@Mark-Simulacrum
Copy link
Member Author

Looks like probably expected breakage -- fallout is not too big, essentially just one root crate. Would be good to figure out if there's a fixed release of said crate that crater didn't pick up, or try to get the crate maintainer to do so. #37390 is the compatibility lint tracking issue.

@pnkfelix
Copy link
Member

expected breakage.

I think the main work item here would be to make sure that the crate in question has a version people can upgrade to, so that they can at least address this in response to the 1.42.0 release.

@pnkfelix
Copy link
Member

triage: P-high to confirm that there is an upgrade path for piston2d-gfx_graphics (potentially a 0.27.1, or maybe just relying on their existing versions)

@pnkfelix pnkfelix self-assigned this Feb 20, 2020
@pnkfelix pnkfelix added the P-high High priority label Feb 20, 2020
@pnkfelix
Copy link
Member

pnkfelix commented Mar 6, 2020

Upgrading piston2d-gfx_graphics from 0.27 to 0.30 seems to do the trick.

The actual problem originates in a macro defined in a dependency (the gfx crate itself).

  • Its non-trivial to upgrade to the first version of gfx (namely 0.12) that does not exhibit the bug. At the very least, you cannot get away with just changing that line of the Cargo.toml for piston2d-gfx_graphics.
  • So, given that there does exist an upgrade path (via gfx_graphics=0.30) and that the most obvious "simpler" solution does not suffice, I'm going to just say the answer here is "upgrade your dependencies to at least gfx_graphics=0.30"

@pnkfelix
Copy link
Member

pnkfelix commented Mar 6, 2020

Closing as expected fallout from the conversion to a hard error.

@pnkfelix pnkfelix closed this as completed Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-high High priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

3 participants