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.

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

Fix Makefile and stdbuf #1084

Merged
merged 4 commits into from
Nov 27, 2017
Merged

Fix Makefile and stdbuf #1084

merged 4 commits into from
Nov 27, 2017

Conversation

Arcterus
Copy link
Collaborator

@Arcterus Arcterus commented Oct 9, 2017

Hopefully this should solve issues #1072, #1064, and #1040. It's a little fragile as it depends on the structure of stuff that cargo generates. I also have not tested the build for Macs as I do not have one accessible at the moment.

@cnd
Copy link
Contributor

cnd commented Oct 10, 2017

@Arcterus what with those test faults? I don't remember where from they come

@Arcterus
Copy link
Collaborator Author

I'm not sure at the moment. I am going to check either tomorrow or the day after.

@mssun
Copy link

mssun commented Nov 12, 2017

Any update on the fix?

@Arcterus
Copy link
Collaborator Author

@mssun I've been busy with school for the past month and haven't had much free time. I'm hoping to get this fixed either this or next weekend.

@mssun
Copy link

mssun commented Nov 15, 2017

Good, thank you.

@Arcterus
Copy link
Collaborator Author

AFAICT, the issue is the same as in #1095. It seems running cargo update broke the tests.

@Arcterus
Copy link
Collaborator Author

Everything should be fixed now other than the issues with chmod and install on beta and nightly (which I will try to fix in another pull request).

@Arcterus
Copy link
Collaborator Author

@mpkh I think this can be merged unless you have any suggestions

@cnd cnd merged commit ef4d09e into uutils:master Nov 27, 2017
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.

None yet

3 participants