-
Notifications
You must be signed in to change notification settings - Fork 30
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
Update Dockerfile TZ env variable #182
base: master
Are you sure you want to change the base?
Conversation
Makes Dockerfile to use the env variable from compose.yml file to set the timezone of the container, as it is it uses UTC even if TZ is set in to a different timezone.
I see no issue with change. Two questions: |
For what I've read of the code Line 116 in cf3dc4d
As for the Dockerfile, it depends on cabernet github's configured workflow to create the docker image,
|
catch invalid date format errors to prevent crashing.
I updated the pull request with another case that I found on processing Dates for EPG, using the same sourceas before: epg.txt, like last time I found a date with a weird format: 2000|2001|2002|2003 it crashes the xmltv.py and prevents the rest of the epg data to be processed and aded to cabernet, so I added some code to catch the error and decided to use today's date as program date, it can be whatever you want it only prevents the crashing. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Need to incorporate this into the current set of changes. |
Makes Dockerfile use the TZ env variable from compose.yml file to set the timezone of the container, as it is it uses UTC even if that variable is set to a different timezone.