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

Merging does not work on OSX #3

Closed
themightyoarfish opened this issue Nov 29, 2017 · 1 comment
Closed

Merging does not work on OSX #3

themightyoarfish opened this issue Nov 29, 2017 · 1 comment

Comments

@themightyoarfish
Copy link

themightyoarfish commented Nov 29, 2017

Running php m4b-tool.phar merge on 10.12 and PHP 7.2 results in:

  [Exception]
  DateTime::__construct(): It is not safe to rely on the system's timezone settings. You are *required* to use the date.time
  zone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting
  this warning, you most likely misspelled the timezone identifier. We selected the timezone 'UTC' for now, but please set d
  ate.timezone to select your timezone.
@sandreas
Copy link
Owner

sandreas commented Dec 1, 2017

Thank you for reporting. This is because you did not set a timezone in your php.ini (which is needed). I did not ran into this, because my php is configured correctly, but i confirm this as an enhancement, because i did not leave a hint in the documentation.

As a temporary workaround i recommend to set date.timezone=UTC in php.ini (you could also use your real timezone, like date.timezone=Europe/Berlin) or you could try to set the configuration property inline:

php -d "date.timezone=UTC" m4b-tool.phar merge

This should also work.

Let me know if there is something else i can do for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants