-
Notifications
You must be signed in to change notification settings - Fork 32
[make] full implementation according specification #160
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
Conversation
348f359 to
a7e977e
Compare
6d45c76 to
a347ee7
Compare
2cd17dd to
6c7862c
Compare
|
@jgarzik ready for review |
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.
Mostly looks OK. Some minor changes.
m4/tests/integration_test.rs
Outdated
| fn test_bsd() { | ||
| init(); | ||
| let output = run_command(Path::new("fixtures/integration_tests/bsd.m4")); | ||
| let output = run_command(&Path::new("fixtures/integration_tests/bsd.m4")); |
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.
merge error. This PR should not touch m4/
|
|
||
| let stderr = String::from_utf8_lossy(&output.stderr); | ||
| assert_eq!(stderr, plan.expected_err); | ||
|
|
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.
Change rejected -- This change deletes some tests across all utilities.
|
@jgarzik took into account. is there anything? |
No description provided.