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

on some systems we need to change LANGUAGE variable #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wodabr
Copy link

@wodabr wodabr commented Oct 8, 2014

Hi,

thanks for excellent pice of software :)

I've spend hours to figure out why this won't work on my box, it seems that on some systems we also need to change LANGUAGE variable - (someone had similar problem - http://php.net/manual/en/ref.gettext.php#29799)

@netson
Copy link
Owner

netson commented Oct 8, 2014

Hey! Thanks for the pull request; on what systems have you tested this modification? I want to make sure this doesn't throw any errors for existing users.

@wodabr
Copy link
Author

wodabr commented Oct 8, 2014

I've tested this on

  1. CentOS release 6.5 - PHP 5.5.13 - lighttpd/1.4.34
  2. Ubuntu 12.04.4 LTS - PHP 5.5.12-2 - nginx/1.6.0

@netson
Copy link
Owner

netson commented Oct 8, 2014

And did you have the issue on both systems as well? Do you know what causes this behavior? I scanned though the comments on the PHP website from your link, but couldn't really determine which conditions lead to this issue.

I'll run some tests on my box this weekend with your code, and pull in your changes if all works well! Thanks for contributing! 👍

@wodabr
Copy link
Author

wodabr commented Oct 8, 2014

nope, I forgot to mention that only centos is giving me hard time :) on my dev box (ubuntu) everything works fine. and no I don't know root cause of this behavior ;(

@netson
Copy link
Owner

netson commented Oct 8, 2014

Alright, thanks for the info. I usually work on Ubuntu boxes as well (12.04 and 14.04) so I'll see if I can whip up a virtual CentOS box to do some testing! Keep you posted! :)

@netson
Copy link
Owner

netson commented Oct 16, 2014

Hey, haven't forgotten about this issue, I've just been crazy busy. Hope to get around to it this weekend. Sorry for the delay!

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