Skip to content

Update most utilities for latest Rust#77

Merged
Cynede merged 2 commits intouutils:masterfrom
Arcterus:master
Feb 6, 2014
Merged

Update most utilities for latest Rust#77
Cynede merged 2 commits intouutils:masterfrom
Arcterus:master

Conversation

@Arcterus
Copy link
Collaborator

@Arcterus Arcterus commented Feb 5, 2014

This updates all the utilities other than tee and those that were covered by @Heather.

@Arcterus
Copy link
Collaborator Author

Arcterus commented Feb 5, 2014

I'm going to allow unused_must_use, at least for the time being, so that there won't be all of the warnings due to the writeln! statements.

@nutsi
Copy link

nutsi commented Feb 5, 2014

Right now, tee and wc don't build. I thik it's the same kind of errors for both (io_error doesn't exist anymore) .
I talk about dev branch

@Arcterus
Copy link
Collaborator Author

Arcterus commented Feb 5, 2014

I know. @Heather's commit was doing stuff to wc, but I'll fix it. tee looked like it's going to require somewhat large changes, so I haven't done it yet.

@Arcterus
Copy link
Collaborator Author

Arcterus commented Feb 6, 2014

@Heather can you merge this one too?

@Cynede
Copy link
Contributor

Cynede commented Feb 6, 2014

@Arcterus you mean merge with my fork?

@Arcterus
Copy link
Collaborator Author

Arcterus commented Feb 6, 2014

No, with this (uutils/coreutils). We're supposed to have other people review our changes before merging.

@Cynede
Copy link
Contributor

Cynede commented Feb 6, 2014

@Arcterus ok, let me see

Cynede added a commit that referenced this pull request Feb 6, 2014
Update most utilities for latest Rust
@Cynede Cynede merged commit e4fa660 into uutils:master Feb 6, 2014
@Arcterus
Copy link
Collaborator Author

Arcterus commented Feb 6, 2014

Thanks. 👍

jbcrail pushed a commit to jbcrail/coreutils that referenced this pull request Apr 29, 2015
Update most utilities for latest Rust
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 this pull request may close these issues.

3 participants