-
Notifications
You must be signed in to change notification settings - Fork 13k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Rename files to match current recommendations.
New standards have arisen in recent months, mostly for the use of rustpkg, but the main Rust codebase has not been altered to match these new specifications. This changeset rectifies most of these issues. - Renamed the crate source files `src/libX/X.rs` to `lib.rs`, for consistency with current styles; this affects extra, rustc, rustdoc, rustpkg, rustuv, std, syntax. - Renamed `X/X.rs` to `X/mod.rs,` as is now recommended style, for `std::num` and `std::terminfo`. - Shifted `src/libstd/str/ascii.rs` out of the otherwise unused `str` directory, to be consistent with its import path of `std::ascii`; libstd is flat at present so it's more appropriate thus. While this removes some `#[path = "..."]` directives, it does not remove all of them, and leaves certain other inconsistencies, such as `std::u8` et al. which are actually stored in `src/libstd/num/` (one subdirectory down). No quorum has been reached on this issue, so I felt it best to leave them all alone at present. #9208 deals with the possibility of making libstd more hierarchical (such as changing the crate to match the current filesystem structure, which would make the module path `std::num::u8`). There is one thing remaining in which this repository is not rustpkg-compliant: rustpkg would have `src/std/` et al. rather than `src/libstd/` et al. I have not endeavoured to change that at this point as it would guarantee prompt bitrot and confusion. A change of that magnitude needs to be discussed first.
- Loading branch information
1 parent
318e1da
commit 0369a41
Showing
12 changed files
with
7 additions
and
10 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
0369a41
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from huonw
at chris-morgan@0369a41
0369a41
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging chris-morgan/rust/filename-consistency = 0369a41 into auto
0369a41
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
chris-morgan/rust/filename-consistency = 0369a41 merged ok, testing candidate = dc079e1
0369a41
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/2307
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-c/builds/153
success: http://buildbot.rust-lang.org/builders/auto-mac-32-nopt-t/builds/153
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/2311
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/1417
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/1416
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/2325
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/1417
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/1417
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/2326
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/1417
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/1417
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/1493
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2311
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/1415
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/1417
success: http://buildbot.rust-lang.org/builders/auto-bsd-64-opt/builds/2089
0369a41
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = dc079e1