Skip to content

Update most utilities for latest Rust #77

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

Merged
merged 2 commits into from
Feb 6, 2014
Merged

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?

@cnd
Copy link
Contributor

cnd 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.

@cnd
Copy link
Contributor

cnd commented Feb 6, 2014

@Arcterus ok, let me see

cnd added a commit that referenced this pull request Feb 6, 2014
Update most utilities for latest Rust
@cnd cnd 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