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.

Sign up for GitHub

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

Jump to bottom

move std::iter to libextra #7343

Closed
thestinger opened this issue Jun 24, 2013 · 0 comments
Closed

move std::iter to libextra #7343

thestinger opened this issue Jun 24, 2013 · 0 comments
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.

Comments

@thestinger
Copy link
Contributor

This module provides internal iterator adaptors, but internal iterators are being replaced throughout the standard library with external ones.

The Times trait is the only part that's really still useful, so it can stay somewhere in libstd until there's an external iterator replacement.

This should also help avoid confusion by making it obvious that external iterators are going to be the primary way to iterate.
@bors bors closed this as completed in 3fe05a9 Jul 3, 2013
flip1995 pushed a commit to flip1995/rust that referenced this issue Jul 1, 2021
@bors
Auto merge of rust-lang#7380 - popzxc:compile-test-helper, r=flip1995
d3327bd 
Improve panic message on "Found multiple rlibs" error in compile-test

Related to rust-lang#7343

When I first met this error I was pretty much confused, so I thought it may be a good idea to:

- Give a hint on what to do to users that don't want to dig into specifics and just want to quickly resolve the issue.
- Give a link for those who are interested in details.

## Old appearance:

<img width="1121" alt="Screenshot 2021-06-20 at 08 30 34" src="https://user-images.githubusercontent.com/12111581/122663361-df8ae780-d1aa-11eb-9236-775b4fd754d5.png">

## New appearance:

<img width="1121" alt="Screenshot 2021-06-20 at 08 32 18" src="https://user-images.githubusercontent.com/12111581/122663363-e4e83200-d1aa-11eb-9c46-f62d83eb79e2.png">

*Please write a short comment explaining your change (or "none" for internal only changes)*

changelog: none
flip1995 pushed a commit to flip1995/rust that referenced this issue Sep 8, 2021
@bors
Auto merge of rust-lang#7631 - camsteffen:depinfo, r=flip1995
27afd6a 
Use binary-dep-depinfo to resolve UI test dependencies

Closes rust-lang#7343
Closes rust-lang#6809
Closes rust-lang#3643

changelog: none

r? `@flip1995`
cc `@Jarcho`
flip1995 pushed a commit to flip1995/rust that referenced this issue Sep 28, 2021
@bors
Auto merge of rust-lang#7646 - camsteffen:relative-target, r=flip1995
b556398 
Target directory cleanup

changelog: none

* .cargo/config now has `target-dir` specified so that it is inherited by child projects. The target directory needs to be shared with clippy_dev, but not necessarily at the project root. (cc rust-lang#7625)
* Uses `std::env::current_exe` (and its parent directories) whenever possible
* `CLIPPY_DRIVER_PATH` and `TARGET_LIBS` are no longer required from rustc bootstrap (but `HOST_LIBS` still is). These can be removed from the rustc side after merging.
* `CLIPPY_DOGFOOD` and the separate target directory are removed. This was originally added to mitigate rust-lang#7343.

r? `@flip1995`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thestinger