Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign up`legacy_imports` future-compatibility warnings #38260
Comments
Mark-Simulacrum
added
the
C-future-compatibility
label
Jun 23, 2017
petrochenkov
referenced this issue
Jul 18, 2017
Merged
Make sufficiently old or low-impact compatibility lints deny-by-default #42894
Mark-Simulacrum
removed
the
C-tracking-issue
label
Jul 22, 2017
petrochenkov
referenced this issue
May 14, 2018
Merged
Turn deprecation lint `legacy_imports` into a hard error #50760
bors
added a commit
that referenced
this issue
May 19, 2018
bors
added a commit
that referenced
this issue
May 19, 2018
bors
closed this
in
#50760
May 19, 2018
Nemo157
referenced this issue
Jun 27, 2018
Closed
bs58 crate fails to build tests in 1.28 beta #51845
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
jseyfried commentedDec 9, 2016
•
edited by petrochenkov
This is the summary issue for the
legacy_importsfuture-compatibility warning and other related errors. The goal of this page is describe why this change was made and how you can fix code that is affected by it. It also provides a place to ask questions or register a complaint if you feel the change should not be made.What is the warning for?
The
legacy_importswarning is issued whenuse super::*) imports a non-pubname,See RFC 1560 for more detailed information.
When will this warning become a hard error?
At the beginning of each 6-week release cycle, the Rust compiler team will review the set of outstanding future compatibility warnings and nominate some of them for Final Comment Period. Toward the end of the cycle, we will review any comments and make a final determination whether to convert the warning into a hard error or remove it entirely.
Current status
legacy_importslint as warn-by-defaultlegacy_importslint deny-by-defaultlegacy_importslint a hard error