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.

Sign up for GitHub

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

Jump to bottom

sed doesn't work on MacOS #447

Closed
yegor256 opened this issue Nov 20, 2014 · 6 comments · Fixed by #448
Closed

sed doesn't work on MacOS #447

yegor256 opened this issue Nov 20, 2014 · 6 comments · Fixed by #448
Labels
30 mins bug DEV

Comments

@yegor256
Copy link
Owner

Nginx class is using sed, which works differently on Mac and Linux. Let's fix it by replacing sed with perl or something else
@yegor256 yegor256 mentioned this issue Nov 20, 2014
Closed
@davvd
Copy link
Collaborator

davvd commented Nov 20, 2014

I'll ask someone to take care of this task soon

@davvd
Copy link
Collaborator

davvd commented Nov 20, 2014

@php-coder it's yours, please help. The budget is 30 mins

@davvd
Copy link
Collaborator

davvd commented Nov 20, 2014

thanks for the ticket, your account was topped for 15 mins, payment 000-64055cfc

@php-coder php-coder mentioned this issue Nov 20, 2014
Merged
@php-coder
Copy link
Contributor

I've created pull request #448

@davvd
Copy link
Collaborator

davvd commented Nov 24, 2014

@rultor only @yegor256 can close this issue, since he opened it

@davvd davvd reopened this Nov 24, 2014
@davvd
Copy link
Collaborator

davvd commented Nov 26, 2014

@php-coder Thanks a lot, I just topped your account for 30 mins, transaction ID AP-09T00358U8188150P

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
30 mins bug DEV
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

NginxTest: unbreak unit tests under MacOS with BSD's sed.
3 participants
@php-coder @yegor256 @davvd