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 test/Makefile for FreeBSD shell redirection compatibility #246

Merged
merged 1 commit into from
May 15, 2015

Conversation

jj1bdx
Copy link
Contributor

@jj1bdx jj1bdx commented May 15, 2015

  • FreeBSD shell does not accept &> as the redirect operator;
    Explicitly rewrite &>/dev/null to >/dev/null 2>&1
    as a workaround
  • This will not break other shells (confirmed on OS X 10.10.3)

* FreeBSD shell does not accept `&>` as the redirect operator;
  Explicitly rewrite `&>/dev/null` to `>/dev/null 2>&1`
  as a workaround
@jj1bdx
Copy link
Contributor Author

jj1bdx commented May 15, 2015

See also #244

@essen
Copy link
Member

essen commented May 15, 2015

Looks fine however it seems I have broken something so I will get back to you.

@jj1bdx
Copy link
Contributor Author

jj1bdx commented May 15, 2015

OK. I discovered &> is bash dependent. FYI. You can merge this together with other stuff.

@essen essen merged commit 30dbfd5 into ninenines:master May 15, 2015
@essen
Copy link
Member

essen commented May 15, 2015

Unrelated to your PR so merging, thanks!

Nothing broken per se either, just messages that shouldn't be there.

@jj1bdx
Copy link
Contributor Author

jj1bdx commented May 15, 2015

👍

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.

2 participants